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

xfd: Update technical RM hidden comment #842

Merged
merged 1 commit into from Feb 10, 2020
Merged

xfd: Update technical RM hidden comment #842

merged 1 commit into from Feb 10, 2020

Conversation

DannyS712
Copy link
Contributor

Onwiki comment updated in [1], per discussion at [2]; the current value results in the request being placed in the Requests to revert undiscussed moves section instead of the intended Uncontroversial technical requests section[3]
[1] Diff: https://w.wiki/GtM
[2] Discussion: https://w.wiki/GtN (permalink to current: https://w.wiki/GtP)
[3] Diff: https://w.wiki/GtQ

Onwiki comment updated in [1], per discussion at [2]; the current value results in the request being placed in the `Requests to revert undiscussed moves` section instead of the intended `Uncontroversial technical requests` section[3]
[1] Diff: https://w.wiki/GtM
[2] Discussion: https://w.wiki/GtN (permalink to current: https://w.wiki/GtP)
[3] Diff: https://w.wiki/GtQ
@Amorymeltzer
Copy link
Collaborator

Thanks @DannyS712 — this is getting annoying. I really don't want to leave a twinkle message, but if folks are gonna keep changing it (unlike every xfd...) then maybe we need to go for it.

@Amorymeltzer Amorymeltzer changed the title [XFD] Update technical RM hidden comment xfd: Update technical RM hidden comment Feb 10, 2020
@Amorymeltzer Amorymeltzer merged commit 7108373 into wikimedia-gadgets:master Feb 10, 2020
@Amorymeltzer Amorymeltzer added this to the March 2020 update milestone Feb 10, 2020
@DannyS712 DannyS712 deleted the patch-9 branch February 10, 2020 18:34
Amorymeltzer added a commit to Amorymeltzer/twinkle that referenced this pull request Feb 14, 2020
… comment

The hidden message has been changing frequently, leaving us behind (e.g. wikimedia-gadgets#791, wikimedia-gadgets#842), so by using a short regex rather than an exact string match, we should be able to handle subsequent changes.  This change also only loads section number 2; possibly overkill, but could theoretically avoid some off-target effects with the page's other hidden comment.
wiki-ST47 pushed a commit to wiki-ST47/twinkle that referenced this pull request Sep 2, 2020
… comment

The hidden message has been changing frequently, leaving us behind (e.g. wikimedia-gadgets#791, wikimedia-gadgets#842), so by using a short regex rather than an exact string match, we should be able to handle subsequent changes.  This change also only loads section number 2; possibly overkill, but could theoretically avoid some off-target effects with the page's other hidden comment.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants