798531 - String change to "Beta News" on /existing/ and /new/ l10n pages

Status: RESOLVED FIXED

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

Description

[Jessilyn Davis]

Attachment: Screenshot of example contact

mozilla.org/newsletter/existing/ logic needs to be updated so that:

If someone is subscribed to the Beta newsletter = they see the Beta newsletter
If someone's language is English = they see the Beta newsletter
If someone is not subscribed to the Beta newsletter, and their language is not English = they should not see the Beta newsletter.

Example user (my testing address):
https://www.mozilla.org/es-ES/newsletter/existing/6d24dfa2-388a-4e02-8863-d4033a197cab/

This user should not be able to see the "Noticias Beta" option

Comment 1

[Jessilyn Davis]

Better/easier:

Let's leave "beta" on the list for l10n /existing/ pages - and add "(English only)" as part of the title of the Beta newsletter.

That means it's a string change on this page rather than technical change.

Comment 2

[Jessilyn Davis]

"Beta News" newsletter title should read "Beta News (English Only)" on both /newsletter/new and /newsletter/existing pages.

Comment 3

[Jessilyn Davis]

No longer need to update /new page - as we're getting rid of it as part of Bedrock migration (Bug 845904).

The following newsletters need (English only) added to the end of their title as a precaution:

MOZILLA_AND_YOU
ABOUT_MOBILE
FIREFOX_TIPS
FIREFOX_BETA_NEWS
AURORA
ABOUT_MOZILLA
STUDENT_REPS
STUDENT_AMBASSADORS (when created)
MOZILLA_PHONE
APP_DEV
AFFILIATES
FLICKS

Comment 4

[Dan Poirier [:dpoirier]]

Someone with admin access to basket can make these changes anytime, and they'll show up on /existing and /updated once 845904 (the /existing changes) goes live.

Is it worth editing these on the PHP side or should we just keep working on migrating over the newsletter-related pages from there to bedrock?

Comment 5

[Jessilyn Davis]

(In reply to Dan Poirier [:dpoirier] from comment #4)
> Someone with admin access to basket can make these changes anytime, and
> they'll show up on /existing and /updated once 845904 (the /existing
> changes) goes live.

Awesome - I just got access from pmac and can update this after a tour next week.

> 
> Is it worth editing these on the PHP side or should we just keep working on
> migrating over the newsletter-related pages from there to bedrock?

I'm unsure on this as I'm not 100% sure of the complexity of PHP vs. bedrock - malexis can you advise? 

My 2 cents: It would be nice to have these description on the l10n pages of /existing (since the "English Only" blurb will really only apply to them as a warning).

Comment 6

[Jessilyn Davis]

Looks great with Prod release today 6/5