Skip to content

Patch T277832 - #90

Open
theresnotime wants to merge 4 commits into
masterfrom
theresnotime/T277832
Open

Patch T277832#90
theresnotime wants to merge 4 commits into
masterfrom
theresnotime/T277832

Conversation

@theresnotime

Copy link
Copy Markdown
Member

Bug: T277832

@jsnshrmn jsnshrmn left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for your work on this! The redaction logic looks good, but it also looks like a couple of paths got missed. Also, please separate the dependency work from the redaction work. short commit messages are the norm here, but it makes it even more important to try to keep each commit's scope tight.

Comment thread hashtagsv2/hashtags/views.py
Comment thread hashtagsv2/hashtags/views.py
Comment thread hashtagsv2/settings/base.py
@theresnotime
theresnotime marked this pull request as draft September 3, 2026 11:22
@theresnotime theresnotime changed the title Patch T277832, remove unsupported django-nose Patch T277832 Sep 3, 2026
@theresnotime

Copy link
Copy Markdown
Member Author

Thanks for your work on this! The redaction logic looks good, but it also looks like a couple of paths got missed.

The download pages now check the rows too, but they refuse a search with more than 5000 rows, because (e.g.) ?query=WPWP has 546,166 rows and one API call checks only 50. To make large downloads work again we must keep the visibility state in our own database, from the mediawiki.revision-visibility-change stream.

@theresnotime
theresnotime marked this pull request as ready for review September 3, 2026 12:02

@jsnshrmn jsnshrmn left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for the changes! One inline question/suggestion.

Comment thread hashtagsv2/hashtags/visibility.py Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants