Skip to content

Maithili - Fix the inability to send out emails from send emails tab #2240

Open
maithili20 wants to merge 2 commits into
developmentfrom
maithili-fix-send-emails-issue
Open

Maithili - Fix the inability to send out emails from send emails tab #2240
maithili20 wants to merge 2 commits into
developmentfrom
maithili-fix-send-emails-issue

Conversation

@maithili20

@maithili20 maithili20 commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

Description

image Fixes # (bug list priority high/medium/low x.y.z)

Related PRS (if any):

fix the issues in. this PR OneCommunityGlobal/HighestGoodNetworkApp#3148

Main changes explained:

  • Updated file src/services/announcements/emails/emailBatchService.js
  • Updated file src/utilities/emailValidators.js

How to test:

  1. check into current branch
  2. do npm install and ... to run this PR locally
  3. Clear site data/cache
  4. log as admin user
  5. go to dashboard→ Other Links→ Send Emails
  6. verify "Email successfully sent" message is shown only when user sends complete request. (i.e. comma separated email addresses, upload header: image_file)
  7. Also veridy if you actually receive an email. Verify sending it to one email address as well as multiple comma seperated email addresses
  8. Emails are sent via Oauth2. Please add following to env file -
    ANNOUNCEMENT_EMAIL
    ANNOUNCEMENT_EMAIL_CLIENT_ID
    ANNOUNCEMENT_EMAIL_CLIENT_SECRET
    ANNOUNCEMENT_EMAIL_REFRESH_TOKEN
    ANNOUNCEMENT_EMAIL_CLIENT_REDIRECT_URI
    Values of this can be found out using the instructions in the video - https://www.youtube.com/watch?v=-rcRf7yswfM&t=10s

Screenshots or videos of changes:

Please refer to video in PR OneCommunityGlobal/HighestGoodNetworkApp#3148 . It explains process to send email

Screenshot 2026-06-12 at 4 51 19 PM Screenshot 2026-05-30 at 2 44 53 PM Web_Photo_Editor

Note:

Include the information the reviewers need to know.

@sonarqubecloud

sonarqubecloud Bot commented Jun 5, 2026

Copy link
Copy Markdown

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.

1 participant