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)
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.
Updated•10 years ago
|
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
Assignee | ||
Comment 3•10 years ago
|
||
Attachment #8445485 -
Flags: review?(jon)
Assignee | ||
Comment 4•10 years ago
|
||
Attachment #8445488 -
Flags: review?(andrea)
Comment 5•10 years ago
|
||
Commit pushed to master at https://github.com/mozilla/makerparty https://github.com/mozilla/makerparty/commit/efa2880feee89a956e19fd43d1fb2859ce79f8a0 Bug 1029528 - Modal for thanks for signing up
Reporter | ||
Updated•10 years ago
|
Attachment #8445485 -
Flags: review?(jon) → review+
Reporter | ||
Comment 6•10 years ago
|
||
This got backed out in https://github.com/mozilla/makerparty/pull/27/files due to it disabling all click handlers on the page
Comment 7•10 years ago
|
||
Comment on attachment 8445488 [details] http://recordit.co/RVHrPrxbrf.gif This looks good!
Assignee | ||
Comment 8•10 years ago
|
||
This is the fixed version of the modal
Attachment #8445485 -
Attachment is obsolete: true
Attachment #8445867 -
Flags: review?(mavis)
Comment 9•10 years ago
|
||
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!
Comment 10•10 years ago
|
||
Commit pushed to master at https://github.com/mozilla/makerparty https://github.com/mozilla/makerparty/commit/b9586293062aa8ee2cbecebec217cfe38ee9d71e Bug 1029528 - Modal for thanks for signing up (with fixes)
Comment 11•10 years ago
|
||
PR is merged. Pushing it to production now. Will update the bug once it's live.
Reporter | ||
Comment 12•10 years ago
|
||
Invalidating / and /js/ui.js now, should be live in 15 minutes.
Comment 13•10 years ago
|
||
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.....
Reporter | ||
Comment 14•10 years ago
|
||
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
Reporter | ||
Comment 15•10 years ago
|
||
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
Comment 16•10 years ago
|
||
Great! I'm doing some final QA now and alerting JP that he should watch load on that page. Thanks everyone!
Updated•10 years ago
|
Attachment #8445867 -
Flags: feedback+
Updated•10 years ago
|
Attachment #8445867 -
Flags: review?(mavis)
Attachment #8445867 -
Flags: review+
Attachment #8445867 -
Flags: feedback+
Assignee | ||
Updated•10 years ago
|
Attachment #8445488 -
Flags: review?(andrea)
You need to log in
before you can comment on or make changes to this bug.
Description
•