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

Add Email Notification When a New Video is Available #1134

Open
ardalis opened this issue Aug 7, 2023 · 7 comments · May be fixed by #1329
Open

Add Email Notification When a New Video is Available #1134

ardalis opened this issue Aug 7, 2023 · 7 comments · May be fixed by #1329
Assignees
Labels
enhancement New feature or request good first issue Good for newcomers hacktoberfest help wanted Extra attention is needed

Comments

@ardalis
Copy link
Collaborator

ardalis commented Aug 7, 2023

Allow members to opt into receiving an email whenever a new video is published.

@ardalis ardalis added hacktoberfest enhancement New feature or request help wanted Extra attention is needed good first issue Good for newcomers labels Oct 3, 2023
@ExitoLab
Copy link

@ardalis Please assign this issue to me

@ardalis
Copy link
Collaborator Author

ardalis commented Oct 17, 2023

@ExitoLab go for it!

@sunecko
Copy link

sunecko commented May 12, 2024

@ardalis It has been completed, otherwise I can take care of it

@ardalis
Copy link
Collaborator Author

ardalis commented May 13, 2024

@hectorrhg sure give it a shot. Note that it requires a few things:

  • A way for members to opt in. This could be on the member's profile or on the Videos page, or both. Maybe a bell icon like YouTube uses for such notifications
  • Create an event when a new video is published. I'm not sure if this exists already (it's been a while since I reviewed that code) but if not, create and dispatch a domain event at that time
  • A handler for the event, which should gather information about the video, get a list of emails that have opted into the notifications, and send the emails
  • We already have email sending built into the app, so use the same infrastructure for this as is used elsewhere.

Let me know if you have any questions.

@ardalis
Copy link
Collaborator Author

ardalis commented Jul 26, 2024

@ShadyNagy this is probably a good one for you at this point.

@ShadyNagy
Copy link
Contributor

Yes great.

@ShadyNagy this is probably a good one for you at this point.

@ShadyNagy ShadyNagy self-assigned this Jul 29, 2024
@ardalis
Copy link
Collaborator Author

ardalis commented Aug 26, 2024

I think this is working on the NP Portal - can you port here to fix this? @ShadyNagy

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers hacktoberfest help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants