Skip to content

Update deprecating maker app message depending on browser - #51176

Merged
fisher-alice merged 1 commit into
stagingfrom
alice/update-maker-setup-non-chromium
Apr 7, 2023
Merged

Update deprecating maker app message depending on browser#51176
fisher-alice merged 1 commit into
stagingfrom
alice/update-maker-setup-non-chromium

Conversation

@fisher-alice

@fisher-alice fisher-alice commented Apr 6, 2023

Copy link
Copy Markdown
Contributor

This PR customizes the Maker App Download message on the 'maker/setup' page based on whether the user is on a chromium browser (Chrome, Edge) which supports WebSerial.

Currently, if a user is on a non-Chromium browser, they don't see the WebSerial option but the message above the 'Download Code.org Maker App' button refer to 'webSerial (instructions above)'. The user is also not informed to use a Chromium browser instead to be able to connect directly with their board. See screenshot below

BEFORE update:
Screen Shot 2023-04-06 at 2 26 14 PM

Now if a user is on a non-Chromium browser, they will see the following message: 'We highly recommend using the Chrome or Edge web browser so that you can directly connect to your Circuit Playground or micro:bit using WebSerial. Because your current browser does not support directly connecting to your board, you can download the Maker App as an alternate way to use the Maker Toolkit.'

Whether the user is on a Chromium or non-Chromium browser, they will see a message that support the Maker app will be ending in June 2023 with. a link to the support article about the deprecation of the app.

Video AFTER update:

deprecate-maker.mp4

Links

Slack discussion

Testing story

Deployment strategy

Follow-up work

Privacy

Security

Caching

PR 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

@fisher-alice
fisher-alice marked this pull request as ready for review April 6, 2023 19:33
@fisher-alice
fisher-alice requested a review from a team as a code owner April 6, 2023 19:33
@fisher-alice
fisher-alice requested review from a team and dancodedotorg April 6, 2023 19:33

@molly-moen molly-moen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for updating this!

@fisher-alice
fisher-alice merged commit e06361d into staging Apr 7, 2023
@fisher-alice
fisher-alice deleted the alice/update-maker-setup-non-chromium branch April 7, 2023 02:05
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.

2 participants