-
Notifications
You must be signed in to change notification settings - Fork 4
Dialog warnings for using MakeCode with a V1 #358
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Preview build will be at |
|
Some changes to messages/ui.en.json are the result of running the i18n compile step and not directly related to this PR. |
| // Open editor if device is not a V1, otherwise show warning dialog. | ||
| const bluetoothConnection = getBluetoothConnection(); | ||
| if (bluetoothConnection) { | ||
| if (bluetoothConnection.getBoardVersion() === "V1") { | ||
| return onOpen(); | ||
| } | ||
| } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might be nice to hide the bluetooth-specific nature of this behind something in the hook as we might do V1 radio bridge at some point.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done here. A bit more work to hold on to the radio remote board version 5dc85ea
radioRemoteBoardVersion is currently not cleared if you go back to the home page and start a new session. It's only cleared on a page refresh.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We're going to ponder this on a separate ticket.
This should also work for the remote device when using radio bridge.
|
Just waiting on copy feedback now, otherwise LGTM. |
|
Copy feedback not likely in time for beta, gone with best guess and will tweak later. |
No description provided.