Page MenuHomePhabricator

RheingoldRiver (Megan "River" Cutrofello)
User

Projects

User does not belong to any projects.

Today

  • No visible events.

Tomorrow

  • No visible events.

Thursday

  • No visible events.

User Details

User Since
Nov 6 2015, 5:56 PM (536 w, 3 d)
Availability
Available
LDAP User
Unknown
MediaWiki User
RheingoldRiver [ Global Accounts ]

Recent Activity

Oct 17 2025

RheingoldRiver created T407666: Early access request for wiki.gg to pre-announce security fixes.
Oct 17 2025, 7:43 PM · SecTeam-Processed, Security, Security-Team

Mar 26 2025

RheingoldRiver added a comment to T31272: Implement Gadgets 2.0.

Does this also mean all those properties get saved in the JSON page? It used to be that way originally, but was removed as it makes the schema harder to evolve (T331222).

Mar 26 2025, 3:58 PM · MW-1.43-notes (1.43.0-wmf.4; 2024-05-07), User-notice, Gadgets-2.0, MediaWiki-extensions-Gadgets

Mar 25 2025

RheingoldRiver added a comment to T31272: Implement Gadgets 2.0.

On wiki.gg we've implemented most of what I described, with one small difference:

Mar 25 2025, 9:30 PM · MW-1.43-notes (1.43.0-wmf.4; 2024-05-07), User-notice, Gadgets-2.0, MediaWiki-extensions-Gadgets
RheingoldRiver added a comment to T281608: Rename Bot Passwords to Application Passwords.

As a note, we've updated messages and the special page to be called Application Passwords on wiki.gg, e.g. https://support.wiki.gg/wiki/Special:ApplicationPasswords

Mar 25 2025, 4:02 PM · MediaWiki-Platform-Team, MediaWiki-Core-AuthManager, MediaWiki-User-management

Feb 13 2025

RheingoldRiver added a comment to T96711: Change "move" to "rename" and update log entries, documentation accordingly.

We decided to do this on wiki.gg, we are renaming Special:MovePage to Special:RenamePage as 1.43 rolls out, and we also have changed all system messages to say 'rename' instead of 'move' (although we probably missed some and will have to fix still). This decision was made independently of this thread but we searched to see if it had been suggested before and figured it's worth bringing up here.

Feb 13 2025, 3:13 AM · MediaWiki-Page-rename

Feb 7 2025

RheingoldRiver added a comment to T380387: Create a design mockup of Special:RecentChanges using a better structure for the layout of information.

In case you didn't know, there is a "Show a confirmation prompt when clicking on a rollback link" option in the Preferences (Appearance menu)

Maybe we're getting slightly out of scope of this task, but that confirmation has a few problems:

Feb 7 2025, 3:49 AM · Moderator-Tools-Team, MediaWiki-Recent-changes, Design

Feb 6 2025

RheingoldRiver added a comment to T380387: Create a design mockup of Special:RecentChanges using a better structure for the layout of information.

My problems with the javascript UI that keep me using non-JS version are:

Feb 6 2025, 11:23 PM · Moderator-Tools-Team, MediaWiki-Recent-changes, Design

Feb 2 2025

RheingoldRiver updated the task description for T385380: Lists viewed in uselang=qqx alphabetize on qqx and so change the order of messages, making it hard to know which string is which.
Feb 2 2025, 3:44 AM · MediaWiki-Internationalization
RheingoldRiver created T385380: Lists viewed in uselang=qqx alphabetize on qqx and so change the order of messages, making it hard to know which string is which.
Feb 2 2025, 3:07 AM · MediaWiki-Internationalization

Oct 26 2024

RheingoldRiver added a comment to T31272: Implement Gadgets 2.0.

Hi, my suggestion for naming conventions is:

Oct 26 2024, 9:45 AM · MW-1.43-notes (1.43.0-wmf.4; 2024-05-07), User-notice, Gadgets-2.0, MediaWiki-extensions-Gadgets

Jul 11 2024

RheingoldRiver created T369779: Support interwikis that will be treated as external links with `nofollow`.
Jul 11 2024, 2:19 AM · MediaWiki-Interwiki, Schema-change, MediaWiki-extensions-Interwiki

Jun 25 2024

RheingoldRiver created T368434: Don't load the VisualEditor extension at all if the current user lacks the edit right.
Jun 25 2024, 5:25 PM · VisualEditor
RheingoldRiver added a comment to T368340: Add tracking category to any page using LST.

Hi, we want to remove this extension from about 800 wikis on wiki.gg. This will be much easier to do if there is a tracking category that we can query on each wiki, get a list of the ones that aren't using it, and then go through the rest by hand.

Jun 25 2024, 2:15 AM · MediaWiki-extensions-LabeledSectionTransclusion
RheingoldRiver created T368340: Add tracking category to any page using LST.
Jun 25 2024, 2:09 AM · MediaWiki-extensions-LabeledSectionTransclusion
RheingoldRiver created T368339: Add tracking category on every page using the Arrays extension.
Jun 25 2024, 2:07 AM · MediaWiki-extensions-Arrays

Jun 6 2024

RheingoldRiver added a comment to T366647: Rename Nuke extension to "MassDelete" or similar.

That would be renaming the extension (or "forking" it and then moving to the new version), so that "Nuke" doesn't show up on Special:Version either, and documentation is listed under MassDelete, etc.

Jun 6 2024, 9:35 PM · Voice & Tone, Moderator-Tools-Team, MediaWiki-extensions-Nuke

Jun 5 2024

RheingoldRiver added a comment to T366647: Rename Nuke extension to "MassDelete" or similar.

For users' familiarity across wiki installations, I might prefer creating one single alternative name and setting a toggle so that wikis can config between the two. Then whichever one is not chosen redirects to the other.

Jun 5 2024, 5:22 PM · Voice & Tone, Moderator-Tools-Team, MediaWiki-extensions-Nuke

Jun 4 2024

RheingoldRiver created T366647: Rename Nuke extension to "MassDelete" or similar.
Jun 4 2024, 7:52 PM · Voice & Tone, Moderator-Tools-Team, MediaWiki-extensions-Nuke

Jun 3 2024

RheingoldRiver added a comment to T365070: Volunteer NDA request for security issue access on behalf of wiki.gg.

Understood, we didn't realize it was such a restricted group of people. We would be happy to have just one person approved, either Alex (preferred) or myself, if this helps.

Jun 3 2024, 5:39 PM · WMF-NDA-Requests

May 28 2024

RheingoldRiver added a comment to T365070: Volunteer NDA request for security issue access on behalf of wiki.gg.

Hi, what are the next steps here? Do we need to file something other than this ticket? (Again the aim of this ticket is to request the NDA)

May 28 2024, 7:41 PM · WMF-NDA-Requests

Apr 24 2024

RheingoldRiver added a comment to T204366: Better varargs for templates.

I've also used a setup that I call Module:ForwardArgs to accomplish something like this. I think the most minimal example would look like:

Apr 24 2024, 1:16 PM · Language and Product Localization, Scribunto, MediaWiki-Parser-Templates, MediaWiki-Parser, Parsoid

Mar 21 2024

RheingoldRiver updated the task description for T360627: Add a hook to Scribunto to support cross-wiki module imports.
Mar 21 2024, 12:44 PM · Scribunto
RheingoldRiver renamed T360627: Add a hook to Scribunto to support cross-wiki module imports from Add a hook to Scribunto to support cross-wiki module imports via DB instead of HTTP to Add a hook to Scribunto to support cross-wiki module imports.
Mar 21 2024, 12:42 PM · Scribunto
RheingoldRiver created T360627: Add a hook to Scribunto to support cross-wiki module imports.
Mar 21 2024, 12:41 PM · Scribunto

Mar 18 2024

RheingoldRiver added a comment to T357606: Add #or parser function to ParserFunctions extension.

Ah sorry, I didn't notice that option when creating this ticket. Updated it with that template now!

Mar 18 2024, 1:31 PM · ParserFunctions
RheingoldRiver updated the task description for T357606: Add #or parser function to ParserFunctions extension.
Mar 18 2024, 1:31 PM · ParserFunctions

Mar 14 2024

RheingoldRiver added a comment to T282499: Consider whether Parsoid will support forced linear parsing..

Right, yes - Cargo has something similar too, where you can query values in the order in which they appear on the page, but it's already unreliable, even with linear parsing.

Mar 14 2024, 12:01 AM · MediaWiki-extensions-ExternalData, Parsoid-Read-Views (Phase 3 - Main namespace of officewiki / mediawiki.org renders with Parsoid), MediaWiki-extensions-Variables, Parsoid, MediaWiki-Parser

Feb 15 2024

RheingoldRiver updated the task description for T357606: Add #or parser function to ParserFunctions extension.
Feb 15 2024, 7:07 AM · ParserFunctions
RheingoldRiver created T357606: Add #or parser function to ParserFunctions extension.
Feb 15 2024, 6:40 AM · ParserFunctions

Sep 23 2022

RheingoldRiver added a comment to T310306: Add two hooks to Scribunto so that Module pages can be changed based on a "branch" parameter set in an invoke.

Hm, it looks like that only allows you to preview the page, and I'd prefer to be able to save it as well (e.g. this is a way for someone to submit changes and propose them so they'd want to link the result instead of just screenshotting). But since I figured a patch to Scribunto is unlikely to happen, previewing would still be a good compromise I think.

Sep 23 2022, 12:31 PM · Scribunto

Jul 4 2022

RheingoldRiver merged T309051: DisplayTitle shows nothing when linking to an anchor on a page with the same display as its anchor into T305623: Links with anchors are not rendered when the anchor is the same as the text.
Jul 4 2022, 4:11 AM · MediaWiki-extensions-DisplayTitle
RheingoldRiver merged task T309051: DisplayTitle shows nothing when linking to an anchor on a page with the same display as its anchor into T305623: Links with anchors are not rendered when the anchor is the same as the text.
Jul 4 2022, 4:11 AM · Patch-For-Review, MediaWiki-extensions-DisplayTitle

Jun 23 2022

RheingoldRiver updated subscribers of T309051: DisplayTitle shows nothing when linking to an anchor on a page with the same display as its anchor.

Oh @cicalese should I go ahead and do that last-resort patch?

Jun 23 2022, 6:22 PM · Patch-For-Review, MediaWiki-extensions-DisplayTitle

Jun 11 2022

RheingoldRiver created T310417: Gantt charts fail when they contain a GROUP BY query.
Jun 11 2022, 2:04 PM · MediaWiki-extensions-Cargo

Jun 9 2022

RheingoldRiver created T310306: Add two hooks to Scribunto so that Module pages can be changed based on a "branch" parameter set in an invoke.
Jun 9 2022, 6:06 PM · Scribunto

May 23 2022

RheingoldRiver created T309051: DisplayTitle shows nothing when linking to an anchor on a page with the same display as its anchor.
May 23 2022, 8:31 PM · Patch-For-Review, MediaWiki-extensions-DisplayTitle

May 1 2022

RheingoldRiver updated the task description for T307313: Jenkins doesn't have CSS-specific whitepsace rules regarding parentheses.
May 1 2022, 9:38 PM · ci-test-error
RheingoldRiver created T307313: Jenkins doesn't have CSS-specific whitepsace rules regarding parentheses.
May 1 2022, 9:37 PM · ci-test-error

Apr 19 2022

RheingoldRiver created T306459: The notice "redirected from" shouldn't be affected by displaytitle.
Apr 19 2022, 5:44 PM · MediaWiki-extensions-DisplayTitle

Apr 12 2022

RheingoldRiver added a comment to T260998: Introduction of a template "Used by" at mediawiki.org.

I like this! It'll help to find example uses in addition to showing "stability" of the extension, I've always appreciated the "wikis using Cargo" page to find examples of Cargo tables. For standardization, maybe the template should have a list of farms and generate the sentence?

Apr 12 2022, 1:40 AM · WMF-General-or-Unknown

Mar 8 2022

RheingoldRiver added a comment to T183099: Provide a button for quickly reactivating filters.

That seems like a reasonable option to me, especially if:

Mar 8 2022, 3:17 AM · AbuseFilter

Feb 24 2022

RheingoldRiver added a comment to T183099: Provide a button for quickly reactivating filters.

However, the filter might still get throttled until the 24 hours (or whatever the configured value is) will have passed.

Feb 24 2022, 8:30 AM · AbuseFilter

Feb 20 2022

RheingoldRiver created T302179: Add the ability to unthrottle an abuse filter without updating its age.
Feb 20 2022, 6:56 PM · AbuseFilter

Sep 24 2021

RheingoldRiver added a comment to T250963: Replace use of removed hook InternalParseBeforeSanitize.

Echoing @Revansx , here's my primary use case:

Sep 24 2021, 2:38 AM · ci-test-error, MediaWiki-extensions-Variables

Apr 30 2021

RheingoldRiver created T281608: Rename Bot Passwords to Application Passwords.
Apr 30 2021, 8:53 PM · MediaWiki-Platform-Team, MediaWiki-Core-AuthManager, MediaWiki-User-management

Apr 25 2021

RheingoldRiver added a comment to T281050: Edit API with appendtext & a contentmodel change gives an internal api error.

afaik the plan is for us to start updating regularly - we're not on forked MediaWiki anymore, but we are behind on versions for sure. I have no idea what the plan for updates is though, just that they're supposed to happen.

Apr 25 2021, 7:29 PM · Platform Team Workboards (Clinic Duty Team), MediaWiki-ContentHandler, MediaWiki-Action-API
RheingoldRiver added a comment to T281050: Edit API with appendtext & a contentmodel change gives an internal api error.

Ah, I didn't realize changecontentmodel existed since we're on 1.33 still, sorry.

Apr 25 2021, 7:15 PM · Platform Team Workboards (Clinic Duty Team), MediaWiki-ContentHandler, MediaWiki-Action-API
RheingoldRiver added a comment to T281050: Edit API with appendtext & a contentmodel change gives an internal api error.

I think it should allow both changes because it's possible for a script to encounter a page that was initialized with the wrong contentmodel, then want to append text to it & fix the contentmodel.

Apr 25 2021, 6:46 PM · Platform Team Workboards (Clinic Duty Team), MediaWiki-ContentHandler, MediaWiki-Action-API
RheingoldRiver created T281050: Edit API with appendtext & a contentmodel change gives an internal api error.
Apr 25 2021, 5:20 AM · Platform Team Workboards (Clinic Duty Team), MediaWiki-ContentHandler, MediaWiki-Action-API

Dec 6 2020

RheingoldRiver created T269515: Categories added via "MediaWiki:" system messages and {{int:}} magic word show incorrectly in translated languages.
Dec 6 2020, 5:40 AM · MediaWiki-Categories

May 25 2020

RheingoldRiver renamed T253500: Moving a reviewed page to a non-FlaggedRevs namespace without redirect leaves behind an unchecked page from [FlaggedRevs] Using review API on all revisions of a page doesn't review the page itself to [FlaggedRevs] Moving a reviewed page to a non-FlaggedRevs namespace without redirect leaves behind an unchecked page.
May 25 2020, 8:14 AM · FlaggedRevs
RheingoldRiver added a comment to T253500: Moving a reviewed page to a non-FlaggedRevs namespace without redirect leaves behind an unchecked page.

Okay so I've found a workaround for my own purposes, which is to leave behind a redirect in the move and then delete the redirect afterwards. This suppresses the "unchecked page." But it's still kind of weird:

  1. Create page - you have an unchecked page at OriginalLocation
  2. Check page - no unchecked page
  3. Move page WITH redirect - now you have a page which exists at OriginalLocation and is not unchecked, but says it's unchecked in Recent Changes
  4. Delete OriginalLocation - no unchecked page anymore
May 25 2020, 8:09 AM · FlaggedRevs
RheingoldRiver updated the task description for T253500: Moving a reviewed page to a non-FlaggedRevs namespace without redirect leaves behind an unchecked page.
May 25 2020, 7:44 AM · FlaggedRevs
RheingoldRiver renamed T253500: Moving a reviewed page to a non-FlaggedRevs namespace without redirect leaves behind an unchecked page from Using review API on all revisions of a page doesn't review the page itself to [FlaggedRevs] Using review API on all revisions of a page doesn't review the page itself.
May 25 2020, 7:37 AM · FlaggedRevs
RheingoldRiver created T253500: Moving a reviewed page to a non-FlaggedRevs namespace without redirect leaves behind an unchecked page.
May 25 2020, 7:37 AM · FlaggedRevs

Nov 6 2019

RheingoldRiver added a comment to T237490: Collect feedback from module and gadget authors for Developer Productivity & onwiki tooling techconf session.

Hi,

  1. I'm the lead admin of https://lol.gamepedia.com/League_of_Legends_Esports_Wiki. I do a lot of work with Lua and Cargo.
Nov 6 2019, 8:03 AM · Wikimedia-Technical-Conference-2019

May 14 2019

RheingoldRiver added a comment to T205817: AphrontCountQueryException when trying to visit my Profile in Phabricator.

Oh, now I'm able to remove - when I opened the ticket, I got the same error come up when I attempted to delete, but now it let me unlink successfully - if that's something you fixed, thanks!

May 14 2019, 11:53 AM · Phabricator

Apr 10 2019

RheingoldRiver added a comment to T205817: AphrontCountQueryException when trying to visit my Profile in Phabricator.

Hi sorry just wanted to follow up on this

Apr 10 2019, 11:29 AM · Phabricator

Nov 15 2018

RheingoldRiver added a comment to T209543: Cargo causes a DBTransactionStateError when attempting to store when a table does not exist..

I sync a lot of code across wikis, and sometimes the newer wiki doesn't have all the tables created yet, having the page completely error can make it pretty hard to figure out what needs to be created. Also with such a large number of wikis & tables, I don't always create everything unless I need it. There's a separate issue which is that (at least in the past) the recreate data api action could only REcreate tables, and not create them in the first place, which contributes to the lag in creating all tables on new wikis, since it's a manual process for each one; regardless, I don't think erroring the entire page is the best behavior when a table doesn't exist.

Nov 15 2018, 5:02 AM · MediaWiki-extensions-Cargo

Oct 19 2018

RheingoldRiver added a comment to T203730: Adding some FlaggedRevs features to Approved Revs.

Ok! I just had a conversation with @Alianin about all this, and I'll write here when I have an update.

Oct 19 2018, 7:05 PM · MediaWiki-extensions-Approved-Revs

Oct 18 2018

RheingoldRiver added a comment to T203730: Adding some FlaggedRevs features to Approved Revs.

So does everything seem good now?

Oct 18 2018, 9:34 PM · MediaWiki-extensions-Approved-Revs

Oct 9 2018

RheingoldRiver updated the task description for T203730: Adding some FlaggedRevs features to Approved Revs.
Oct 9 2018, 7:28 PM · MediaWiki-extensions-Approved-Revs
RheingoldRiver added a comment to T203730: Adding some FlaggedRevs features to Approved Revs.

Yeah unapprovable vs no current approved revision is important, since no current approved revision will be a "scary" warning message that the page is completely unreviewed by staff. I'll remove 2e then.

Oct 9 2018, 7:28 PM · MediaWiki-extensions-Approved-Revs

Oct 8 2018

RheingoldRiver added a comment to T203730: Adding some FlaggedRevs features to Approved Revs.
  • ah, my mistake in the later part, this doesn't depend on 1b, only 1a
Oct 8 2018, 8:51 PM · MediaWiki-extensions-Approved-Revs
RheingoldRiver updated the task description for T203730: Adding some FlaggedRevs features to Approved Revs.
Oct 8 2018, 8:50 PM · MediaWiki-extensions-Approved-Revs
RheingoldRiver updated the task description for T203730: Adding some FlaggedRevs features to Approved Revs.
Oct 8 2018, 8:47 PM · MediaWiki-extensions-Approved-Revs
RheingoldRiver added a comment to T203730: Adding some FlaggedRevs features to Approved Revs.

Oh yeah....Ok re-renumbered, and updated sub-part numbers as well as references.

Oct 8 2018, 11:08 AM · MediaWiki-extensions-Approved-Revs
RheingoldRiver updated the task description for T203730: Adding some FlaggedRevs features to Approved Revs.
Oct 8 2018, 11:07 AM · MediaWiki-extensions-Approved-Revs

Oct 7 2018

RheingoldRiver added a comment to T203730: Adding some FlaggedRevs features to Approved Revs.

Hah, yeah I was debating between temporarily doing Part 0.5 and that...Relabeled now. If anyone is following the discussion and confused, here is the old numbering/lettering. Hopefully I didn't miss any changes.

Oct 7 2018, 4:07 AM · MediaWiki-extensions-Approved-Revs
RheingoldRiver updated the task description for T203730: Adding some FlaggedRevs features to Approved Revs.
Oct 7 2018, 4:06 AM · MediaWiki-extensions-Approved-Revs
RheingoldRiver updated the task description for T203730: Adding some FlaggedRevs features to Approved Revs.
Oct 7 2018, 4:03 AM · MediaWiki-extensions-Approved-Revs

Oct 6 2018

RheingoldRiver added a comment to T203730: Adding some FlaggedRevs features to Approved Revs.

Ok updated a bit, and I switched some things around in order. I still haven't renamed stuff but if everything still seems good I'll do that next to have a final description that looks nice. And for 1 b and d, as long as the URL actions and classes are defined, it doesn't matter to me at all if this is built in.

Oct 6 2018, 5:06 AM · MediaWiki-extensions-Approved-Revs
RheingoldRiver updated the task description for T203730: Adding some FlaggedRevs features to Approved Revs.
Oct 6 2018, 5:04 AM · MediaWiki-extensions-Approved-Revs
RheingoldRiver added a comment to T203730: Adding some FlaggedRevs features to Approved Revs.

Yeah, I think everything here seems good, though there's still the question about whether 1b and/or d are going to be added to the extension or something I do in js/css provided the classes are added (which tbh I think they should be regardless, just for extra custom styling)

Oct 6 2018, 4:55 AM · MediaWiki-extensions-Approved-Revs

Oct 2 2018

RheingoldRiver updated the task description for T203730: Adding some FlaggedRevs features to Approved Revs.
Oct 2 2018, 6:39 PM · MediaWiki-extensions-Approved-Revs
RheingoldRiver updated the task description for T203730: Adding some FlaggedRevs features to Approved Revs.
Oct 2 2018, 6:35 PM · MediaWiki-extensions-Approved-Revs
RheingoldRiver updated the task description for T203730: Adding some FlaggedRevs features to Approved Revs.
Oct 2 2018, 6:31 PM · MediaWiki-extensions-Approved-Revs
RheingoldRiver added a comment to T203730: Adding some FlaggedRevs features to Approved Revs.

Ah, yeah that would work too. FR doesn't, but I'll make a note of it in the overview. And 1(b) potentially has the same issue too, but could also be fixed the same way by adding a class to the overall page.

Oct 2 2018, 6:29 PM · MediaWiki-extensions-Approved-Revs

Oct 1 2018

RheingoldRiver added a comment to T203730: Adding some FlaggedRevs features to Approved Revs.

Ok I made a couple more small edits. One thing for 1d - I realized I wasn't clear about this before, that link should only be in RC if there actually are post-approval/pending changes present for that page. (Since the link also serves as a visual indicator that a page has pending changes that need to be checked.) So maybe not as easy to do in JS.

Oct 1 2018, 8:53 PM · MediaWiki-extensions-Approved-Revs
RheingoldRiver updated the task description for T203730: Adding some FlaggedRevs features to Approved Revs.
Oct 1 2018, 8:51 PM · MediaWiki-extensions-Approved-Revs
RheingoldRiver updated the task description for T203730: Adding some FlaggedRevs features to Approved Revs.
Oct 1 2018, 6:27 PM · MediaWiki-extensions-Approved-Revs
RheingoldRiver added a comment to T203730: Adding some FlaggedRevs features to Approved Revs.

That's how FR does it - If you click "accept revision" (or alt+shift+S) here it just processes it without navigating away, it's just a lot faster to go through a lot of pages that way, since there's no waiting for the page to reload. I think that's the only FR action that you can do without any page navigation.

Oct 1 2018, 6:26 PM · MediaWiki-extensions-Approved-Revs
RheingoldRiver created T205817: AphrontCountQueryException when trying to visit my Profile in Phabricator.
Oct 1 2018, 7:19 AM · Phabricator
RheingoldRiver updated the task description for T203730: Adding some FlaggedRevs features to Approved Revs.
Oct 1 2018, 6:27 AM · MediaWiki-extensions-Approved-Revs
RheingoldRiver updated the task description for T203730: Adding some FlaggedRevs features to Approved Revs.
Oct 1 2018, 6:27 AM · MediaWiki-extensions-Approved-Revs
RheingoldRiver added a comment to T203730: Adding some FlaggedRevs features to Approved Revs.

Yeah, that's correct - though of course I would prefer to do as little JS as possible since we already have a crazy amount of JS all over the place.

Oct 1 2018, 6:20 AM · MediaWiki-extensions-Approved-Revs

Sep 30 2018

RheingoldRiver added a comment to T203730: Adding some FlaggedRevs features to Approved Revs.

Okay, I kind of do like the idea of having that action available, but it would definitely be not at all important, as long as the extension has a built-in hotkey on the current version of the page & any diff of (any older version to current) that set current version as approved (mirrors FR behavior, where alt+shift+S does this).

Sep 30 2018, 7:18 AM · MediaWiki-extensions-Approved-Revs

Sep 28 2018

RheingoldRiver added a comment to T203730: Adding some FlaggedRevs features to Approved Revs.

Oh sorry. Monitor how much I actually need the [pending changes] (or action=diffapprovedtolatest) links vs the regular diff link in RC. A lot of the time, they aren't the same link & I really did need the one provided by FR. So basically, 1(d) is actually as important as I thought it was.

Sep 28 2018, 6:45 PM · MediaWiki-extensions-Approved-Revs
RheingoldRiver added a comment to T203730: Adding some FlaggedRevs features to Approved Revs.

Sorry, I don't think I understand this. What is the "diff" you're talking about - the standard MediaWiki "diff" link, or some link from FR? If it's the latter, I don't understand.

Sep 28 2018, 5:16 AM · MediaWiki-extensions-Approved-Revs

Sep 27 2018

RheingoldRiver added a comment to T203730: Adding some FlaggedRevs features to Approved Revs.

Ok I updated the task with the diffapprovedtolatest URL action in b and d, and removed c since that's part of b now (after everything I'll re-letter but that's probably too confusing for now).

Sep 27 2018, 5:26 PM · MediaWiki-extensions-Approved-Revs
RheingoldRiver updated the task description for T203730: Adding some FlaggedRevs features to Approved Revs.
Sep 27 2018, 5:17 PM · MediaWiki-extensions-Approved-Revs
RheingoldRiver updated the task description for T203730: Adding some FlaggedRevs features to Approved Revs.
Sep 27 2018, 5:16 PM · MediaWiki-extensions-Approved-Revs
RheingoldRiver added a comment to T203730: Adding some FlaggedRevs features to Approved Revs.

Now I think I fully understand the issue - in your case, you want an approval to happen as quickly as possible, even if the latest revision is not 100% perfect, as long as it doesn't contain any obvious flaws. You want to approve in haste and correct in leisure, to coin a phrase.

Yeah, exactly this. I view AR/FR as something I really don't want to have, and we only use it out of necessity to avoid super incorrect things that are put live every now and then either through user error or vandalism. So our approval process is a very minimal and quick review just to make sure nothing is blatantly false. This is also why the permissions thing is such a big deal to me, I want a lot of people to bypass this review altogether, and we'll just monitor via patrols what has actually been carefully inspected or not.

Sep 27 2018, 6:06 AM · MediaWiki-extensions-Approved-Revs

Sep 26 2018

RheingoldRiver added a comment to T203730: Adding some FlaggedRevs features to Approved Revs.

No, I always want that link regardless of number of users. Even if I did want to check the history in that case, I'd still first look at the net diff just to have consistency in workflow. Patrol is this. Basically approving an edit via FR/AR = "this edit is not wrong and so it can be live" - this needs to be as fluid as possible to happen, and happen quickly. Patrolling is a much more leisurely and careful thing, and would avoid the issue that you're describing being problematic.

Sep 26 2018, 7:33 PM · MediaWiki-extensions-Approved-Revs
RheingoldRiver updated the task description for T203730: Adding some FlaggedRevs features to Approved Revs.
Sep 26 2018, 6:39 PM · MediaWiki-extensions-Approved-Revs
RheingoldRiver added a comment to T203730: Adding some FlaggedRevs features to Approved Revs.

@Alianin could answer about FR being phased out, I'm not sure the details about it.

Sep 26 2018, 6:37 PM · MediaWiki-extensions-Approved-Revs
RheingoldRiver added a comment to T203730: Adding some FlaggedRevs features to Approved Revs.

Ah and re: 1b), what if it was on every view of the page? That would fix 1c and provide more consistency in doing so.

Sep 26 2018, 5:26 PM · MediaWiki-extensions-Approved-Revs
RheingoldRiver updated the task description for T203730: Adding some FlaggedRevs features to Approved Revs.
Sep 26 2018, 5:26 PM · MediaWiki-extensions-Approved-Revs
RheingoldRiver added a comment to T203730: Adding some FlaggedRevs features to Approved Revs.

No, I almost never bother looking at page history before clicking to a net pending diff. It's almost always irrelevant to the change made on the page. My wiki is basically a news site, so page edits are usually the addition of new information that didn't exist in the previous version of the page because it hadn't happened yet. I guess this is different from most wikis which might be the reason for a lot of our disconnect.

Sep 26 2018, 5:24 PM · MediaWiki-extensions-Approved-Revs
RheingoldRiver added a comment to T203730: Adding some FlaggedRevs features to Approved Revs.

Wait in 1(b) - would that link be at the top of every single view of the page? My understanding was that was only when viewing current version of the page, but it sounds like it would be on edit/history/viewing diffs/viewing old versions/etc ? If so then we can strike 1(c) no problem, although I would still like to keep 1(d) (that link that FR provides from RC is super important).

Sep 26 2018, 4:45 AM · MediaWiki-extensions-Approved-Revs