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.

Sign up for GitHub

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

Jump to bottom

Implemented Additional Notification Creation Logic #976

Merged
merged 2 commits into from
Dec 10, 2023
Merged

Implemented Additional Notification Creation Logic #976

merged 2 commits into from
Dec 10, 2023

Conversation

halisbal
Copy link
Contributor

@halisbal halisbal commented Dec 10, 2023 •
edited
Loading

A new notification is created for a user in these conditions

  • A private group member should receive a notification when a new post is posted in that group.
  • A post/comment owner should receive a notification when his post/comment receives first vote or 5&15&25 overall votes.
  • A post/comment owner should receive a notification when his post/comment receives a reply.
  • Closes Implementing Additional Notification Creation Logic #951

@halisbal halisbal added Type: Feature Issues related to adding new features or functionality to the project. Team: Backend Issues related to backend team labels Dec 10, 2023
@halisbal halisbal self-assigned this Dec 10, 2023
zeynep-baydemir
zeynep-baydemir approved these changes Dec 10, 2023
View reviewed changes
Copy link
Contributor

@zeynep-baydemir zeynep-baydemir left a comment

Choose a reason for hiding this comment

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

I have tested your work and it works properly, thank you

@zeynep-baydemir zeynep-baydemir merged commit b9ffa92 into main Dec 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zeynep-baydemir zeynep-baydemir zeynep-baydemir approved these changes

Assignees

@halisbal halisbal

Labels
Team: Backend Issues related to backend team Type: Feature Issues related to adding new features or functionality to the project.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Implementing Additional Notification Creation Logic
3 participants
@halisbal @zeynep-baydemir @haliskunduz