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

added option in settings to change whether replies notify the user #6858

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

Noxor11
Copy link

@Noxor11 Noxor11 commented Apr 18, 2024

First time contributor checklist:

Contributor checklist:

  • My contribution is not related to translations.
  • My commits are in nice logical chunks with good commit messages
  • My changes are rebased on the latest main branch
  • A yarn ready run passes successfully (more about tests here)
  • My changes are ready to be shipped to users

Description

Added an optional setting to set notifications to messages from the user being replied by another user (disabled by default). This way the user can choose whether to get notified or not for replies the same way a mention would.

This was inspired by me missing some messages because the group chat is muted and they don't ping me all the time. This way I don't have to check the chat as often to check if they replied or not.

I tested the functionality by enabling and disabling the setting and watching whether the notifications for a reply show up when I receive a reply.

Tested with EndeavourOS

notifications

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants