Closed Bug 1255123 Opened 8 years ago Closed 8 years ago

Remove Marketplace from Email Pref Center

Categories

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

Production
defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: jdavis, Assigned: kohei)

Details

Attachments

(2 files)

Attached image Marketplace_EPC.png
Last year some (lovely) custom work was done on the Email Preference Center to accommodate our email "interest" preferences: Bug 1087579

With the recent change in direction with Marketplace and other marketing priorities, we're no longer supporting Marketplace one-off emails.

I can turn off the interest/newsletter flags in Basket admin - but there is some custom code that makes the interests below Marketplace become nested in the "Marketplace" blurb - and the "Marketplace" blurb itself still remains (it's been hard-coded).


See attached screenshot.

Can someone remove the Marketplace code bit in the Email Preference Center?

Thanks!
I can take this.
Assignee: nobody → kohei.yoshino
Status: NEW → ASSIGNED
Attached file pull request
Commits pushed to master at https://github.com/mozilla/bedrock

https://github.com/mozilla/bedrock/commit/abd10cb58d6c10a6479a4764bc45d1c7b2d5dd9a
Fix Bug 1255123 - Remove Marketplace from Email Pref Center

https://github.com/mozilla/bedrock/commit/825d822a4d9b013886ce0490a4ead8e6c38ede0e
Merge pull request #4064 from kyoshino/bug-1255123-marketplace-newsletters

Fix Bug 1255123 - Remove Marketplace from Email Pref Center
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Sweet! Thanks Kohei! Can you let me know when this goes live so I can be sure to check it out & disable these subscription interest flags?

Thanks!
I think there will be a production push today or tomorrow. Will update this bug when it's done.
For some reason there was no push last week. The change is live on production today.
Flags: needinfo?(jdavis)
Thanks Kohei! It looks wonderful! 

I just disabled the subscription interest flags and all looks perfect. :)

Thanks again!
Status: RESOLVED → VERIFIED
Flags: needinfo?(jdavis)
Yay!
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: