Closed Bug 1029528 Opened 10 years ago Closed 10 years ago

Display "Thanks for signing up" modal on party.wmkr.org after conversion from welcome.wmkr.org

Categories

(Webmaker Graveyard :: webmaker.org, defect)

x86
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: jon, Assigned: kate)

References

Details

(Whiteboard: [snippet] [june27])

Attachments

(2 files, 1 obsolete file)

After a user signs up on welcome.wm.o, we should present a "Thanks for signing up" modal on party.wm.o.
Summary: Have some sort of "Thanks for signing up" modal on paty.wm.o after conversion from welcome.wm.o → Display "Thanks for signing up" modal on party.wmkr.org after conversion from welcome.wmkr.org
* This one's blocking final QA.
Flags: needinfo?(jon)
Kate's on it
Assignee: jon → kate
Flags: needinfo?(jon)
Attachment #8445485 - Flags: review?(jon)
Attachment #8445488 - Flags: review?(andrea)
Attachment #8445485 - Flags: review?(jon) → review+
This got backed out in https://github.com/mozilla/makerparty/pull/27/files due to it disabling all click handlers on the page
This is the fixed version of the modal
Attachment #8445485 - Attachment is obsolete: true
Attachment #8445867 - Flags: review?(mavis)
Hi, can you let me know ETA on this going live? The Firefox snippet team is at-the-ready to deploy our snippet tests. We need to get this up, working, and QA the user flow still today. Thanks for your help!
PR is merged.  Pushing it to production now.  Will update the bug once it's live.
Invalidating / and /js/ui.js now, should be live in 15 minutes.
hey guys, i just tried it out on Firefox and the screen darkened slightly, but no modal. Can you QA, fix, and make sure it's working? This has delayed us over 24 hrs.....
Current status: I cannot for the life of me invalidate the root from the CDN. I'm currently seeing if I can switch party.webmaker.org to point directly at Heroku
Current status: I am dumb, and I should've tried this before, but with mavis' help we are golden!

https://party.webmaker.org/?thanks=mavis
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Great! I'm doing some final QA now and alerting JP that he should watch load on that page. Thanks everyone!
Attachment #8445867 - Flags: feedback+
Attachment #8445867 - Flags: review?(mavis)
Attachment #8445867 - Flags: review+
Attachment #8445867 - Flags: feedback+
Attachment #8445488 - Flags: review?(andrea)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: