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

Changed logic of email notifications #384

Merged
merged 7 commits into from
Sep 4, 2024
Merged

Conversation

DmytroDmytruk
Copy link
Collaborator

GreenCity PR
ita-social-projects/GreenCity#7377

Summary Of Changes 🔥
Changed email notification logic, unread notifications with types

ECONEWS_COMMENT_LIKE
ECONEWS_LIKE
EVENT_COMMENT_LIKE
ECONEWS_COMMENT
EVENT_COMMENT
ECONEWS_COMMENT_REPLY
EVENT_COMMENT_REPLY
FRIEND_REQUEST_RECEIVED
FRIEND_REQUEST_ACCEPTED

are sent using new endpoint /scheduled/notification
Issue Link 📋
ita-social-projects/GreenCity#7377

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

Copy link

@DmytroDmytruk DmytroDmytruk merged commit d29733b into dev Sep 4, 2024
3 checks passed
@DmytroDmytruk DmytroDmytruk deleted the email-notifications branch September 4, 2024 10:21
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