-
-
Notifications
You must be signed in to change notification settings - Fork 863
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
Don't send notifications on post/comment edits #1925
Comments
That might be unavoidable, because if they edited the comment, they might've added a mention. If we don't email edits, you might miss a mention. |
This is fixed by #1914. |
I think this also effects pure lemmy users. Some of the edits were just tiny typo corrections like adding an "s". Maybe there could be some filter to send only emails when a significant enough edit is made? |
Oh this is about notifications on the lemmy website? Then #1914 has nothing to do with it, my bad. |
Well, the intention on opening this up was due to email notifications. I don't know that the site was harassing me, I didn't catch any notifications in the browser about it. |
I wouldn't want to add any complicated logic to edit notifications, I'd rather them just be on or off. |
Dont send email notifications for edited comments (fixes #1925)
Recently had an email notification when someone responded to a comment of mine.
And then 5 or 6 more email notifications as they edited and saved a bunch of times. The extra notifications seem unnecessary.
The text was updated successfully, but these errors were encountered: