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

[FEATURE]: Email Alert on Deal Submission #223

Open
jamesqquick opened this issue Jul 9, 2024 · 5 comments
Open

[FEATURE]: Email Alert on Deal Submission #223

jamesqquick opened this issue Jul 9, 2024 · 5 comments

Comments

@jamesqquick
Copy link
Collaborator

Is your feature request related to a problem? Please describe.
As an admin, I would like to receive an email when a new deal is submitted.

Describe the solution you'd like
I'd like to trigger the send of an email to a list of admins any time a new deal is submitted.

Describe the benefits of your solution
This will help admins stay on track of new submissions.

This will require (Eventually) an admin dashboard for managing who admins are

@ScottDalessandro
Copy link
Contributor

Hey - I'm interested in working on this. Please let me know if I can start this, thanks!

@jamesqquick
Copy link
Collaborator Author

Hey @ScottDalessandro you're good to go! Let me know if you need any help!

@ScottDalessandro
Copy link
Contributor

Thanks @jamesqquick , I haven't used next in a while so I'm reviewing it and the stack used in this project.

I'm still reviewing the stack used but the first thing that comes to mind is some type of job queue (to not send emails in one long process to all admins) to trigger and resend emails if failed, something like trigger.dev, do we have anything like that in this stack? No worries in getting back to me asap, I'm still reviewing everything, thanks!

@jamesqquick
Copy link
Collaborator Author

@ScottDalessandro We are actually working in parallel to incorporate Inngest to handle async event handling. That's what we'll want to incorporate this feature with. Do you want to start by creating the new email template using react-email and then an API endpoint that can be called that accepts an HTML string and a to email address? That would be the endpoint that we'll have Inngest call eventually.

Then, you can start to look at integrating with Inngest by adding the event to Inngest after a new deal has been added? Let me know if you want to chat more in Discord on video!

@ScottDalessandro
Copy link
Contributor

@jamesqquick thank you! it would be great to sync up, I had a few questions a few of them in discord, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants