forked from microbit-foundation/cctd-ml-machine
-
Notifications
You must be signed in to change notification settings - Fork 4
Update onboarding with nudges and new welcome dialog #629
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
Open
microbit-grace
wants to merge
34
commits into
beta
Choose a base branch
from
onboarding-update
base: beta
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Upgrade ml-trainer-microbit package * Add a placeholder homepage-short-clip.mp4 for theme
Squashed commit of the following: commit a22fe19 Author: Grace <grace@microbit.org> Date: Mon Nov 3 12:56:22 2025 +0000 Add press button B diagram to record hint commit 8ce9561 Author: Grace <grace@microbit.org> Date: Mon Nov 3 12:09:10 2025 +0000 Remove unused import commit 4cc37ef Author: Grace <grace@microbit.org> Date: Mon Nov 3 12:07:48 2025 +0000 Disable data samples table when move microbit hint is shown commit d08bbe6 Author: Grace <grace@microbit.org> Date: Wed Oct 29 11:47:41 2025 +0000 Tweak graph nudge copy commit f6835af Author: Grace <grace@microbit.org> Date: Wed Oct 29 11:44:46 2025 +0000 Fix video autoplay commit 4a713e8 Author: Grace <grace@microbit.org> Date: Thu Oct 23 17:47:16 2025 +0100 Replace pre-connect placeholder animation commit 5cf370f Author: Grace <grace@microbit.org> Date: Mon Oct 20 15:19:58 2025 +0100 Prettier commit 41772b9 Author: Grace <grace@microbit.org> Date: Mon Oct 20 15:15:54 2025 +0100 Tweak copy commit ddc5c19 Author: Grace <grace@microbit.org> Date: Mon Oct 20 15:13:26 2025 +0100 Tweak assistant hint for live graph commit 360b9cb Author: Matt Hillsdon <matt.hillsdon@microbit.org> Date: Fri Oct 17 16:46:50 2025 +0100 Whoops commit b1f80b0 Author: Matt Hillsdon <matt.hillsdon@microbit.org> Date: Fri Oct 17 16:31:05 2025 +0100 Logical rather than hacky timing commit 1240503 Author: Matt Hillsdon <matt.hillsdon@microbit.org> Date: Fri Oct 17 16:22:20 2025 +0100 Sane hint logic, animate Train model. commit a9dbf5d Author: Matt Hillsdon <matt.hillsdon@microbit.org> Date: Fri Oct 17 14:53:30 2025 +0100 Works commit 60f0278 Author: Matt Hillsdon <matt.hillsdon@microbit.org> Date: Fri Oct 17 14:26:39 2025 +0100 Lint commit 2298534 Author: Matt Hillsdon <matt.hillsdon@microbit.org> Date: Fri Oct 17 14:20:59 2025 +0100 Better wait for move logic commit fc91b8f Author: Matt Hillsdon <matt.hillsdon@microbit.org> Date: Fri Oct 17 13:58:35 2025 +0100 Move micro:bit commit 697e905 Author: Matt Hillsdon <matt.hillsdon@microbit.org> Date: Fri Oct 17 13:38:14 2025 +0100 WIP commit cdabb08 Author: Matt Hillsdon <matt.hillsdon@microbit.org> Date: Fri Oct 17 13:21:53 2025 +0100 Better animation commit 90af0fb Author: Matt Hillsdon <matt.hillsdon@microbit.org> Date: Fri Oct 17 12:27:12 2025 +0100 Decompose the emoji and add support for tick eyes. commit 257eb41 Merge: f266fb1 7f0d52d Author: Matt Hillsdon <matt.hillsdon@microbit.org> Date: Fri Oct 17 10:46:33 2025 +0100 Merge branch 'emoji-split' into alt-flow-2 commit 7f0d52d Author: Matt Hillsdon <matt.hillsdon@microbit.org> Date: Fri Oct 17 10:39:22 2025 +0100 Break down the eyes commit f266fb1 Merge: a9949ff 49b495a Author: Matt Hillsdon <matt.hillsdon@microbit.org> Date: Fri Oct 17 10:04:40 2025 +0100 Merge branch 'main' into alt-flow-2 commit 0b51ae9 Author: Matt Hillsdon <matt.hillsdon@microbit.org> Date: Tue Oct 14 11:11:39 2025 +0100 WIP commit a9949ff Author: Matt Hillsdon <matt.hillsdon@microbit.org> Date: Fri Oct 10 16:47:58 2025 +0100 Have the first action in place always commit 30ee2b3 Author: Matt Hillsdon <matt.hillsdon@microbit.org> Date: Fri Oct 10 15:14:09 2025 +0100 Lint commit e0ec830 Author: Matt Hillsdon <matt.hillsdon@microbit.org> Date: Fri Oct 10 15:11:23 2025 +0100 A minimal version of the skeleton idea commit 9b49d24 Author: Matt Hillsdon <matt.hillsdon@microbit.org> Date: Fri Oct 10 14:58:58 2025 +0100 Clearer intentions commit a1e45b6 Author: Matt Hillsdon <matt.hillsdon@microbit.org> Date: Fri Oct 10 14:52:16 2025 +0100 Remove debug commit f72e022 Author: Matt Hillsdon <matt.hillsdon@microbit.org> Date: Fri Oct 10 14:51:35 2025 +0100 Experiment with a non-tour prompt to move the micro:bit commit a738e60 Author: Matt Hillsdon <matt.hillsdon@microbit.org> Date: Fri Oct 10 13:49:52 2025 +0100 animation experiment; enable train model always commit c420a9b Author: Matt Hillsdon <matt.hillsdon@microbit.org> Date: Fri Oct 10 13:30:30 2025 +0100 Lint commit 26f18e6 Author: Matt Hillsdon <matt.hillsdon@microbit.org> Date: Fri Oct 10 13:25:16 2025 +0100 Skip e2e commit 25ae243 Author: Matt Hillsdon <matt.hillsdon@microbit.org> Date: Fri Oct 10 13:24:57 2025 +0100 Better commit f9503f3 Author: Matt Hillsdon <matt.hillsdon@microbit.org> Date: Fri Oct 10 12:32:29 2025 +0100 WIP
- Don't trigger move nudge when navigating back to data samples - Suppress triggering train or add action nudge when importing hex file, json file, or projects - Suppress triggering train or add action nudge when loading data samples page - Always show placeholder first action even when importing hex/json without data samples or actions
|
Preview build will be at |
Refactor to DRY connect first dialog logic
microbit-grace
commented
Nov 14, 2025
| env: | ||
| NODE_AUTH_TOKEN: ${{ secrets.GITHUB_TOKEN }} | ||
| - run: npm install --no-save @microbit-foundation/ml-trainer-microbit@0.2.0-dev.75 @microbit-foundation/website-deploy-aws@0.6 @microbit-foundation/website-deploy-aws-config@0.10 | ||
| - run: npm install --no-save @microbit-foundation/ml-trainer-microbit@0.2.0-dev.78 @microbit-foundation/website-deploy-aws@0.6 @microbit-foundation/website-deploy-aws-config@0.10 |
Author
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.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Also replaces empty data samples page state with an initial placeholder action.
Testing
Reduced motion preference can be tested on iOS Settings via Accessibility > Display > Toggle 'Reduce motion'