Page MenuHomePhabricator

Niharika (Niharika)
Product Manager, Community Tech team

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Wednesday

  • Clear sailing ahead.

User Details

User Since
Nov 27 2014, 3:32 AM (262 w, 4 d)
Availability
Available
IRC Nick
Niharika
LDAP User
Niharika29
MediaWiki User
NKohli (WMF) [ Global Accounts ]

Recent Activity

Today

Niharika updated subscribers of T237669: Expose the device in the user-agent from the app requests.

Cool, this is now deployed in the latest Android app. I'm untagging Android, and leaving iOS to handle on their end.

Mon, Dec 9, 6:42 PM · Wikipedia-iOS-App-Backlog, iOS-app-Bugs, Anti-Harassment, CheckUser

Tue, Nov 26

Niharika moved T239259: Security Review for whichbrowser/parser from Untriaged to Tracking work by other teams on the Anti-Harassment board.
Tue, Nov 26, 10:10 PM · Security-Team-Reviews, MediaWiki-Vendor, Anti-Harassment, CheckUser
Niharika moved T238782: CU 2.0: Display data in the Compare tab from Cards ready to be discussed to Cards ready for development on the Anti-Harassment board.
Tue, Nov 26, 5:11 PM · Anti-Harassment, CheckUser
Niharika moved T238782: CU 2.0: Display data in the Compare tab from Design to Cards ready to be discussed on the Anti-Harassment board.
Tue, Nov 26, 4:46 PM · Anti-Harassment, CheckUser

Mon, Nov 25

Niharika closed T200938: Special:CentralAuth should provide the same blocking information as Special:BlockList does, a subtask of T190350: Epic: ⚡️ Partial blocks, as Resolved.
Mon, Nov 25, 9:08 PM · Anti-Harassment, MediaWiki-User-management
Niharika closed T200938: Special:CentralAuth should provide the same blocking information as Special:BlockList does, a subtask of T226651: Replace local namespaces with real links on Special:CentralAuth, as Resolved.
Mon, Nov 25, 9:08 PM · Anti-Harassment, MediaWiki-extensions-CentralAuth
Niharika closed T200938: Special:CentralAuth should provide the same blocking information as Special:BlockList does as Resolved.
Mon, Nov 25, 9:08 PM · MW-1.35-notes (1.35.0-wmf.5; 2019-11-05), Core Platform Team Workboards (Clinic Duty Team), Anti-Harassment (The Letter Song), Patch-For-Review, MediaWiki-extensions-CentralAuth, Stewards-and-global-tools, MediaWiki-User-management
Niharika closed T200938: Special:CentralAuth should provide the same blocking information as Special:BlockList does, a subtask of T229895: Refactor user management logic into a UserManager service, as Resolved.
Mon, Nov 25, 9:08 PM · Patch-For-Review, MediaWiki-extensions-CentralAuth
Niharika closed T200938: Special:CentralAuth should provide the same blocking information as Special:BlockList does, a subtask of T238235: Update action=globaluserinfo to include the same blocking info on the special page, as Resolved.
Mon, Nov 25, 9:08 PM · MediaWiki-extensions-CentralAuth
Niharika awarded T238960: Conversion to volunteer NDA for MaxSem a Love token.
Mon, Nov 25, 8:52 PM · LDAP-Access-Requests, SRE-Access-Requests, Operations

Fri, Nov 22

Niharika updated the task description for T237532: Preferences are saved as GlobalPreferences when autoglobal is enabled for the field.
Fri, Nov 22, 7:53 PM · Anti-Harassment, Community-Tech, MediaWiki-extensions-GlobalPreferences
Niharika claimed T237039: [Epic] CheckUser 2.0: Preliminary check.
Fri, Nov 22, 6:44 PM · Epic, CheckUser, Anti-Harassment
Niharika added a comment to T237299: CU 2.0: Sticky highlighting in Preliminary check and Compare tabs.

@Prtksxna We also talked about the sticky highlights showing up in the filters at one point. Are you still planning to add that in?

Fri, Nov 22, 6:43 PM · CheckUser, Anti-Harassment
Niharika renamed T238808: CU 2.0: Pagination in the compare tab from CU 2.0: Pagination in the preliminary check and compare tabs to CU 2.0: Pagination in the compare tab.
Fri, Nov 22, 6:40 PM · Anti-Harassment, CheckUser
Niharika added a comment to T175587: Add a user-agent parser to CheckUser.

For each UA we have the three parts. For 22 records, that is 66 pieces. So for each library, how many of those were right/wrong would be helpful to know.

I updated the spreadsheet: https://docs.google.com/spreadsheets/d/1Hgv3waJxzBMuHiFIDyWBSNJqPO5ILHUYhWLnYfAwvGM/edit?usp=sharing
I gave Green to the best answer, Yellow to an acceptable answer, but not complete (from what I could tell), Blue for an acceptable answer, but maybe not accurate, and Red for an answer that is wrong. I only subtracted credit for the completely wrong answers.

Fri, Nov 22, 6:31 PM · Anti-Harassment, CheckUser

Thu, Nov 21

Niharika renamed T237298: CheckUser 2.0: Display the data fetched on special page (preliminary check) from CheckUser 2.0: Display the data fetched on special page to CheckUser 2.0: Display the data fetched on special page (preliminary check).
Thu, Nov 21, 5:51 PM · Patch-For-Review, Anti-Harassment (The Letter Song), CheckUser
Niharika updated the task description for T238714: CU 2.0: Fetch information to be displayed in Compare tab.
Thu, Nov 21, 5:37 PM · Patch-For-Review, Anti-Harassment (The Letter Song), CheckUser
Niharika moved T237300: CU 2.0: Sorting in the Compare tab from Cards ready for development to The Letter Song on the Anti-Harassment board.
Thu, Nov 21, 5:36 PM · Anti-Harassment (The Letter Song), CheckUser
Niharika moved T237300: CU 2.0: Sorting in the Compare tab from Cards ready to be discussed to Cards ready for development on the Anti-Harassment board.
Thu, Nov 21, 5:35 PM · Anti-Harassment (The Letter Song), CheckUser
Niharika set the point value for T237300: CU 2.0: Sorting in the Compare tab to 3.
Thu, Nov 21, 5:35 PM · Anti-Harassment (The Letter Song), CheckUser
Niharika moved T238782: CU 2.0: Display data in the Compare tab from Cards ready for development to Design on the Anti-Harassment board.
Thu, Nov 21, 5:32 PM · Anti-Harassment, CheckUser
Niharika moved T238714: CU 2.0: Fetch information to be displayed in Compare tab from Cards ready for development to The Letter Song on the Anti-Harassment board.
Thu, Nov 21, 5:32 PM · Patch-For-Review, Anti-Harassment (The Letter Song), CheckUser
Niharika moved T238827: Reorganise preliminary check service for paginated results from Cards ready for development to The Letter Song on the Anti-Harassment board.
Thu, Nov 21, 5:32 PM · Patch-For-Review, Anti-Harassment (The Letter Song), CheckUser
Niharika moved T238782: CU 2.0: Display data in the Compare tab from Cards ready to be discussed to Cards ready for development on the Anti-Harassment board.
Thu, Nov 21, 5:32 PM · Anti-Harassment, CheckUser
Niharika set the point value for T238782: CU 2.0: Display data in the Compare tab to 2.
Thu, Nov 21, 5:29 PM · Anti-Harassment, CheckUser
Niharika moved T238714: CU 2.0: Fetch information to be displayed in Compare tab from Cards ready to be discussed to Cards ready for development on the Anti-Harassment board.
Thu, Nov 21, 5:27 PM · Patch-For-Review, Anti-Harassment (The Letter Song), CheckUser
Niharika updated the task description for T238782: CU 2.0: Display data in the Compare tab.
Thu, Nov 21, 5:24 PM · Anti-Harassment, CheckUser
Niharika moved T238827: Reorganise preliminary check service for paginated results from Cards ready to be discussed to Cards ready for development on the Anti-Harassment board.
Thu, Nov 21, 5:19 PM · Patch-For-Review, Anti-Harassment (The Letter Song), CheckUser
Niharika set the point value for T238827: Reorganise preliminary check service for paginated results to 3.
Thu, Nov 21, 5:18 PM · Patch-For-Review, Anti-Harassment (The Letter Song), CheckUser
Niharika added a comment to T175587: Add a user-agent parser to CheckUser.

It would be helpful to quantify how often we got accurate results from both the libraries.

What do you mean? errr... how do you define accuracy? Can you provide an example?

Thu, Nov 21, 5:07 PM · Anti-Harassment, CheckUser
Niharika added a comment to T175587: Add a user-agent parser to CheckUser.

I retrieved 20 random User Agent strings from the cu_changes table on eswiki and put them into a spreadsheet and compared the parsing of devicedetector.net and whichbrowser.net.
The results can be seen here:
https://docs.google.com/spreadsheets/d/1Hgv3waJxzBMuHiFIDyWBSNJqPO5ILHUYhWLnYfAwvGM/edit?usp=sharing
whichbrowser.net (and their free library) seems a lot better than the alternative (on average).

Thu, Nov 21, 4:58 PM · Anti-Harassment, CheckUser
Niharika moved T238782: CU 2.0: Display data in the Compare tab from Design to Cards ready to be discussed on the Anti-Harassment board.
Thu, Nov 21, 4:50 PM · Anti-Harassment, CheckUser
Niharika moved T238714: CU 2.0: Fetch information to be displayed in Compare tab from The Letter Song to Cards ready to be discussed on the Anti-Harassment board.
Thu, Nov 21, 4:50 PM · Patch-For-Review, Anti-Harassment (The Letter Song), CheckUser
Niharika triaged T238827: Reorganise preliminary check service for paginated results as Medium priority.
Thu, Nov 21, 4:48 PM · Patch-For-Review, Anti-Harassment (The Letter Song), CheckUser
Niharika moved T237299: CU 2.0: Sticky highlighting in Preliminary check and Compare tabs from Design to Cards ready to be discussed on the Anti-Harassment board.
Thu, Nov 21, 4:29 PM · CheckUser, Anti-Harassment
Niharika moved T237595: [Epic] CheckUser 2.0: Timeline from Untriaged to Backlog on the Anti-Harassment board.
Thu, Nov 21, 1:58 AM · Epic, Anti-Harassment, CheckUser
Niharika triaged T238809: CU 2.0: Filters in the Compare tab as Medium priority.
Thu, Nov 21, 1:53 AM · Anti-Harassment, CheckUser
Niharika moved T238809: CU 2.0: Filters in the Compare tab from Untriaged to Cards ready to be discussed on the Anti-Harassment board.
Thu, Nov 21, 1:52 AM · Anti-Harassment, CheckUser
Niharika created T238809: CU 2.0: Filters in the Compare tab.
Thu, Nov 21, 1:52 AM · Anti-Harassment, CheckUser
Niharika updated subscribers of T238603: Placeholder text on Special:Investigate seems unnecessary.

The only difference between this field and that one, is that this field allows multiple values. As far as I know, everyone who has access to CheckUser should also have access to Block, so everyone is already familiar with this pattern.

While this is true, I would not presume everyone using CheckUser does frequent Special:Block too. I'm kinda leaning towards Prateek's point about providing some more guidance here as this UI is pretty different from what they are using currently.
@cwylo Do you have an opinion on this ticket?

Thu, Nov 21, 1:26 AM · CheckUser, Anti-Harassment
Niharika triaged T238808: CU 2.0: Pagination in the compare tab as Medium priority.
Thu, Nov 21, 1:21 AM · Anti-Harassment, CheckUser
Niharika moved T238808: CU 2.0: Pagination in the compare tab from Untriaged to Cards ready to be discussed on the Anti-Harassment board.
Thu, Nov 21, 1:20 AM · Anti-Harassment, CheckUser
Niharika created T238808: CU 2.0: Pagination in the compare tab.
Thu, Nov 21, 1:20 AM · Anti-Harassment, CheckUser
Niharika added a comment to T237034: CheckUser 2.0: Input form.

Thanks for that information, @JJMC89. As I mentioned, we have no intention of doing that. :)

Thu, Nov 21, 1:09 AM · MW-1.35-notes (1.35.0-wmf.8; 2019-11-26), Anti-Harassment (The Letter Song), CheckUser

Wed, Nov 20

Niharika assigned T238782: CU 2.0: Display data in the Compare tab to Prtksxna.
Wed, Nov 20, 8:10 PM · Anti-Harassment, CheckUser
Niharika assigned T237299: CU 2.0: Sticky highlighting in Preliminary check and Compare tabs to Prtksxna.

@Prtksxna assigning this to you for the mock and design specs.

Wed, Nov 20, 8:10 PM · CheckUser, Anti-Harassment
Niharika renamed T237299: CU 2.0: Sticky highlighting in Preliminary check and Compare tabs from Allow the highlight to be "locked" if a user clicks to CU 2.0: Sticky highlighting in Preliminary check and Compare tabs.
Wed, Nov 20, 8:09 PM · CheckUser, Anti-Harassment
Niharika removed a subtask for T237039: [Epic] CheckUser 2.0: Preliminary check: T237300: CU 2.0: Sorting in the Compare tab.
Wed, Nov 20, 7:42 PM · Epic, CheckUser, Anti-Harassment
Niharika added a subtask for T237593: [Epic] CheckUser 2.0: Compare: T237300: CU 2.0: Sorting in the Compare tab.
Wed, Nov 20, 7:42 PM · Epic, Anti-Harassment, CheckUser
Niharika edited parent tasks for T237300: CU 2.0: Sorting in the Compare tab, added: T237593: [Epic] CheckUser 2.0: Compare; removed: T237039: [Epic] CheckUser 2.0: Preliminary check.
Wed, Nov 20, 7:42 PM · Anti-Harassment (The Letter Song), CheckUser
Niharika renamed T237300: CU 2.0: Sorting in the Compare tab from Add filtering, sorting, pagination(?) to the data table to CU 2.0: Sorting in the Compare tab.
Wed, Nov 20, 7:42 PM · Anti-Harassment (The Letter Song), CheckUser
Niharika moved T238782: CU 2.0: Display data in the Compare tab from Untriaged to Design on the Anti-Harassment board.

@Prtksxna Putting this in your column for the couple mocks needed.

Wed, Nov 20, 7:09 PM · Anti-Harassment, CheckUser
Niharika created T238782: CU 2.0: Display data in the Compare tab.
Wed, Nov 20, 7:07 PM · Anti-Harassment, CheckUser
Niharika updated the task description for T238714: CU 2.0: Fetch information to be displayed in Compare tab.
Wed, Nov 20, 6:49 PM · Patch-For-Review, Anti-Harassment (The Letter Song), CheckUser
Niharika moved T238714: CU 2.0: Fetch information to be displayed in Compare tab from Cards ready for development to The Letter Song on the Anti-Harassment board.
Wed, Nov 20, 6:36 PM · Patch-For-Review, Anti-Harassment (The Letter Song), CheckUser
Niharika moved T238714: CU 2.0: Fetch information to be displayed in Compare tab from Untriaged to Cards ready for development on the Anti-Harassment board.
Wed, Nov 20, 6:36 PM · Patch-For-Review, Anti-Harassment (The Letter Song), CheckUser
Niharika triaged T238714: CU 2.0: Fetch information to be displayed in Compare tab as Medium priority.
Wed, Nov 20, 6:35 PM · Patch-For-Review, Anti-Harassment (The Letter Song), CheckUser
Niharika updated the task description for T237593: [Epic] CheckUser 2.0: Compare.
Wed, Nov 20, 6:23 PM · Epic, Anti-Harassment, CheckUser
Niharika created T238714: CU 2.0: Fetch information to be displayed in Compare tab.
Wed, Nov 20, 12:55 AM · Patch-For-Review, Anti-Harassment (The Letter Song), CheckUser
Niharika added a comment to T237034: CheckUser 2.0: Input form.

@Niharika the field is short for a textual description and a link. Either a longer 2-line textfield or 2 fields would be enough.
In any case, the comments should instruct to paste the link to the cause and form validation should check its presence.
The check is a bit simpler with a separate field, but it is less busy with one longer field.

Wed, Nov 20, 12:30 AM · MW-1.35-notes (1.35.0-wmf.8; 2019-11-26), Anti-Harassment (The Letter Song), CheckUser
Niharika added a comment to T175587: Add a user-agent parser to CheckUser.

I see. I think we need some baseline estimate of how often it is wrong. If it's like 1 in 10 cases, then maybe we could split up the UA and give them a disclaimer that it might be wrong and give them an option to see the complete UA. If it's 1 in 2 cases, we probably don't want to do the splitting.

makes sense. I'm not sure how we would figure that out other than taking some random UAs and testing them manually.

Wed, Nov 20, 12:29 AM · Anti-Harassment, CheckUser
Niharika added a comment to T200938: Special:CentralAuth should provide the same blocking information as Special:BlockList does.

or rather, the problem does not exist on he, it only exists on en.

Wed, Nov 20, 12:28 AM · MW-1.35-notes (1.35.0-wmf.5; 2019-11-05), Core Platform Team Workboards (Clinic Duty Team), Anti-Harassment (The Letter Song), Patch-For-Review, MediaWiki-extensions-CentralAuth, Stewards-and-global-tools, MediaWiki-User-management
Niharika added a comment to T175587: Add a user-agent parser to CheckUser.

Hmm, if we can have a sense of how inaccurate a parsed UA is (say, 80% or 20%) then based on that we can decide whether to display the complete UA or not. One of the use cases of this is to also know when the user might have spoofed their UA.

Hmm... I guess since it's three parts, I guess we are really saying how accurate each part is.. I'm not sure you can tell this without looking at it.
For instance, with the UA generated in T237669#5661529
throwing WikipediaApp/2.7.50302-r-2019-11-13 (Android 8.1.0; Phone; Nexus 6P Build/OPM6.171019.030.E1) Alpha Channel into https://whichbrowser.net/tryout/ we get: Android Browser on a Huawei Nexus 6P running Android 8.1.0. two of those parts are correct, but one isn't.

Wed, Nov 20, 12:25 AM · Anti-Harassment, CheckUser
Niharika added a comment to T200938: Special:CentralAuth should provide the same blocking information as Special:BlockList does.

@dbarratt Has this been deployed already? In that case we can verify if the problem exists on the actual hewiki.

Wed, Nov 20, 12:22 AM · MW-1.35-notes (1.35.0-wmf.5; 2019-11-05), Core Platform Team Workboards (Clinic Duty Team), Anti-Harassment (The Letter Song), Patch-For-Review, MediaWiki-extensions-CentralAuth, Stewards-and-global-tools, MediaWiki-User-management
Niharika updated subscribers of T175587: Add a user-agent parser to CheckUser.

Thanks for doing an investigation on this, @dbarratt.

One strategy could be to utilize a parsing library. Sadly, a lot of user agents lie, so it might not be completely accurate.

Hmm, if we can have a sense of how inaccurate a parsed UA is (say, 80% or 20%) then based on that we can decide whether to display the complete UA or not. One of the use cases of this is to also know when the user might have spoofed their UA.

Wed, Nov 20, 12:10 AM · Anti-Harassment, CheckUser
Niharika updated subscribers of T237669: Expose the device in the user-agent from the app requests.

@Dbrant @Charlotte Yes, that seems about right. Thanks for picking this up!

Wed, Nov 20, 12:03 AM · Wikipedia-iOS-App-Backlog, iOS-app-Bugs, Anti-Harassment, CheckUser
Niharika added a comment to T237034: CheckUser 2.0: Input form.

The cause of doing CU is often an SPI or a user request or suspicious edit. In any case, the reason should link to this cause and the form should require this link for the reason.
Would a separate field for the link be better or a two-line reason field?
With two separate fields I think it would be cleaner:

Wed, Nov 20, 12:02 AM · MW-1.35-notes (1.35.0-wmf.8; 2019-11-26), Anti-Harassment (The Letter Song), CheckUser

Tue, Nov 19

Niharika added a comment to T237593: [Epic] CheckUser 2.0: Compare.
  • Getting the data, sorting and pagination
  • Display the data
  • Building the filters and filtering
  • Adding the new data records
Tue, Nov 19, 7:40 PM · Epic, Anti-Harassment, CheckUser
Niharika updated the task description for T237593: [Epic] CheckUser 2.0: Compare.
Tue, Nov 19, 7:01 PM · Epic, Anti-Harassment, CheckUser
Niharika updated the task description for T237593: [Epic] CheckUser 2.0: Compare.
Tue, Nov 19, 6:55 PM · Epic, Anti-Harassment, CheckUser
Niharika triaged T237593: [Epic] CheckUser 2.0: Compare as Medium priority.
Tue, Nov 19, 6:39 PM · Epic, Anti-Harassment, CheckUser

Mon, Nov 18

Niharika moved T238603: Placeholder text on Special:Investigate seems unnecessary from Untriaged to Cards ready to be discussed on the Anti-Harassment board.

@Prtksxna Flagging this for you.

Mon, Nov 18, 10:40 PM · CheckUser, Anti-Harassment
Niharika added a comment to T238561: [Safari 10] opacity problem for RC filters vertical scrolling bar .

I actually kinda like this bug. :)

Mon, Nov 18, 10:15 PM · Growth-Team, MediaWiki-Recent-changes
Niharika added a comment to T237034: CheckUser 2.0: Input form.

@Prtksxna This seems really busy to me...


the placeholder text seems unnecessary.... do we even need it at all?

Mon, Nov 18, 7:37 PM · MW-1.35-notes (1.35.0-wmf.8; 2019-11-26), Anti-Harassment (The Letter Song), CheckUser

Fri, Nov 15

Niharika updated the task description for T237593: [Epic] CheckUser 2.0: Compare.
Fri, Nov 15, 5:28 PM · Epic, Anti-Harassment, CheckUser
Niharika updated the task description for T237593: [Epic] CheckUser 2.0: Compare.
Fri, Nov 15, 4:47 PM · Epic, Anti-Harassment, CheckUser
Niharika updated the task description for T237593: [Epic] CheckUser 2.0: Compare.
Fri, Nov 15, 4:39 PM · Epic, Anti-Harassment, CheckUser
Niharika updated the task description for T237593: [Epic] CheckUser 2.0: Compare.
Fri, Nov 15, 4:34 PM · Epic, Anti-Harassment, CheckUser
Niharika added a comment to T237296: CheckUser 2.0: Create service and methods for new CU extension.

@Niharika What is the acceptance criteria for this task? Should ip users also be supported as part of this or is it supporting only registered accounts enough?

Fri, Nov 15, 4:00 PM · MW-1.35-notes (1.35.0-wmf.8; 2019-11-26), Patch-For-Review, Anti-Harassment (The Letter Song), CheckUser
Niharika moved T237593: [Epic] CheckUser 2.0: Compare from Untriaged to Backlog on the Anti-Harassment board.
Fri, Nov 15, 3:34 PM · Epic, Anti-Harassment, CheckUser

Thu, Nov 14

Niharika added a comment to T237034: CheckUser 2.0: Input form.

@Niharika @Prtksxna A few questions about the acceptance criteria:

  • Input box for usernames. [...]
    • Placeholder: Add users

Should the placeholder be the default "Add more..." instead? Might make more sense since this is not just for adding users.

Yeah, it can be a user or IP address. I'm a bit iffy about Add more... as it kinda means they already added something. How about Add username or IP address instead?

  • Input box for the reason for the check.
    • [...]
    • Placeholder text: Example: Investigating Apples for suspected sockpuppetry

Are we OK with the fact that Apples can be a real username?

Ooh, good catch! It could be Investigating User:... for suspected sockpuppetry. @Prtksxna thoughts?

Finally, just wanted to ask whether we should make the help text inline? It's the default and recommended for accessibility, though it does make the page a bit busier:

inline: trueinline: false

Should be @Prtksxna's call.

Thu, Nov 14, 4:34 PM · MW-1.35-notes (1.35.0-wmf.8; 2019-11-26), Anti-Harassment (The Letter Song), CheckUser

Tue, Nov 12

Niharika updated the task description for T234557: Get rough stats for time taken to resolve a case from CU log.
Tue, Nov 12, 10:31 PM · Anti-Harassment (The Letter Song), CheckUser
Niharika added a comment to T233996: Mobile block notice not showing block reason or block log entry for global blocks.

@Tchanders Should we poke more people for code review on this?

Tue, Nov 12, 10:17 PM · MW-1.35-notes (1.35.0-wmf.8; 2019-11-26), Anti-Harassment, Stewards-and-global-tools, VisualEditor, Mobile, MediaWiki-User-management
Niharika moved T175587: Add a user-agent parser to CheckUser from Untriaged to Cards ready to be discussed on the Anti-Harassment board.
Tue, Nov 12, 8:44 PM · Anti-Harassment, CheckUser
Niharika added a project to T175587: Add a user-agent parser to CheckUser: Anti-Harassment.
Tue, Nov 12, 8:44 PM · Anti-Harassment, CheckUser
Niharika updated the task description for T236981: CheckUser 2.0: Create a new Special page in CheckUser for the redesigned version.
Tue, Nov 12, 3:56 PM · Patch-For-Review, MW-1.35-notes (1.35.0-wmf.8; 2019-11-26), Anti-Harassment (The Letter Song), CheckUser
Niharika added a comment to T236981: CheckUser 2.0: Create a new Special page in CheckUser for the redesigned version.

Proposed acceptance criteria in more detail:

  • Name tbc

I updated the description with the name after the previous round of comments. It's Special:Investigate.

  • If config is set to unavailable, the special page URL points to a page with the title "No such special page" (similar to Special:Mute)

Yep.

  • If config is set to available:
    • Unblocked user with 'checkuser' right sees the page title and a submit button that does nothing
    • Blocked user with 'checkuser' right sees a user blocked error (similar to Special:CheckUser)
    • User with 'checkuser' right sees the special page listed at Special:SpecialPages under the section "Users and rights" (similar to Special:CheckUser)
    • User without 'checkuser' right sees a permissions error

Yep, all this sounds good. I will copy this to the task description. Thanks for more detail.

Tue, Nov 12, 3:55 PM · Patch-For-Review, MW-1.35-notes (1.35.0-wmf.8; 2019-11-26), Anti-Harassment (The Letter Song), CheckUser

Nov 8 2019

Niharika added a comment to T236981: CheckUser 2.0: Create a new Special page in CheckUser for the redesigned version.

Imo CheckUserTest or CheckUserDev or CheckUserAlpha. I assume when it goes to production (new version enabled in config) it will replace the current CheckUser and this in-development name will be dropped.

Nov 8 2019, 10:07 PM · Patch-For-Review, MW-1.35-notes (1.35.0-wmf.8; 2019-11-26), Anti-Harassment (The Letter Song), CheckUser
Niharika updated the task description for T236981: CheckUser 2.0: Create a new Special page in CheckUser for the redesigned version.
Nov 8 2019, 10:05 PM · Patch-For-Review, MW-1.35-notes (1.35.0-wmf.8; 2019-11-26), Anti-Harassment (The Letter Song), CheckUser
Niharika added a comment to T236981: CheckUser 2.0: Create a new Special page in CheckUser for the redesigned version.

@AronManning The redesigned version is so radically different that I don't expect folks to switch back and forth often. The idea is that we will remove the older page at some point and the new page will just be Special:CheckUser. Until then I would like the page name to be distinctive enough so when we share it people do not confuse the new page with the old.
I ran this by our Community Engagement folks and their recommendation was to go with Special:Investigate that Prateek chose for the mocks. So let's go with that.
@dbarratt ^ I'll update the ticket.

Nov 8 2019, 10:04 PM · Patch-For-Review, MW-1.35-notes (1.35.0-wmf.8; 2019-11-26), Anti-Harassment (The Letter Song), CheckUser
Niharika added a comment to T174553: Create a mechanism that allows fetching geolocation and subnet data for IP addresses.

@Niharika - This is something that AHT team wants, correct? Huji's already done some work to add the geoip2 library and some related MaxMind components as a dependency to core (which actually dates back to our work on LoginNotify), so we could go ahead and request an initial security review, if this seems like it would be useful to you (for CentralAuth and/or LoginNotify).

Nov 8 2019, 10:00 PM · User-Daimona, Patch-For-Review, User-Huji, MediaWiki-extension-requests, Stewards-and-global-tools, MediaWiki-extensions-LoginNotify, CheckUser
Niharika added a comment to T237595: [Epic] CheckUser 2.0: Timeline.

@AronManning Don't worry, I can take care of adding the mocks and expanding the task description. I think the mocks are likely to change based on the on-wiki feedback, hence I'm holding back on adding those for now.

Nov 8 2019, 1:59 AM · Epic, Anti-Harassment, CheckUser

Nov 7 2019

Niharika added a comment to T237669: Expose the device in the user-agent from the app requests.

This isn't very difficult to add, but wouldn't it have security implications? (i.e. wouldn't it reduce the anonymity of the user?)

Nov 7 2019, 11:26 PM · Wikipedia-iOS-App-Backlog, iOS-app-Bugs, Anti-Harassment, CheckUser
Niharika updated subscribers of T94735: Rewrite Special:CheckUser using server side templates.

@AronManning Can you provide more context on this ticket? I'm not sure what's being requested and why.

Nov 7 2019, 10:55 PM · Epic, Stewards-and-global-tools, CheckUser
Niharika moved T237298: CheckUser 2.0: Display the data fetched on special page (preliminary check) from Cards ready to be discussed to The Letter Song on the Anti-Harassment board.
Nov 7 2019, 10:16 PM · Patch-For-Review, Anti-Harassment (The Letter Song), CheckUser
Niharika moved T237034: CheckUser 2.0: Input form from Cards ready for development to The Letter Song on the Anti-Harassment board.
Nov 7 2019, 10:15 PM · MW-1.35-notes (1.35.0-wmf.8; 2019-11-26), Anti-Harassment (The Letter Song), CheckUser
Niharika moved T237296: CheckUser 2.0: Create service and methods for new CU extension from Cards ready for development to The Letter Song on the Anti-Harassment board.
Nov 7 2019, 10:15 PM · MW-1.35-notes (1.35.0-wmf.8; 2019-11-26), Patch-For-Review, Anti-Harassment (The Letter Song), CheckUser
Niharika moved T236981: CheckUser 2.0: Create a new Special page in CheckUser for the redesigned version from Cards ready for development to The Letter Song on the Anti-Harassment board.
Nov 7 2019, 10:15 PM · Patch-For-Review, MW-1.35-notes (1.35.0-wmf.8; 2019-11-26), Anti-Harassment (The Letter Song), CheckUser
Niharika closed T236862: Add tests to OutputPage::sendCacheControl, a subtask of T236751: Allow OutputPage to set maxage, as Resolved.
Nov 7 2019, 10:10 PM · Anti-Harassment, MediaWiki-General, Patch-For-Review
Niharika closed T236862: Add tests to OutputPage::sendCacheControl as Resolved.

Resolving this as it seems low risk and Dom already has a bunch on his plate (column).

Nov 7 2019, 10:10 PM · MW-1.35-notes (1.35.0-wmf.8; 2019-11-26), Anti-Harassment (The Letter Song), MediaWiki-General