-
Notifications
You must be signed in to change notification settings - Fork 2
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
Feature/sophiakevin/mailing list #37
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! I added the delete and undo delete functionality into the table as well as integrated it with the admin dashboard frontend from main. We want to use the existing subscribers mongo collection instead of creating a new collection since the mailing list showcases all the emails and sign ups from the existing website. The table looks great, this frontend was pretty complicated, so great work guys!!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Everything looks good to me! Great job with the table, I know that frontend was tricky but it looks great. Also, really like the copy paste feature, it turns out nicely with a format easy to use for mailing lists.
Tracking Info
Resolves #31
Changes
DELETE functionality
Confirmation of Change
TODO