Closed Bug 1046888 Opened 10 years ago Closed 10 years ago

Make newsletter work in about:apps

Categories

(Marketplace Graveyard :: Consumer Pages, defect, P3)

defect

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 :)
Blocks: 1046577
Allowed anonymous users to subscribe to the newsletter, with rate-limiting, in https://github.com/mozilla/zamboni/commit/cc5094be6c7d4f2e4aa16bf95eb978a6777277d6
Status: NEW → ASSIGNED
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: nobody → mpillard
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
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.