Closed AtlasMaxima-zz closed 6 years ago
Thanks @AtlasMaxima. I know its a pain but please be a bit more verbose with the descriptions and summary. Is this for email notifications or for website notifications once invite
users has been submitted? I ask as if mailgun
we should test this on the development instance to check mailgun is working etc. for dispatching email notifications.
Looking good. Well done on splitting that PR apart!
hey @AtlasMaxima when you get a chance can you let me know answers to the above questions?
This was for website notification on the client side but I will create another issue for the mailgun which is the server side after implementing the email authenticity on the client side which will set more constraints beforehand.
great - thanks @AtlasMaxima. The main issue on the email side is with the template. I think we just need to swap it over to use html in the message rather than markdown. We will just need to amend the merge conflict in HEAD.jsx if you're happy with this PR.
invite notification will pop up after users click on 'Submit' after completing the 'Invite' form for website notification on client side.
i.e. green success box will appear after the user submits the email to invite