Add API endpoint for newsletter signup; add login settings for newsletter

RESOLVED FIXED in 2013-05-30

Status

Marketplace
API
P1
normal
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: basta, Assigned: ashort)

Tracking

2013-05-30
x86_64
Windows 7
Points:
---
Dependency tree / graph

Details

(Whiteboard: [qa-])

(Reporter)

Description

4 years ago
Add an API endpoint for bug 874932

Once the newsletter has been submitted, the login endpoint (for SSA) should return a new settings "newsletter" or something like that which indicates whether the user has signed up for the newsletter before (a simple boolean is sufficient).
I don't think we can tell you if the user has signed up before or not, fwiw.
Priority: -- → P1
CCing Paul for the configuration we need.  Paul: can you tell us the BASKET_URL and the Marketplace newsletter name?  We also need a BASKET_URL for dev/stage.

Thanks!
Assignee: nobody → ashort
Flags: needinfo?(pmac)
The BASKET_URL should be 'https://basket.mozilla.org'. There are dev and stage instances of basket, but they all point to the production exact-target, so it's best not to use them unless you're testing some new function of basket that's not in prod.

I'm not sure what the marketplace newsletter name is. My best guess from the available newsletters is 'app-dev'. Jessilyn Davis is person who'll know for real.
Flags: needinfo?(pmac) → needinfo?(jdavis)

Comment 4

4 years ago
This will be a new newsletter flag called "MARKETPLACE"
Flags: needinfo?(jdavis)
So we'll probably use 'marketplace' as the ID for use in the API as well. Right Jess?

Comment 6

4 years ago
correct
(Assignee)

Comment 7

4 years ago
https://github.com/mozilla/zamboni/commit/3090dcf
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2013-05-30

Comment 8

4 years ago
Please add STRs or mark it [qa-] if no QA verification is needed for this bug.
(Reporter)

Comment 9

4 years ago
[qa-] since we'll check it when we implement the form. The form can be QA'd
Whiteboard: [qa-]
The dependent bug will verify both.  -> qa-

Updated

4 years ago
Blocks: 884598
You need to log in before you can comment on or make changes to this bug.