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
[Don't merge until DP Launch] Congrats: swap Dance Party for Minecraft #25754
Conversation
@@ -813,9 +815,6 @@ | |||
"minecraftAquaticButton": "Go to Minecraft Education", | |||
"minecraftAquaticDesc": "Continue learning with Minecraft.", | |||
"minecraftAquaticTitle": "Minecraft Education", | |||
"minecraftMarketingButton": "Start", | |||
"minecraftMarketingDesc": "Minecraft is back for the Hour of Code with a brand new activity! Journey through Minecraft with code.", | |||
"minecraftMarketingTitle": "Minecraft: Hero’s Journey", |
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.
i'm sure you checked, but just double-checking that these "minecraft_" strings aren't used elsewhere
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.
Yep, thanks for the reminder!
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.
💃 left an inline question, but everything looks great!
Heads up @breville and @madelynkasula I just updated this PR to include the final copy (hooray!) for the Dance Party and Dance Party follow-up cards. |
…code-dot-org into u13-congrats-swap-in-dance
One last tweak to address issue: code-dot-org/dance-party#295 |
The last of this year's /congrats changes outline in the spec.
Students younger than 13 who finish tutorials other than Dance, one of the Minecrafts or AppLab, will see a card promoting Dance Party rather than Minecraft Hero.
BEFORE:
AFTER: