Closed
Bug 1046888
Opened 10 years ago
Closed 10 years ago
Make newsletter work in about:apps
Categories
(Marketplace Graveyard :: Consumer Pages, defect, P3)
Tracking
(Not tracked)
RESOLVED
FIXED
2014-08-19
People
(Reporter: mat, Assigned: mat)
References
Details
(Whiteboard: [qa-])
I added the newsletter block (with lorem-ipsum text...) in https://github.com/mozilla/discoplace/commit/a0af6193d3f30e1f960f96f75407cda65d492ddf but it hits the marketplace newsletter API endpoint... which only allows authenticated users. Figure out what to do and do it :)
Assignee | ||
Comment 1•10 years ago
|
||
Allowed anonymous users to subscribe to the newsletter, with rate-limiting, in https://github.com/mozilla/zamboni/commit/cc5094be6c7d4f2e4aa16bf95eb978a6777277d6
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•10 years ago
|
||
From information in https://bugzilla.mozilla.org/show_bug.cgi?id=1048522 we need to subscribe the users to 'mozilla-and-you,marketplace-desktop' instead of 'marketplace', so I'll need to modify the newsletter API a little bit to allow the caller to choose which newsletters to subscribe to.
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → mpillard
Assignee | ||
Comment 3•10 years ago
|
||
Currently disabled but everything is in place for when we want to enable the block: - https://github.com/mozilla/discoplace/commit/397c1437d47c5889526f19242f32a62f24852bb5 - https://github.com/mozilla/discoplace/commit/98832e77dd1f4038fe24a431d803a8360a27666b - https://github.com/mozilla/zamboni/commit/943cab20dbd0c59b63e0b00cc033072a5a3d3510
No longer blocks: 1046577
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Priority: -- → P3
Resolution: --- → FIXED
Target Milestone: --- → 2014-08-19
Comment 4•10 years ago
|
||
Since it is disabled, it cannot be verified. I'm marking this [qa-] for now.
Whiteboard: [qa-]
You need to log in
before you can comment on or make changes to this bug.
Description
•