-
-
Notifications
You must be signed in to change notification settings - Fork 449
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
Fix: check reply-parent-user-id for blocked user #4502
Fix: check reply-parent-user-id for blocked user #4502
Conversation
we shouldn't hide messages from non blocked users, |
9f7e5c9
to
5d1f002
Compare
Make's sense, it now replaces the message content with *blocked message*. Another option may be to treat the message as if the "show reply context" setting was disabled which probably follows closer to what blocking a user achieves normally. This made more sense after hearing: https://www.twitch.tv/videos/1782533448?t=01h10m21s I'll leave the PR as draft for now. |
5d1f002
to
1e1b015
Compare
I think leaving reply context visible is better solution, because it let user know given message responds to blocked account and can be ignored right away. |
I'm putting in my vote for |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems to work as expected, also tested replying to a block user just to make sure nothing crazy would happen, and that worked as expected as well.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only a nitpick on the changelog entry, implementation looks good to me 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - only issue I would have would be making sure our native show-blocked-messages-if-you're-a-mod
works, but I have a vague memory of testing that myself.
@2547techno Thanks for your contribution! As a first-time contributor, you can now add yourself to the contributors list that's shown inside the application. To do so, open a new PR where you modify the contributors.txt file and add yourself to the list. Make sure to read the comments at a top for instructions. |
Pull request checklist:
CHANGELOG.md
was updated, if applicableDescription
Closes #4481 assuming option 2. from #4481 (comment) .
Reply context message is censored if from blocked user