-
-
Notifications
You must be signed in to change notification settings - Fork 4
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
feat(28): adding testing packages and sendEmail for confirmation #67
Conversation
b2e6dad
to
6b44b9f
Compare
Please rebase |
email send email and dev emails
Co-authored-by: Miki Lombardi <[email protected]>
b9ab42b
to
1312865
Compare
@TheJoin95 rebased |
@404answernotfound please check the CI |
Overall looking good, I think we can merge it after the previous comments are going to be resolved, thanks! |
…ge for members to actually send them
Using this PR to also get a start on #29 and #30 which will require admin membership page to be completed before moving forward. This is enough to get us started with making emails work tho, so it's a closing for #28 and a starting point for debugging #29 and #30 To your points @TheJoin95 , we can add env vars |
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.
Approving so that after fixing the lint issues you can merge it
Should have fixed |
🔗 Linked issue
#28
/emails
page where you will find a button to send a confirmation email.❓ Type of change
📚 Description
📝 Checklist