Skip to content

Send emails to members using templates#523

Open
ValeriyaMix wants to merge 35 commits intoWomen-Coding-Community:mainfrom
ValeriyaMix:send-emails-to-members-using-templates
Open

Send emails to members using templates#523
ValeriyaMix wants to merge 35 commits intoWomen-Coding-Community:mainfrom
ValeriyaMix:send-emails-to-members-using-templates

Conversation

@ValeriyaMix
Copy link
Contributor

Description

  • Added all email templates from the shared Google document to the codebase in .yml format.

  • Introduced a new endpoint sendTemplateEmail, which allows sending emails based on a selected template with dynamic parameters.

Related Issue

This PR closes the issue #439

Change Type

  • Bug Fix
  • New Feature
  • Code Refactor
  • Documentation
  • Test
  • Other

Screenshots

New end-point in Swagger:

image

Added .yml file for the email templates:

image

Pull request checklist

Please check if your PR fulfills the following requirements:

… the email request. Add methods to the EmailService and EmailTemplateService classes.
@ValeriyaMix ValeriyaMix force-pushed the send-emails-to-members-using-templates branch from 11e2675 to 03c26b2 Compare February 17, 2026 21:52
@dricazenck
Copy link
Collaborator

@ValeriyaMix do you have any update here?

@ValeriyaMix
Copy link
Contributor Author

ValeriyaMix commented Feb 21, 2026

@ValeriyaMix do you have any update here?

Hi @dricazenck ,

I’m seeing a failing integration test in the pipeline, although it passes consistently on my local environment. I’ve tried a few approaches to resolve it but I haven’t been able to identify the root cause yet. Please could you advise on what might be causing the issue?

EmailTemplateControllerTest > previewValidRequestReturnsRenderedTemplate() FAILED
    org.springframework.web.client.RestClientException at EmailTemplateControllerTest.java:50
        Caused by: org.springframework.http.converter.HttpMessageNotReadableException at EmailTemplateControllerTest.java:50
            Caused by: com.fasterxml.jackson.databind.exc.UnrecognizedPropertyException at EmailTemplateControllerTest.java:50

Thanks in advance for your help.


Update: I have identified the issue and fixed the failing integration test.

@ValeriyaMix
Copy link
Contributor Author

Hi @dricazenck , this PR is ready for review. All tests are passing and I have addressed all review comments.

emailBuilder.bcc(request.getBcc());
}

if (request.getReplyTo() != null && !request.getReplyTo().isEmpty()) {
Copy link
Collaborator

@dricazenck dricazenck Feb 26, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ValeriyaMix you could use here and for other null/empty checks

 if (StringUtils.isNotBlank(request.getReplyTo())) {


<p>
You can also express your interest in volunteering with us by submitting the following form:
<a href="{{volunteerFormUrl}}">Volunteer with Women Coding Community</a>,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ValeriyaMix - just a heads up that I changed the text here as per this comment. You can either delete this file as it was merged, or resolve the conflict keeping the text that was merged :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants