feat: wires emails to Adhoc & LongTerm flows#703
Conversation
…07/wcc-backend into feat/add-email-template
Sowmiya07
requested review from
dricazenck,
goelsonali,
lauracabtay,
nora-weisser and
nverbic
| "mentor_email", mentor.getEmail(), | ||
| "mentee_email", mentee.getEmail(), | ||
| "mentor_calendly_link", Optional.ofNullable(mentor.getCalendlyLink()).orElse(""), | ||
| "meeting_link", "", |
There was a problem hiding this comment.
This will be populated in next PR
| emailService.sendEmail(emailRequest); | ||
| log.info("{} notification successfully sent to {}", templateType, recipientEmails); | ||
| } catch (EmailSendException e) { | ||
| } catch (Exception e) { |
There was a problem hiding this comment.
In local dev, application was crashing when Email couldn't be sent successfully with MailAuthenticationException.
There was a problem hiding this comment.
I think we should handle this MailAuthenticationException
Sowmiya07
changed the title
dricazenck
changed the title
| * @param templateParams the parameters to use for rendering the template | ||
| */ | ||
| @SuppressWarnings( | ||
| "PMD.AvoidCatchingGenericException") // intentional: email failures must never break the |
There was a problem hiding this comment.
I don't think we should handle
| <p> | ||
| <strong>Please note:</strong> If you have not logged into the dashboard before, you should have received a password reset email from <strong>admin@womencodingcommunity.com</strong> prior to this email. | ||
| If you cannot find it, please check your spam/junk folder first. If you still haven’t received it, please reply to this email and we’ll resend it for you. | ||
| <strong>Please note:</strong> If you have not logged into the dashboard before, please reply to this email, so we can send a password reset email from <strong>admin@womencodingcommunity.com</strong>. |
There was a problem hiding this comment.
should be send the email to mentorship@womencodingcommunity.com not to admin, the admin does not receive any email. This was been disabled.
Please load the property from:
notification:
mentorship-email: mentorship@womencodingcommunity.com
You can search in the code by:
notificationConfig.getMentorshipEmail()
| </p> | ||
|
|
||
| <ul> | ||
| <li><a href="https://mentorship.womencodingcommunity.com/mentorship/resources">Mentorship Resources</a></li> |
There was a problem hiding this comment.
The website -frontend is not fixed yet, I think @Deejarh was the one to fix this changes.
| <br> | ||
| Follow us:<br> | ||
| <a href="https://www.linkedin.com/company/womencodingcommunity/">LinkedIn</a> | | ||
| <a href="https://www.instagram.com/women_coding_community/">Instagram</a> | |
There was a problem hiding this comment.
We don't use Instagram anymore, as far as I am aware.
| <a href="https://www.youtube.com/@womencodingcommunity">YouTube</a> | | ||
| <a href="https://www.meetup.com/women-coding-community">Meetup</a> | | ||
| <a href="https://github.com/Women-Coding-Community">GitHub</a> | | ||
| <a href="https://womencodingcommunity.slack.com/join">Slack</a> | |
There was a problem hiding this comment.
this link does not work, please review the one we already have in our footer:
https://join.slack.com/t/womencodingcommunity/shared_invite/zt-2hpjwpx7l-rgceYBIWp6pCiwc0hVsX8A
|
2 participants
Rename NEW_MENTEES_LONG to NEW_MENTEES_REVIEW to accommodate both Long Term and Adhoc
Adds 2 template for Long Term and Adhoc matching
Wire the above templates to following flows
Related issues #659 #681
Tested in local
Admin Approves an application
Admin confirm match
