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.

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

Add an AFCH preference to turn off auto watchlisting #298

Closed
NovemLinguae opened this issue Nov 6, 2023 · 2 comments · Fixed by #300
Closed

Add an AFCH preference to turn off auto watchlisting #298

NovemLinguae opened this issue Nov 6, 2023 · 2 comments · Fixed by #300

Comments

@NovemLinguae
Copy link
Member

Reported by Significa liberdade and Vanderwaalforces in Discord

Steps to reproduce

  • while logged out, create a draft with the template {{AfC submission}}. this will put it in a submitted state
  • log in and visit the draft
  • accept it

What happens?

  • Article and article talk are watchlisted
  • User talk message is not watchlisted

What should happen instead?

  • Should add an AFCH preference to turn off auto watchlisting. If that preference is selected, then article and article talk do not get watchlisted
image

Notes

  • This may be caused by the user's Special:Preferences. If so, the fix may be to read AFCH's watchlist preference, and if turned off, do an API query right after the accept to unwatchlist the two pages.
image
@MPGuy2824
Copy link

Setting the correct preference for the watchlist param while calling the edit api (https://www.mediawiki.org/wiki/API:Edit) should be enough.

@zanhecht
Copy link
Contributor

zanhecht commented Nov 8, 2023

I submitted pull request #300 for a patch which adds a "Do not add pages to watchlist" preference. When that preference is checked, it adds "watchlist: 'nochange'" to all the edit and move requests made by the script. It also makes it so the script never adds [[Wikipedia:Articles for creation/recent]] to the watchlist, since there is no value in doing so each time a draft is approved.

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 a pull request may close this issue.

3 participants