Add confetti to certificate congrats page #34460
Merged
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.
Follow-up to #34451
Adds some confetti to the certificate congrats page upon personalization!
Open Question
Currently, this only affects the page at studio.code.org/congrats, not the page at code.org/congrats. It's not super clear to me why we still have both of these, but the latter one doesn't currently support React. Rather than trying to add this functionality to that page, I'd suggest the best thing to do would be to just combine these two pages into one. That would of course be more work in the short term than would just adding this feature to both pages, but would of course be better in the long term.
So, are there any objections or concerns to just adding this confetti to one of the congrats pages for now?
Testing story
Tested manually, relying on existing unit tests for long-term testing.
Reviewer Checklist: