Skip to content

Promote Dance 2019 on /congrats #32256

Merged
Erin007 merged 4 commits into
stagingfrom
promo-dp-on-congrats
Dec 4, 2019
Merged

Promote Dance 2019 on /congrats #32256
Erin007 merged 4 commits into
stagingfrom
promo-dp-on-congrats

Conversation

@Erin007

@Erin007 Erin007 commented Dec 3, 2019

Copy link
Copy Markdown
Contributor

A few changes to the promotional card sets on studio.code.org/congrats:

1.) We've been sending people to last year's dance party, so I updated the link to /s/dance-2019/reset

2.) Signed in users in English, now regardless of age, who complete an 'other' tutorial (anything except the Minecrafts, Dance, Dance 2019 or AppLab) will now see Dance 2019 instead of AppLab
Screen Shot 2019-12-03 at 1 54 14 PM

3.) Signed out users who complete an 'other' tutorial will now see Dance 2019 instead of AppLab
Screen Shot 2019-12-03 at 1 52 56 PM

@breville breville left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Assuming this means that all links to dance-2019 now have updated images, then this is great.

@ghost

ghost commented Dec 4, 2019

Copy link
Copy Markdown

@Erin007 i like change #1 and change #3, but not sure about #2. for students above 13, we wanted to advertise app lab. is it possible to keep that app lab in that scenario?

@Erin007

Erin007 commented Dec 4, 2019

Copy link
Copy Markdown
Contributor Author

@Erin007 i like change #1 and change #3, but not sure about #2. for students above 13, we wanted to advertise app lab. is it possible to keep that app lab in that scenario?

Yes. Updated so that signed in users over 13 will now see:
Screen Shot 2019-12-04 at 11 33 12 AM

@maddiedierker maddiedierker self-requested a review December 4, 2019 19:38
@Erin007 Erin007 merged commit a3ae00e into staging Dec 4, 2019
@Erin007 Erin007 deleted the promo-dp-on-congrats branch December 4, 2019 20:18
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.

3 participants