Page MenuHomePhabricator

Mooeypoo (Moriel Schottlender)
Tech Lead, CommTech and Anti Harassment Team

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Wednesday

  • Clear sailing ahead.

User Details

User Since
Nov 18 2014, 11:57 PM (274 w, 5 d)
Availability
Available
IRC Nick
mooeypoo
LDAP User
Mooeypoo
MediaWiki User
Mooeypoo [ Global Accounts ]

Recent Activity

Fri, Feb 21

Mooeypoo added a comment to T245499: Improve performance of Compare query for Special:Investigate.

A couple of small clarifications from the meeting. The third option is the one that's laid out in the description of this ticket, including the product implications.
The first two options are dependent on a little bit of experimentation; we've discussed trying to mock a few tens of millions of rows so that we can test the potential queries to see how they'd work. The two options seem encouraging to us, but we can't quite be sure that they will resolve the issue yet, so we should add that to the list of potential complications. That said, they do *potentially* give us more data to display.

Fri, Feb 21, 11:25 PM · Performance Issue, CheckUser, Anti-Harassment
Mooeypoo added a comment to T245793: [Firefox 73] Infusing a RadioOptionWidget changes first-child alignment.

Confirmed; on Firefox v73, even on not too large of a screen.

Fri, Feb 21, 12:00 AM · Patch-For-Review, Browser-Support-Firefox, OOUI, Desktop Improvements

Thu, Feb 20

Mooeypoo updated the task description for T245181: Improve test coverage for Special:Investigate.
Thu, Feb 20, 7:25 PM · Anti-Harassment (The Letter Song), CheckUser
Mooeypoo added a comment to T244100: Spike: New/Improved OCR tool [8 hours].

We don't have that in production so far, so I'm not sure, especially due to security concerns. We might need to have a gadget where each user must whitelist the toolforge URL individually for security and privacy.

Thu, Feb 20, 4:02 PM · Community-Tech (Kanban-Q3-2019-20), Internet-Archive
Mooeypoo moved T245660: PRU: enable pru on test wikidata from Needs Review/Feedback to Product sign-off on the Community-Tech (Kanban-Q3-2019-20) board.

PRU is now available on all test wikis:

Thu, Feb 20, 12:25 AM · Community-Tech (Kanban-Q3-2019-20), Password-Reset-Update

Wed, Feb 19

Mooeypoo closed T240094: Create required table for new Watchlist Expiry feature as Resolved.

The table has been created. Thank you for all the advice and guidance and help @Marostegui! And thanks, @Anomie, for the input and guidance on the best ways to approach the purging.

Wed, Feb 19, 6:19 PM · MW-1.35-notes (1.35.0-wmf.19; 2020-02-11), DBA, Community-Tech (Kanban-Q3-2019-20), Core Platform Team Workboards (Clinic Duty Team), Expiring-Watchlist-Items
Mooeypoo closed T235005: Spike #2: Watchlist Expiry [12 hours] as Resolved.

Investigation resolved. Next steps on followup tickets. Thank you for everyone's input!

Wed, Feb 19, 6:17 PM · Community-Tech (Kanban-Q3-2019-20), Expiring-Watchlist-Items, TCB-Team
Mooeypoo moved T243300: Spike: Investigate Named References in VE [8 hours] from Needs Review/Feedback to Product sign-off on the Community-Tech (Kanban-Q3-2019-20) board.

Investigation is done; we will await next steps to create followup tickets.

Wed, Feb 19, 6:14 PM · Editing-team (Tracking), Community-Tech (Kanban-Q3-2019-20), VisualEditor

Tue, Feb 18

Mooeypoo moved T238961: PRU: Improve Security & Standardize Experience for Password Reset [medium] from Needs Review/Feedback to Product sign-off on the Community-Tech (Kanban-Q3-2019-20) board.
Tue, Feb 18, 10:33 PM · MW-1.35-notes (1.35.0-wmf.20; 2020-02-18), Community-Tech (Kanban-Q3-2019-20), Password-Reset-Update

Mon, Feb 17

Mooeypoo updated subscribers of T245358: Compress table watchlist_expiry.

Will do, @Marostegui ; FYI, I'm also tracking the creation of the table through this ticket: T244631: Create `watchlist_expiry` table in production after wmf.19 is available.

Mon, Feb 17, 8:18 AM · DBA

Sat, Feb 15

Mooeypoo added a comment to T245082: Add site-wide $wgDefaultWatchlistExpiry configuration variable.

How will the relative bits work with translation and localization if this is truly configurable per wiki?

Sat, Feb 15, 7:44 AM · Expiring-Watchlist-Items, TCB-Team, Community-Tech

Fri, Feb 14

Mooeypoo added a comment to T238961: PRU: Improve Security & Standardize Experience for Password Reset [medium].

Just to clarify for engineering related thing, am I right to say that, basically, the message ("Both username and email address are required to receive a temporary password via email.") should never appear anywhere, @ifried ?

Fri, Feb 14, 10:28 PM · MW-1.35-notes (1.35.0-wmf.20; 2020-02-18), Community-Tech (Kanban-Q3-2019-20), Password-Reset-Update
Mooeypoo updated subscribers of T244579: MediaWiki does not support consistent pagination on non-unique fields.

The product we're discussing (an expansion to CheckUser) deals with potentially large chunks of data from the database, which triggers a number of problems with regard to performance. Regardless of what method we use, we have to come up with a good way to deal with that. After discussing with the team and talking things with @Catrope for some advice around how CheckUser has historically extracted potentially large chunks of information to process, I think we have a slightly changed strategy, and have a plan.

Fri, Feb 14, 4:07 AM · MediaWiki-General, TechCom
Mooeypoo created T245224: Cleanup Watchlist expiry: matching of watchlist_expiry item with removal of a watchlist item.
Fri, Feb 14, 12:14 AM · TCB-Team, Community-Tech, Expiring-Watchlist-Items

Thu, Feb 13

Mooeypoo created T245082: Add site-wide $wgDefaultWatchlistExpiry configuration variable.
Thu, Feb 13, 12:14 AM · Expiring-Watchlist-Items, TCB-Team, Community-Tech
Mooeypoo created T245081: Special:EditWatchlist: Filter out expired items [medium].
Thu, Feb 13, 12:12 AM · Community-Tech (Kanban-Q3-2019-20), Expiring-Watchlist-Items, TCB-Team
Mooeypoo created T245080: Special:Watchlist: Filter out expired items [medium].
Thu, Feb 13, 12:11 AM · Community-Tech (Kanban-Q3-2019-20), Expiring-Watchlist-Items, TCB-Team
Mooeypoo created T245079: Watchlist expiry API: Filter out expired items when outputting the watchlist [medium].
Thu, Feb 13, 12:10 AM · Community-Tech (Kanban-Q3-2019-20), TCB-Team, Expiring-Watchlist-Items
Mooeypoo created T245078: Watchlist expiry API: Add a expiry parameter when adding an item to the watchlist [medium].
Thu, Feb 13, 12:08 AM · Patch-For-Review, Community-Tech (Kanban-Q3-2019-20), TCB-Team, Expiring-Watchlist-Items

Wed, Feb 12

Mooeypoo moved T243300: Spike: Investigate Named References in VE [8 hours] from In Development to Needs Review/Feedback on the Community-Tech (Kanban-Q3-2019-20) board.
Wed, Feb 12, 12:09 AM · Editing-team (Tracking), Community-Tech (Kanban-Q3-2019-20), VisualEditor

Tue, Feb 11

Mooeypoo updated the task description for T244804: Create a purge routine for `watchlist_expiry` [medium].
Tue, Feb 11, 10:46 PM · Patch-For-Review, Community-Tech (Kanban-Q3-2019-20), TCB-Team, Expiring-Watchlist-Items
Mooeypoo updated subscribers of T243300: Spike: Investigate Named References in VE [8 hours].

Thank you for the analysis, @HMonroy, this is super helpful!

Tue, Feb 11, 7:25 PM · Editing-team (Tracking), Community-Tech (Kanban-Q3-2019-20), VisualEditor
Mooeypoo added a comment to T244507: [wip] Temporary Watchlist UI .

That's a good point also for the no-JS version.

Tue, Feb 11, 12:06 AM · Community-Tech, Expiring-Watchlist-Items, TCB-Team

Mon, Feb 10

Mooeypoo added a comment to T244804: Create a purge routine for `watchlist_expiry` [medium].

The purging operation should be done every X edits, where X is a configuration variable that has a default but can be set separately per wiki.

I would suggest adding a maintenance script (as a thin wrapper around whatever code you write) that can be cronjob'd/similar too like we do in other situations
https://github.com/wikimedia/mediawiki/blob/master/maintenance/purgeExpiredUserrights.php and https://github.com/wikimedia/puppet/blob/b347052863d4d2e87b37d6c2d9f44f833cfd9dc2/modules/mediawiki/manifests/maintenance/purge_expired_userrights.pp

Mon, Feb 10, 11:18 PM · Patch-For-Review, Community-Tech (Kanban-Q3-2019-20), TCB-Team, Expiring-Watchlist-Items
Mooeypoo added a comment to T244804: Create a purge routine for `watchlist_expiry` [medium].

Note: This process is already implemented for the RecentChanges table, which is being purged every few edits. see: https://gerrit.wikimedia.org/g/mediawiki/core/+/799eeb583a6747ad96888ec0fef8f76e45079129/includes/Storage/DerivedPageDataUpdater.php#1493

Mon, Feb 10, 11:17 PM · Patch-For-Review, Community-Tech (Kanban-Q3-2019-20), TCB-Team, Expiring-Watchlist-Items
Mooeypoo created T244804: Create a purge routine for `watchlist_expiry` [medium].
Mon, Feb 10, 10:58 PM · Patch-For-Review, Community-Tech (Kanban-Q3-2019-20), TCB-Team, Expiring-Watchlist-Items

Sat, Feb 8

Mooeypoo created T244631: Create `watchlist_expiry` table in production after wmf.19 is available.
Sat, Feb 8, 2:33 AM · TCB-Team, Expiring-Watchlist-Items, Community-Tech, Release-Engineering-Team-TODO (2020-01 to 2020-03 (Q3))
Mooeypoo added a comment to T244507: [wip] Temporary Watchlist UI .

These look great, @Prtksxna !

Sat, Feb 8, 12:50 AM · Community-Tech, Expiring-Watchlist-Items, TCB-Team

Wed, Feb 5

Mooeypoo added a comment to T240094: Create required table for new Watchlist Expiry feature.

Not sure I get this, if we move to a model where we'd be purging based on the number of edits, we should be good now?
Say we purge every 1000 edits, if that operation fails, we'd be purging once it reaches 2000?
I think one retry at least is always good, servers can be overloaded, they could be connection issues, contention issues etc...so at least a retry should be attempted I believe.

Wed, Feb 5, 5:15 AM · MW-1.35-notes (1.35.0-wmf.19; 2020-02-11), DBA, Community-Tech (Kanban-Q3-2019-20), Core Platform Team Workboards (Clinic Duty Team), Expiring-Watchlist-Items
Mooeypoo added a comment to T124752: [RFC] Expiring watch list entries.

To clarify -- the wl_id was already added to the table. When we checked into this feature, we believed this RFC to be done because the details (wl_id in the watchlist table) were done, even though the original proposers then deprioritized the work on the feature. When we consulted, we were told no RFC is needed for the actual feature.

Wed, Feb 5, 3:00 AM · Community-Tech, User-Addshore, Expiring-Watchlist-Items, German-Community-Wishlist, TCB-Team, TechCom-RFC
Mooeypoo added a comment to T124752: [RFC] Expiring watch list entries.

We're working on this as part of the wishlist items. We're going with the strategy of adding a new table that references the watchlist item and has an expiration.

Wed, Feb 5, 2:35 AM · Community-Tech, User-Addshore, Expiring-Watchlist-Items, German-Community-Wishlist, TCB-Team, TechCom-RFC

Tue, Feb 4

Mooeypoo added a comment to T240094: Create required table for new Watchlist Expiry feature.

Thank you for the quick response, @Marostegui.

Our hope was that having this data in a separate table would allow us to tune the automated deletions that will happen as part of expiration. If we do this in the existing watchlist table, we risk causing table locks that could significantly affect app performance.

How are you going to determine the size of the DELETE batches. Deleting is an expensive operation in MySQL, so ideally we'd prefer often but smaller transaction rather than less but bigger ones.
I also assume normal "wait for replication protections" will be used on the scripts.

Tue, Feb 4, 1:35 AM · MW-1.35-notes (1.35.0-wmf.19; 2020-02-11), DBA, Community-Tech (Kanban-Q3-2019-20), Core Platform Team Workboards (Clinic Duty Team), Expiring-Watchlist-Items

Sun, Jan 26

Mooeypoo added a comment to T243626: Provide instructions to install and configure WWT for any wiki.

Thank you for the kind words, I'm really glad you like this, and very happy to see the positive reactions overall.

Sun, Jan 26, 3:54 AM · Documentation, Community-Tech, Who-Wrote-That

Jan 23 2020

Mooeypoo added a comment to T239680: CU 2.0: Persist the form state.

The patch as written by @dbarratt looks good to me, but I'd like to make sure this complies with your recommendations, @sbassett, especially in regards to the given packages and the versioning. Since this is going to be what we base the rest of the product on, can you be so kind to do a quick pass, @sbassett https://gerrit.wikimedia.org/r/c/mediawiki/extensions/CheckUser/+/554229

Jan 23 2020, 10:28 PM · MW-1.35-notes (1.35.0-wmf.19; 2020-02-11), Security-Team, Patch-For-Review, CheckUser, Anti-Harassment (The Letter Song)

Jan 22 2020

Mooeypoo added a comment to T240094: Create required table for new Watchlist Expiry feature.

Now that we have things organized and settled, I'm tagging DBA for assistance. Please let us know if there's anything we need to further expand on, explain, or help with. Your help is super appreciated, thank you!

Jan 22 2020, 10:32 PM · MW-1.35-notes (1.35.0-wmf.19; 2020-02-11), DBA, Community-Tech (Kanban-Q3-2019-20), Core Platform Team Workboards (Clinic Duty Team), Expiring-Watchlist-Items
Mooeypoo added a project to T240094: Create required table for new Watchlist Expiry feature: DBA.
Jan 22 2020, 10:30 PM · MW-1.35-notes (1.35.0-wmf.19; 2020-02-11), DBA, Community-Tech (Kanban-Q3-2019-20), Core Platform Team Workboards (Clinic Duty Team), Expiring-Watchlist-Items
Mooeypoo added a comment to T242918: Spike: Investigate WWT Expansion [4 hours].

After some thinking, I unfortunately think that the extension route might actually not be feasible because of the external API request (I don't think we allow for any of that in extensions in production) but either way, a question -- @MusikAnimal

Jan 22 2020, 10:16 PM · Community-Tech (Kanban-Q3-2019-20), Spike, Who-Wrote-That
Mooeypoo moved T242759: WWT: Standardise how we do attribution percentage in translations [small] from Needs Review/Feedback to QA on the Community-Tech (Kanban-Q3-2019-20) board.
Jan 22 2020, 8:40 PM · Community-Tech (Kanban-Q3-2019-20), Who-Wrote-That
Mooeypoo updated the task description for T240094: Create required table for new Watchlist Expiry feature.
Jan 22 2020, 6:20 PM · MW-1.35-notes (1.35.0-wmf.19; 2020-02-11), DBA, Community-Tech (Kanban-Q3-2019-20), Core Platform Team Workboards (Clinic Duty Team), Expiring-Watchlist-Items
Mooeypoo renamed T242760: Move WSExport to VPS from Move WSexport to VPS to Move WSExport to VPS.
Jan 22 2020, 12:45 AM · Community-Tech (Kanban-Q3-2019-20), E-Book-Export-Reliability

Jan 21 2020

Mooeypoo updated the task description for T243300: Spike: Investigate Named References in VE [8 hours].
Jan 21 2020, 10:57 PM · Editing-team (Tracking), Community-Tech (Kanban-Q3-2019-20), VisualEditor
Mooeypoo added a comment to T242091: Spike: WWT: WhoColor API does not work with some tokens.

This is a weird bug, but I don't think it's a problem of the character, I think there might be an issue in WhoColor with this specific article?

Jan 21 2020, 10:48 PM · Who-Wrote-That, Community-Tech
Mooeypoo added a comment to T242918: Spike: Investigate WWT Expansion [4 hours].

There is another option we should add to the investigation, @MusikAnimal -- let's also check into making WWT into a MediaWiki extension.

Jan 21 2020, 9:38 PM · Community-Tech (Kanban-Q3-2019-20), Spike, Who-Wrote-That
Mooeypoo added a comment to T233091: Revision Details Pop-up sometimes in wrong place .

Okay, thanks, Dom. I'll keep it in the 'to be estimated/discussed' column for now, and we'll check in again at a later time to see if the behavior resurfaces.
Also, @Mooeypoo, do you think this issue was fixed in the process of the refactor work in T241004? Thanks!

Jan 21 2020, 8:25 PM · Who-Wrote-That, Community-Tech
Mooeypoo moved T242954: WWT Popup: Padding changes [x-small] from Needs Review/Feedback to QA on the Community-Tech (Kanban-Q3-2019-20) board.
Jan 21 2020, 6:36 PM · Community-Tech (Kanban-Q3-2019-20), Who-Wrote-That
Mooeypoo moved T241004: WWT: Refactor the relationship between the RevisionPopupWidget, Model and Controller for clarity from Needs Review/Feedback to QA on the Community-Tech (Kanban-Q3-2019-20) board.
Jan 21 2020, 6:36 PM · Community-Tech (Kanban-Q3-2019-20), Who-Wrote-That
Mooeypoo claimed T242954: WWT Popup: Padding changes [x-small].

Added as part of https://github.com/wikimedia/WhoWroteThat/pull/138

Jan 21 2020, 4:35 PM · Community-Tech (Kanban-Q3-2019-20), Who-Wrote-That
Mooeypoo added a comment to T241004: WWT: Refactor the relationship between the RevisionPopupWidget, Model and Controller for clarity.

OK I fixed the comments, and also fixed up the request from T242954: WWT Popup: Padding changes [x-small] so this is ready for re-review.

Jan 21 2020, 4:35 PM · Community-Tech (Kanban-Q3-2019-20), Who-Wrote-That

Jan 16 2020

Mooeypoo added a comment to T242954: WWT Popup: Padding changes [x-small].

Perfect. This can be fixed / added to the ongoing PR https://github.com/wikimedia/WhoWroteThat/pull/138

Jan 16 2020, 4:52 PM · Community-Tech (Kanban-Q3-2019-20), Who-Wrote-That
Mooeypoo claimed T242993: WWT: Investigate Console Error.
Jan 16 2020, 4:40 PM · Community-Tech (Kanban-Q3-2019-20), Who-Wrote-That
Mooeypoo edited projects for T242993: WWT: Investigate Console Error, added: Community-Tech (Kanban-Q3-2019-20); removed Community-Tech.
Jan 16 2020, 4:39 PM · Community-Tech (Kanban-Q3-2019-20), Who-Wrote-That
Mooeypoo edited projects for T242993: WWT: Investigate Console Error, added: Community-Tech (Kanban-Q3-2019-20); removed Community-Tech.
Jan 16 2020, 4:39 PM · Community-Tech (Kanban-Q3-2019-20), Who-Wrote-That
Mooeypoo added a comment to T242993: WWT: Investigate Console Error.

Fixed with https://github.com/wikimedia/WhoWroteThat/pull/145

Jan 16 2020, 4:39 PM · Community-Tech (Kanban-Q3-2019-20), Who-Wrote-That
Mooeypoo added a comment to T242954: WWT Popup: Padding changes [x-small].

Yes we can, and there's a PR for the popup css pending, but I didn't have a reference for the previous padding. I can do it by eye, or do you want to provide a number? I am guessing 0.5em and 1em but my eye is much less design oriented than yours :)

Jan 16 2020, 10:05 AM · Community-Tech (Kanban-Q3-2019-20), Who-Wrote-That

Jan 13 2020

Mooeypoo updated subscribers of T241004: WWT: Refactor the relationship between the RevisionPopupWidget, Model and Controller for clarity.

Fix is in PR https://github.com/wikimedia/WhoWroteThat/pull/138

Jan 13 2020, 7:33 PM · Community-Tech (Kanban-Q3-2019-20), Who-Wrote-That

Jan 9 2020

Mooeypoo added a comment to T241062: WWT: Handle hidden edit summaries in revision details pop-up [small].

Note: I'm fairly sure this will be a lot easier to do (if not outright fixed?) with the refactoring PR at https://github.com/wikimedia/WhoWroteThat/pull/128

Jan 9 2020, 12:33 PM · Community-Tech (Kanban-Q3-2019-20), Who-Wrote-That

Jan 8 2020

Mooeypoo updated the task description for T240094: Create required table for new Watchlist Expiry feature.
Jan 8 2020, 7:10 PM · MW-1.35-notes (1.35.0-wmf.19; 2020-02-11), DBA, Community-Tech (Kanban-Q3-2019-20), Core Platform Team Workboards (Clinic Duty Team), Expiring-Watchlist-Items
Mooeypoo claimed T240094: Create required table for new Watchlist Expiry feature.
Jan 8 2020, 6:12 PM · MW-1.35-notes (1.35.0-wmf.19; 2020-02-11), DBA, Community-Tech (Kanban-Q3-2019-20), Core Platform Team Workboards (Clinic Duty Team), Expiring-Watchlist-Items
Mooeypoo moved T241004: WWT: Refactor the relationship between the RevisionPopupWidget, Model and Controller for clarity from In Development to Needs Review/Feedback on the Community-Tech (Kanban-Q3-2019-20) board.
Jan 8 2020, 6:05 PM · Community-Tech (Kanban-Q3-2019-20), Who-Wrote-That
Mooeypoo moved T241834: WWT: Extension should not be loaded on Turkish Special pages from Ready to QA on the Community-Tech (Kanban-Q3-2019-20) board.
Jan 8 2020, 8:43 AM · Community-Tech (Kanban-Q3-2019-20), Who-Wrote-That
Mooeypoo edited projects for T241834: WWT: Extension should not be loaded on Turkish Special pages, added: Community-Tech (Kanban-Q3-2019-20); removed Community-Tech.
Jan 8 2020, 8:43 AM · Community-Tech (Kanban-Q3-2019-20), Who-Wrote-That

Jan 7 2020

Mooeypoo added a comment to T148353: Option not to review article when adding maintenance tags.

That's great!! Let us know if you need any help on those (though it looks like you're super on top of everything :)

Jan 7 2020, 9:03 PM · MW-1.35-notes (1.35.0-wmf.15; 2020-01-14), User-DannyS712, Growth-Team, Collaboration-Team-Triage, User-Josve05a, Documentation, Regression, PageCuration
Mooeypoo moved T241371: WWT: Update Guided Tour Instructional Language from Needs Review/Feedback to QA on the Community-Tech (Kanban-Q3-2019-20) board.
Jan 7 2020, 6:59 PM · Community-Tech (Kanban-Q3-2019-20), Who-Wrote-That
Mooeypoo updated subscribers of T239680: CU 2.0: Persist the form state.

Tagging Security-Team for advice here. I think @sbassett helped us before (you might be a bit more in the loop on this?) but wasn't sure who is best to tag from your team, so I am tagging the team in general.

Jan 7 2020, 4:01 PM · MW-1.35-notes (1.35.0-wmf.19; 2020-02-11), Security-Team, Patch-For-Review, CheckUser, Anti-Harassment (The Letter Song)
Mooeypoo added a comment to T241992: WWT: Do not flip the plus/minus sign in size diff when doing RTL [small].

This is a great catch, @dom_walden !!

Jan 7 2020, 2:27 PM · Community-Tech (Kanban-Q3-2019-20), Who-Wrote-That

Jan 5 2020

Mooeypoo added a comment to T148353: Option not to review article when adding maintenance tags.

I've reviewed the pending patch at https://gerrit.wikimedia.org/r/c/mediawiki/extensions/PageTriage/+/530649

Jan 5 2020, 10:58 AM · MW-1.35-notes (1.35.0-wmf.15; 2020-01-14), User-DannyS712, Growth-Team, Collaboration-Team-Triage, User-Josve05a, Documentation, Regression, PageCuration
Mooeypoo moved T241004: WWT: Refactor the relationship between the RevisionPopupWidget, Model and Controller for clarity from Needs Review/Feedback to In Development on the Community-Tech (Kanban-Q2-2019-20) board.
Jan 5 2020, 10:39 AM · Community-Tech (Kanban-Q3-2019-20), Who-Wrote-That
Mooeypoo moved T241004: WWT: Refactor the relationship between the RevisionPopupWidget, Model and Controller for clarity from Ready to Needs Review/Feedback on the Community-Tech (Kanban-Q2-2019-20) board.
Jan 5 2020, 10:38 AM · Community-Tech (Kanban-Q3-2019-20), Who-Wrote-That
Mooeypoo claimed T241004: WWT: Refactor the relationship between the RevisionPopupWidget, Model and Controller for clarity.
Jan 5 2020, 10:38 AM · Community-Tech (Kanban-Q3-2019-20), Who-Wrote-That
Mooeypoo moved T233169: WWT: Support RTL languages for WWT from In Development to QA on the Community-Tech (Kanban-Q2-2019-20) board.

I've reviewed WWT in Hebrew, and it looks great!

Jan 5 2020, 8:15 AM · Community-Tech (Kanban-Q3-2019-20), I18n, RTL, Who-Wrote-That
Mooeypoo added a comment to T234713: WWT: Make the links functional again if there is an API error [small].

I didn't know how to force an API error so I switched off the Wifi to get a generic error, is that the same?
At the moment I am not able to re-produce this either. Closing for now.

Jan 5 2020, 8:13 AM · Community-Tech (Kanban-Q2-2019-20), Who-Wrote-That
Mooeypoo updated subscribers of T241920: WWT: "Lego" message includes 'contact us' in the middle of a sentence creates a weird translation .

Tagging @Prtksxna and @ifried since the choice should be product/UX one. Language/translation-wise, both options are valid, but I'm not sure which one looks and feels better to users.

Jan 5 2020, 8:10 AM · Community-Tech, Who-Wrote-That
Mooeypoo updated the task description for T241920: WWT: "Lego" message includes 'contact us' in the middle of a sentence creates a weird translation .
Jan 5 2020, 8:10 AM · Community-Tech, Who-Wrote-That
Mooeypoo created T241920: WWT: "Lego" message includes 'contact us' in the middle of a sentence creates a weird translation .
Jan 5 2020, 8:08 AM · Community-Tech, Who-Wrote-That

Jan 3 2020

Mooeypoo added a comment to T241256: WWT: Improve look on MonoBook skin.

We should make this white since that is part of our design elements and isn't re-using any Monobook elements.

Jan 3 2020, 4:42 PM · Who-Wrote-That, Community-Tech
Prtksxna awarded T240262: WWT: Address Opacity Overlap Bug [small] a Like token.
Jan 3 2020, 10:58 AM · Community-Tech (Kanban-Q3-2019-20), Who-Wrote-That

Jan 2 2020

Mooeypoo updated subscribers of T239840: No translations on Firefox Add-Ons repository.

We do have translations for zh-hant, but Firefox wants zh-tn and zh-tw; are either of these equivalent? (Translatewiki provides zh-hant, zh-hans, and zh-hk.)

Jan 2 2020, 5:26 PM · Community-Tech (Kanban-Q3-2019-20), translatewiki.net, Who-Wrote-That
Mooeypoo updated the task description for T240094: Create required table for new Watchlist Expiry feature.
Jan 2 2020, 7:55 AM · MW-1.35-notes (1.35.0-wmf.19; 2020-02-11), DBA, Community-Tech (Kanban-Q3-2019-20), Core Platform Team Workboards (Clinic Duty Team), Expiring-Watchlist-Items
Mooeypoo claimed T234713: WWT: Make the links functional again if there is an API error [small].
Jan 2 2020, 6:49 AM · Community-Tech (Kanban-Q2-2019-20), Who-Wrote-That
Mooeypoo added a comment to T234713: WWT: Make the links functional again if there is an API error [small].

@Prtksxna code-wise, this should already be happening; we're only disabling links in the html we're getting from WhoColor -- and if there's an API error we aren't getting that and are not replacing anything.

Jan 2 2020, 6:47 AM · Community-Tech (Kanban-Q2-2019-20), Who-Wrote-That
Mooeypoo moved T241004: WWT: Refactor the relationship between the RevisionPopupWidget, Model and Controller for clarity from New & TBD Tickets to To Be Estimated/Discussed on the Community-Tech board.
Jan 2 2020, 6:25 AM · Community-Tech (Kanban-Q3-2019-20), Who-Wrote-That
Mooeypoo claimed T240262: WWT: Address Opacity Overlap Bug [small].
Jan 2 2020, 6:25 AM · Community-Tech (Kanban-Q3-2019-20), Who-Wrote-That
Mooeypoo added a comment to T240262: WWT: Address Opacity Overlap Bug [small].

We still have the "problem" of the lighter line crossing the infobox, but now it only happens when you hover over the specific template, which (at least in my opinion) gives the user some sense of what it's attached to as a box on the screen.

Jan 2 2020, 6:24 AM · Community-Tech (Kanban-Q3-2019-20), Who-Wrote-That
Mooeypoo moved T236702: WWT: Decouple Bytes from Edit Summary [small] from Needs Review/Feedback to QA on the Community-Tech (Kanban-Q2-2019-20) board.

The more robust architecture refactoring to do things a bit more clearer is covered in T241004: WWT: Refactor the relationship between the RevisionPopupWidget, Model and Controller for clarity

Jan 2 2020, 6:17 AM · Community-Tech (Kanban-Q2-2019-20), Who-Wrote-That
Mooeypoo moved T232460: [BUG] Unresponsive script when loading articles in non-default language [8 hours] from Needs Review/Feedback to QA on the Community-Tech (Kanban-Q2-2019-20) board.

The upstream fix was merged (and deployed) into MediaWiki core, so this should be fixed without our quickfix. Moving to QA to verify.

Jan 2 2020, 6:15 AM · Community-Tech (Kanban-Q3-2019-20), Who-Wrote-That
Mooeypoo moved T235015: WWT: Support for Latest Version of a Page after VE Edits from Needs Review/Feedback to QA on the Community-Tech (Kanban-Q2-2019-20) board.

Nice catch

Jan 2 2020, 6:13 AM · Community-Tech (Kanban-Q3-2019-20), Who-Wrote-That
Mooeypoo moved T240690: WWT: Fix Incorrect Tool Access on Main & Article Pages from Needs Review/Feedback to QA on the Community-Tech (Kanban-Q2-2019-20) board.
Jan 2 2020, 6:11 AM · Community-Tech (Kanban-Q3-2019-20), Who-Wrote-That
Mooeypoo moved T239840: No translations on Firefox Add-Ons repository from Needs Review/Feedback to QA on the Community-Tech (Kanban-Q2-2019-20) board.
Jan 2 2020, 6:10 AM · Community-Tech (Kanban-Q3-2019-20), translatewiki.net, Who-Wrote-That

Dec 17 2019

Mooeypoo created T241004: WWT: Refactor the relationship between the RevisionPopupWidget, Model and Controller for clarity.
Dec 17 2019, 10:32 PM · Community-Tech (Kanban-Q3-2019-20), Who-Wrote-That

Dec 13 2019

Mooeypoo claimed T240690: WWT: Fix Incorrect Tool Access on Main & Article Pages.

Fixed here: https://github.com/wikimedia/WhoWroteThat/pull/122

Dec 13 2019, 6:51 PM · Community-Tech (Kanban-Q3-2019-20), Who-Wrote-That

Dec 10 2019

Mooeypoo added a comment to T240262: WWT: Address Opacity Overlap Bug [small].

After a brief chat in RL, I suggested this to @ifried :

Dec 10 2019, 10:22 PM · Community-Tech (Kanban-Q3-2019-20), Who-Wrote-That
Mooeypoo added a comment to T240262: WWT: Address Opacity Overlap Bug [small].

Just a quick comment here: css filters also have their own css stack context, so they won't help :(

Dec 10 2019, 9:36 PM · Community-Tech (Kanban-Q3-2019-20), Who-Wrote-That
Mooeypoo updated subscribers of T235130: WWT: Implement Opacity for Templates [medium].

I did some digging into this, since it looked weird that the template starts out being behind the infobox but then ends up *in front of it* when in WWT mode. (Thanks @Catrope for the debug help here)

Dec 10 2019, 1:04 AM · Community-Tech (Kanban-Q2-2019-20), Who-Wrote-That
Mooeypoo added a comment to T235130: WWT: Implement Opacity for Templates [medium].

This is a problem with rendering templates on top of templates on wikis, and there's very little we can do as a generalized solution, since the bugs are not generalized themselves. Case in point -- if you edit this example article in VisualEditor, you'll see the same problem; hovering over the bottom template shows the "overlay" as over the image itself.

Dec 10 2019, 12:24 AM · Community-Tech (Kanban-Q2-2019-20), Who-Wrote-That

Dec 6 2019

Mooeypoo added a comment to T239680: CU 2.0: Persist the form state.
In T239680#5717468, @AronManning wrote:

My concern is whether one check (pressing the "Submit" button, switching to "Compare" tab) for multiple users should create one or multiple cu_log entries.

Dec 6 2019, 7:21 AM · MW-1.35-notes (1.35.0-wmf.19; 2020-02-11), Security-Team, Patch-For-Review, CheckUser, Anti-Harassment (The Letter Song)
Mooeypoo added a comment to T239680: CU 2.0: Persist the form state.
In T239680#5717435, @AronManning wrote:

CU log entries and CU investigations have a 1-1 relation. Calling it a "log" is arbitrary, reflecting the original use-case. Maybe calling it "CU investigations" in the DB would be more appropriate while calling it "CU log" on the UI.
I see only one limiting factor with the current cu_log table: cul_user is an N-1 relation, that cannot accommodate for multiple users checked in one investigation. That will require a new table (schema change).

Dec 6 2019, 4:47 AM · MW-1.35-notes (1.35.0-wmf.19; 2020-02-11), Security-Team, Patch-For-Review, CheckUser, Anti-Harassment (The Letter Song)

Dec 5 2019

Mooeypoo updated subscribers of T235005: Spike #2: Watchlist Expiry [12 hours].

During TechConf, I've discussed the implication of a periodic purge of the Watchlist table with several people, and there seems to be consensus that the general approach is sound with regards to the new table and the periodic purge of the Watchlist based on indexed timestamps of the watchlist_expiry table (ticket upcoming for the creation of that table+consultation with DBAs)

Dec 5 2019, 1:07 AM · Community-Tech (Kanban-Q3-2019-20), Expiring-Watchlist-Items, TCB-Team

Dec 4 2019

Mooeypoo moved T235130: WWT: Implement Opacity for Templates [medium] from Needs Review/Feedback to QA on the Community-Tech (Kanban-Q2-2019-20) board.
Dec 4 2019, 8:36 PM · Community-Tech (Kanban-Q2-2019-20), Who-Wrote-That
Mooeypoo added a comment to T239680: CU 2.0: Persist the form state.

Repeating what we discussed (briefly) in an RL meeting -- We need to come up with a way to make sure the pagination works with the same state, but I am not convinced that sessions are the best way to go.

Dec 4 2019, 7:16 PM · MW-1.35-notes (1.35.0-wmf.19; 2020-02-11), Security-Team, Patch-For-Review, CheckUser, Anti-Harassment (The Letter Song)
Mooeypoo added a comment to T237593: [Epic] CheckUser 2.0: Compare.

@Rxy The AHT team has taken on rewriting and redesigning CheckUser as part of the general strategy around this area. This means that we are *heavily* editing the code, and the plan is to replace the current CheckUser special page with the new one that we're building.

Dec 4 2019, 5:06 PM · Epic, Anti-Harassment, CheckUser