860460 - Port /newsletter to Bedrock

Status: VERIFIED FIXED

(www.mozilla.org :: Newsletters, defect, P2)

Comment 1

[Dan Poirier [:dpoirier]]

FYI, on the current site, when you submit the signup form on the /newsletter page, you end up at /newsletter/new. Jess Davis had passed along earlier that we'll be getting rid of the /new page with this port to Bedrock, and just display a thanks message at the same URL.

Comment 2

[Dan Poirier [:dpoirier]]

Pull request is open here: https://github.com/mozilla/bedrock/pull/950

Comment 3

[Paul [:pmac] McLanahan]

Ran into another issue. We'll get it in the morning and have it ready for testing when you're ready tomorrow.

Comment 4

[Jessilyn Davis]

Adding rbillings to know when/where this will be ready to test.

Comment 5

[Paul [:pmac] McLanahan]

Let the testing commence!

https://www-demo1.allizom.org/newsletter/

Comment 6

[Rebecca Billings [:rbillings]]

Verified site/language display/email/welcome on demo1 for en-US, en-GB, pl, pt-BR, es-AR

Comment 7

[[github robot]]

Commit pushed to master at https://github.com/mozilla/bedrock

https://github.com/mozilla/bedrock/commit/7b65a3c9b47ba7a94ed39375419dd689c838262a
Bug 860460: Port /newsletter page to bedrock

Comment 8

[Rebecca Billings [:rbillings]]

Verified

Comment 9

[Jessilyn Davis]

Bug 892696 needs to be fixed before this can go live.

Comment 10

[Jessilyn Davis]

Looks and works great! :D  Thanks team!

Comment 11

[Rebecca Billings [:rbillings]]

Verified