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
MBS-12191: Allow admins to see spammer profiles #2408
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
mwiencek
approved these changes
Feb 4, 2022
I blocked these a bit too well, to the point where Freso himself cannot see them either anymore. This allows admins to see the page, but adds a visible warning that they are already banned so that they don't have to go check the flags to make sure.
@brainzbot, retest this please |
reosarevok
added a commit
that referenced
this pull request
Feb 8, 2022
* master: Update POT files using the production database Update translations from Transifex Add cleanup tests for Lantis URLs after 6aac5a2 Add lantis.jp sidebar Add lantis.jp external link favicon MBS-12022: Update URL cleanup of Lantis.jp MBS-12162: Do not require auth for non-private tag WS queries (#2387) MBS-12191: Allow admins to see spammer profiles (#2408) MBS-12189: Support "names" prefix for DAHR artists (#2406) Add back JSON-LD test for empty artist Document Artist::Works test Document Artist::Tags test Document Artist::Split test Document Artist::Show test Document Artist::Releases test Document Artist::Relationships test Document Artist::Recordings test Document Artist::Ratings test Document Artist::Merge test Merge mostly useless Artist::Edits test and Artist::OpenEdits Ensure the right error message is shown Move URL-related test to EditExternalLinks Document Artist::EditRelationships test Document Artist::EditExternalLinks test Document Artist::EditAlias test Document Artist::Edit test Check more details data in Artist::Details test Document Artist::Details test Also check edit note is required on Artist::DeleteAlias Document Artist::DeleteAlias test Document Artist::Create test Document Artist::AnnotationRevision test Remove useless use PartialDate calls from tests Document Artist::Aliases test and remove cruft Document Artist::AddAnnotation test Document Artist::AddAlias test Also test for contained areas in Area::Users test Document Area::Users test Document Area::Tags test Document Area::Show test Document Area::Edit test Document Area::Create test Document Area::Aliases test Document Admin::DeleteEditor test Document Admin::WikiDoc::Edit test Document Admin::WikiDoc::Delete test Test non-privileged users can't access restricted WikiDoc pages Document Admin::WikiDoc::Create test Readd scope before testing token expiration Document Authentication::WS test Document RemoveEmptyURLs test Separate DataStore::Redis test all into named tests Actually check the deleted key in DataStore::Redis Add t/author tests to circleci Document t/author tests Change tabs to spaces Escape dots in eol.t and notabs.t regex MBS-12188: Allow SNAC links for places (#2404) MBS-12187: Load ReleaseEvents with manifest in release merges (#2407) Update HACKING.md with recent changes (#2389) MBS-12160: Use 'canonical' to display sorted edit data JSON (#2386) MBS-12147: State that an email search had no results (#2381) MBS-2313: Avoid pointless empty annotation edits (#2397) MBS-12168: Avoid breaking list formatting on user bios and collections (#2388) Merge Node and browser JS tests (#2403) MBS-11532: Don't pass undef ISO variable to localizeAreaName (#2401) MBS-12092: Serialize edit_action form after errors are added Remove duplicate form serializing
reosarevok
added a commit
that referenced
this pull request
Feb 14, 2022
* beta: Update POT files using the production database Update translations from Transifex Update POT files using the production database Update translations from Transifex Add cleanup tests for Lantis URLs after 6aac5a2 Add lantis.jp sidebar Add lantis.jp external link favicon MBS-12022: Update URL cleanup of Lantis.jp MBS-12162: Do not require auth for non-private tag WS queries (#2387) MBS-12191: Allow admins to see spammer profiles (#2408) MBS-12189: Support "names" prefix for DAHR artists (#2406) Add back JSON-LD test for empty artist Document Artist::Works test Document Artist::Tags test Document Artist::Split test Document Artist::Show test Document Artist::Releases test Document Artist::Relationships test Document Artist::Recordings test Document Artist::Ratings test Document Artist::Merge test Merge mostly useless Artist::Edits test and Artist::OpenEdits Ensure the right error message is shown Move URL-related test to EditExternalLinks Document Artist::EditRelationships test Document Artist::EditExternalLinks test Document Artist::EditAlias test Document Artist::Edit test Check more details data in Artist::Details test Document Artist::Details test Also check edit note is required on Artist::DeleteAlias Document Artist::DeleteAlias test Document Artist::Create test Document Artist::AnnotationRevision test Remove useless use PartialDate calls from tests Document Artist::Aliases test and remove cruft Document Artist::AddAnnotation test Document Artist::AddAlias test Also test for contained areas in Area::Users test Document Area::Users test Document Area::Tags test Document Area::Show test Document Area::Edit test Document Area::Create test Document Area::Aliases test Document Admin::DeleteEditor test Document Admin::WikiDoc::Edit test Document Admin::WikiDoc::Delete test Test non-privileged users can't access restricted WikiDoc pages Document Admin::WikiDoc::Create test Readd scope before testing token expiration Document Authentication::WS test Document RemoveEmptyURLs test Separate DataStore::Redis test all into named tests Actually check the deleted key in DataStore::Redis Add t/author tests to circleci Document t/author tests Change tabs to spaces Escape dots in eol.t and notabs.t regex MBS-12188: Allow SNAC links for places (#2404) MBS-12187: Load ReleaseEvents with manifest in release merges (#2407) Update HACKING.md with recent changes (#2389) MBS-12160: Use 'canonical' to display sorted edit data JSON (#2386) MBS-12147: State that an email search had no results (#2381) MBS-2313: Avoid pointless empty annotation edits (#2397) MBS-12168: Avoid breaking list formatting on user bios and collections (#2388) Merge Node and browser JS tests (#2403) MBS-11532: Don't pass undef ISO variable to localizeAreaName (#2401) MBS-12092: Serialize edit_action form after errors are added Remove duplicate form serializing
akshaaatt
pushed a commit
that referenced
this pull request
Mar 8, 2022
I blocked these a bit too well, to the point where Freso himself cannot see them either anymore. This allows admins to see the page, but adds a visible warning that they are already banned so that they don't have to go check the flags to make sure.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Implement MBS-12191
I blocked these a bit too well, to the point where Freso himself cannot see them either anymore.
This allows admins to see the page, but adds a visible warning that they are already banned so that they don't have to go check the flags to make sure.