Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Display redaction reason #5604

Closed
wants to merge 3 commits into from
Closed

Display redaction reason #5604

wants to merge 3 commits into from

Conversation

SimonBrandner
Copy link
Contributor

@SimonBrandner SimonBrandner commented Feb 1, 2021 •
edited
Loading

This PR adds displaying of redaction reasons.

Depends on Decrypt redaction events #1589.

@t3chguy t3chguy requested a review from a team February 1, 2021 13:01
turt2live
turt2live suggested changes Feb 2, 2021
View reviewed changes
Copy link
Member

@turt2live turt2live left a comment

Choose a reason for hiding this comment

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

We don't display redaction reasons because they are an abuse vector. This is the sort of thing that probably needs design work before much progress can be made.

@SimonBrandner
Copy link
Contributor Author

@turt2live Oh, I didn't know that. Should I leave this open?

@turt2live
Copy link
Member

up to you, I'd say. I'm not sure how soon a designer will be able to visit to take a look.

@SimonBrandner
Copy link
Contributor Author

Closing to lower the number of open PRs in the repo. I might reopen this one day.

@SimonBrandner SimonBrandner mentioned this pull request May 11, 2021
Closed
@HarHarLinks HarHarLinks mentioned this pull request May 11, 2021
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@turt2live turt2live turt2live requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@SimonBrandner @turt2live