Skip to content

Comments

fix(docs-infra): support comparing reveal answer with user work-in-progress#67240

Open
LMazilu wants to merge 1 commit intoangular:mainfrom
LMazilu:fix/tutorial-reveal-restore-wip
Open

fix(docs-infra): support comparing reveal answer with user work-in-progress#67240
LMazilu wants to merge 1 commit intoangular:mainfrom
LMazilu:fix/tutorial-reveal-restore-wip

Conversation

@LMazilu
Copy link

@LMazilu LMazilu commented Feb 24, 2026

Preserve learner work in progress when revealing answers and add a dedicated action to return to personal code without losing the default reset behavior. Fixes #52656.

@pullapprove pullapprove bot requested a review from AndrewKushnir February 24, 2026 10:19
@angular-robot angular-robot bot added the area: docs-infra Angular.dev application and infrastructure label Feb 24, 2026
@ngbot ngbot bot added this to the Backlog milestone Feb 24, 2026
@LMazilu
Copy link
Author

LMazilu commented Feb 24, 2026

Local validation completed:

  • pnpm exec bazelisk test //adev:test --jobs=1 --test_output=errors passed.
    • Manual behavior verified in tutorials:
      • Reveal Answer shows the solution.
      • Back to My Code restores the learner work-in-progress.
      • Reset restores the tutorial’s initial state.

Waiting for workflow approval and review feedback.

@angular angular deleted a comment from google-cla bot Feb 24, 2026
@JeanMeche JeanMeche requested review from JeanMeche and removed request for AndrewKushnir February 24, 2026 10:47
…ogress

Preserve learner work in progress when revealing answers and add a dedicated action to return to personal code without losing the default reset behavior. Fixes angular#52656.
@LMazilu LMazilu force-pushed the fix/tutorial-reveal-restore-wip branch from cee74ba to ffdd167 Compare February 24, 2026 11:42
@LMazilu
Copy link
Author

LMazilu commented Feb 24, 2026

Rebased on latest main; checks are pending maintainer workflow approval.

@JeanMeche
Copy link
Member

Could you rebase on main again, we had an infra issue that we just fixed that was responsible for the build errors you see.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

adev: preview area: docs-infra Angular.dev application and infrastructure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

docs: Allow users to compare answers to work in progress code in Tutorials

2 participants