-
Notifications
You must be signed in to change notification settings - Fork 5
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
Set up email #6
Comments
Also for verifying email addresses, I assume? 📦 |
Our policy should probably be like Scratch - you can make an account but until your email is verified you can't upload resources. |
Ruling out sendgrid because it wants my life story |
Sendgrid approved my account O_o I just need info from @NanaLan |
Such as? Also I'm hoping to use some sort of |
API keys? Btw obviously hashed right? |
Just store it in an environment variable. Or actually almost anything but on a git repo :P |
+1 for environment variables. dotenv is now in package.json so we can easily do environment vars on our local copies Just don't remove |
Any progress on this? I guess it blocks the "forgot password" feature of #84 |
I have a SendGrid account set up, I'm just waiting for someone to ask me for API keys. |
asks |
I've implemented SendGrid in a84fd4f - you can send emails now, but somebody needs to make/get a cross-client email template... |
@MegaApuTurkUltra email template pls |
We should do the template in handlebars. Can handlebars render to string? |
yep, it can render anything (not just html). |
Just set up templating - see https://github.com/OpenSprites/OpenSprites-next/blob/master/server/utils/email.js#L44-L65 Currently an html version and a text version are templated and sent. I haven't actually tested sending an email though, for lack of a sendgrid api key. |
Email addresses for general contact us, email verification, and DMCA hotline.
The text was updated successfully, but these errors were encountered: