Skip to content
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

#5331 - Eliminate Possibilities of Sending Multiple Project Update Emails #5968

Merged
merged 3 commits into from
Jul 12, 2023

Conversation

Aadesh-Baral
Copy link
Contributor

This PR addresses the issue #5331, which was not reproducible locally. The aim of this fix is to identify and eliminate potential factors that may contribute to the problem of sending multiple project update emails.

  • Makes sure distinct project contributors are fetched from database.
  • Makes sure project is project_update_email field is updated before starting to send project update emails.
  • Adds a logger to capture email type, project id and email address so that we can examine logs to make sure if the issue has been fixed.

@sonarcloud
Copy link

sonarcloud bot commented Jul 10, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@Aadesh-Baral Aadesh-Baral requested a review from d-rita July 10, 2023 05:56
@Aadesh-Baral Aadesh-Baral linked an issue Jul 10, 2023 that may be closed by this pull request
@Aadesh-Baral Aadesh-Baral merged commit fc2f65b into develop Jul 12, 2023
8 checks passed
@Aadesh-Baral Aadesh-Baral deleted the fix/5331-multiple-emails branch July 12, 2023 04:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Multiple project progress email for same project
2 participants