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

Redacting an event should redact all of the reactions #286

Closed
manuroe opened this issue Jul 3, 2019 · 3 comments
Closed

Redacting an event should redact all of the reactions #286

manuroe opened this issue Jul 3, 2019 · 3 comments
Labels
A-Aggregations A-Reactions A-Redaction

Comments

@manuroe
Copy link
Member

manuroe commented Jul 3, 2019

From riot-web created by turt2live: vector-im/riot-web#9843

This probably needs server-side support to avoid making potentially thousands of calls to /redact
@manuroe
Copy link
Member Author

manuroe commented Jul 3, 2019

Is it right that redacting an event should redact all of the reactions?

If I say something inflamatory and everybody boos, is it right that I can rewrite history by redacting the inflamatory statement?

I think I would prefer that the record at least show that @bob:example.com said something and got 9000 👎s (of course all of the booers would be welcome to redact their boos if they chose)

@manuroe manuroe mentioned this issue Jul 3, 2019
Open
@tulir
Copy link
Contributor

tulir commented Jul 13, 2019

Duplicate of element-hq/element-web#176?

@kittykat kittykat removed the phase:4 label Oct 15, 2021
@bmarty bmarty removed the feature label Oct 27, 2021
@kittykat
Copy link
Contributor

kittykat commented Oct 5, 2022

Duplicate of #176

@kittykat kittykat marked this as a duplicate of #176 Oct 5, 2022
@kittykat kittykat closed this as not planned Won't fix, can't repro, duplicate, stale Oct 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-Aggregations A-Reactions A-Redaction
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@kittykat @bmarty @tulir @manuroe