Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create an endpoint for newsletter subscriptions #212

Open
shubhsheth opened this issue Feb 20, 2022 · 2 comments
Open

Create an endpoint for newsletter subscriptions #212

shubhsheth opened this issue Feb 20, 2022 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@shubhsheth
Copy link
Member

Add an endpoint in the API that take in a POST request with following fields:

  • First Name
  • Last Name
  • Email

Validate this input using express-validator

Send this information to sendgrid mailing list using Sendgrid API, link to sendgrid documentation (https://docs.sendgrid.com/api-reference/contacts-api-lists/add-a-single-recipient-to-a-list)

Notes:

  • May need to create a test Sendgrid account to test the changes
@shubhsheth shubhsheth added this to To do in CougarCS-Backend Kanban via automation Feb 20, 2022
@shubhsheth shubhsheth added the enhancement New feature or request label Feb 20, 2022
mayankdp added a commit to mayankdp/CougarCS-Backend that referenced this issue Mar 22, 2022
mayankdp added a commit to mayankdp/CougarCS-Backend that referenced this issue Mar 22, 2022
@mayankdp
Copy link
Collaborator

I have created a backend for this newsletter similar to email.js.
I am pushing my codes to the newsletterendpoint branch. Please check it and let me know.
I want to create a pull request for approval so that I can submit the summary of this issue to my professor by the end of the work.

mayankdp added a commit to mayankdp/CougarCS-Backend that referenced this issue Apr 20, 2022
@shubhsheth
Copy link
Member Author

@mayankdp
The newsletter follows the same format as email endpoint but instead of sending email it stores the email in the mailing list. (https://docs.sendgrid.com/api-reference/contacts/add-or-update-a-contact)

I looked at your code, I think it's not ready to merge into master branch for the above reasons. You can create a pull request to merge it in a feature branch 'feature/newsletter' and then continue working on it after submitting the summary.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Development

No branches or pull requests

2 participants