Skip to content
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

micro:bit on Chromebooks #37815

Merged
merged 13 commits into from Dec 1, 2020
Merged

micro:bit on Chromebooks #37815

merged 13 commits into from Dec 1, 2020

Conversation

epeach
Copy link

@epeach epeach commented Nov 12, 2020

Getting micro:bit working on Chromebooks. Similar to the Circuit Playgrounds on Chromebook, we use chrome-serialport to communicate with the boards. chrome-serialport does not use promises like most of our code in this area, so there are some artifacts of working around that in the changes here.

Added JIRA tasks for: micro:bit celebration (non-blocking), getting micro:bit firmata/firmware version (non-blocking (and might not be worth investing in depending on our Chrome App deprecation plan) ), and getting micro:bit board version (non-block (also should depend on deprecation plan) ).

I have an adhoc for this branch to test on Chromebooks. LMK if you want me to turn it on to test.

Screenshot of MB setup in Maker App on Linux:
Screenshot from 2020-11-11 16-36-18

Screenshot of MB setup in Chrome on Chromebook:
chromeapp

Reviewer Checklist:

  • Tests provide adequate coverage
  • Privacy and Security impacts have been assessed
  • Code is well-commented
  • New features are translatable or updates will not break translations
  • Relevant documentation has been added or updated
  • User impact is well-understood and desirable
  • Pull Request is labeled appropriately
  • Follow-up work items (including potential tech debt) are tracked and linked

@epeach epeach requested a review from a team November 12, 2020 17:59
@@ -1,4 +1,7 @@
/** @file Some misc. browser check methods for maker */
/* global SerialPort */ // Maybe provided by the Code.org Browser
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are we not sure?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh I've never read it like that! We are sure that if we're in the Code.org (Maker) Browser, then we have the SerialPort. I believe the 'Maybe' applies to that we might be in the Code.org Browser or in the Chrome Browser with the Chrome App (Chromebook).

Copy link
Member

@breville breville left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know this code very well, but this seems well thought-through!

@epeach epeach merged commit 72f732e into staging Dec 1, 2020
@epeach epeach deleted the chromeserialport-microbit branch December 1, 2020 21:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants