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

Send notifications by email #6737

Merged
merged 21 commits into from
Dec 20, 2023
Merged

Send notifications by email #6737

merged 21 commits into from
Dec 20, 2023

Conversation

Yurii-Midianyi
Copy link
Contributor

@Yurii-Midianyi Yurii-Midianyi commented Dec 4, 2023

GreenCity PR

Link on issue 6587

Summary Of Changes 🔥

Standardized email sending fucntionality.
Added overloaded method for sending email notifications. It works in a parallel thread.
Added basic email notifications for such actions:

  • Eco news created
  • Eco news liked
  • Eco news commented
  • Reply to comments
  • Like of a comment
  • Event created
  • Event commented
  • Event was changed, then every attender receives a notifications
  • Event was canceled, then every attender receives a notifications
  • New attender joins your event
  • New friend request
  • Friend request accepted

Related tests were rewritten

PR Checklist Forms

(to be filled out by PR submitter)

  • Code is up-to-date with the dev branch.
  • You've successfully built and run the tests locally.
  • There are new or updated unit tests validating the change.
  • JIRA/ Github Issue number & title in PR title (ISSUE-XXXX: Ticket title)
  • This template filled (above this section).
  • Sonar's report does not contain bugs, vulnerabilities, security issues, code smells ar duplication
  • NEED_REVIEW and READY_FOR_REVIEW labels are added.
  • All files reviewed before sending to reviewers

@Yurii-Midianyi Yurii-Midianyi self-assigned this Dec 4, 2023
@Yurii-Midianyi Yurii-Midianyi linked an issue Dec 4, 2023 that may be closed by this pull request
@LiliaMokhnatska LiliaMokhnatska self-requested a review December 20, 2023 12:34
Copy link

@Yurii-Midianyi Yurii-Midianyi merged commit 7179c1b into dev Dec 20, 2023
3 checks passed
@Yurii-Midianyi Yurii-Midianyi deleted the send_notifications_by_email branch December 20, 2023 14:51
@VinuVicho VinuVicho mentioned this pull request Jan 29, 2024
8 tasks
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.

Create method for sending email notifications.
7 participants