Add id and ru newsletter to the list of newsletters with double confirmation on signup

RESOLVED FIXED in Future

Status

RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: rik, Assigned: rik)

Tracking

unspecified
Future
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: u=dev c=email p=1)

Comment hidden (empty)
(Assignee)

Comment 1

6 years ago
Pull Request is at https://github.com/mozilla/basket/pull/3
(Assignee)

Comment 2

6 years ago
Jess: In bug 745011 comment 8, you mention that the key for RU should be ru_confirmation_email_2. That's a difference with other keys. You can see the current list at https://github.com/mozilla/basket/blob/master/apps/news/tasks.py#L25

Can you confirm we really want this "_2" ?
(Assignee)

Comment 3

6 years ago
Winston: Can you help out with comment 2?

Comment 4

6 years ago
Hi Anthony, 

I believe what Jess mentioned in 745011 is correct.  The external key for that particular email does contain the _2.  It reads ru_confirmation_email_2

Updated

6 years ago
Whiteboard: u=dev c=email
Target Milestone: --- → Future

Comment 5

6 years ago
"ru_confirmation_email_2" is the correct key for the Russian newsletter welcome email.

Can you give a timeline ETA for implementation?

Thanks!

Jess

Comment 6

6 years ago
Hey Anthony, Welcome back from vacation!  I hope you had a lovely week.

Checking in on this Bug for Q3: does Sept 27th for implementation sound achievable?
(Assignee)

Updated

6 years ago
Blocks: 792098
(Assignee)

Comment 7

6 years ago
The code is ready. Opened bug 792098 to get this deployed.

Comment 8

6 years ago
Anthony, can you let me know when the links are live on Stage? I'd like to run them through tests before they go live.

Thank you.
(Assignee)

Updated

6 years ago
No longer blocks: 792098
Depends on: 792098
(Assignee)

Updated

6 years ago
Whiteboard: u=dev c=email → u=dev c=email p=1
(Assignee)

Comment 9

6 years ago
Closing as the backend work has been done.

We now have to deploy the content of bug 745011 and bug 745032 after tests are positive.
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.