-
Notifications
You must be signed in to change notification settings - Fork 0
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
Sending mails #61
Sending mails #61
Conversation
ChiragJS
commented
Jun 9, 2024
- Dont merge this pr for now , not complete, will make further commits
- add the credentials in application-dev.properties to test the feature out in dev profile(username and password)
- For testing purposes i called the mailing function on the root url for the app, will have to place the function calls on particular functions
- Not yet implemented the logic for sending attachments
- No need for starttls property in application props , as we are using port 465
- Will make the necessary changes in the next Commit on this branch
This pr addresses issue #20 |
…when the first set of mails go out
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.
Looks good now..Maybe for later,we can keep beautification of mail upon user creation.
I think you can keep the same template, change the template content |
Yep,will do that before we go for the real launch..but can be merged for now |