Change 815695 had a related patch set uploaded (by Sergio Gimeno; author: Sergio Gimeno):

[mediawiki/extensions/GrowthExperiments@master] Mentee overview(vue): allow to set empty values for edit count filters

https://gerrit.wikimedia.org/r/815695

Change 815695 merged by jenkins-bot:

[mediawiki/extensions/GrowthExperiments@master] MentorDashboard.Vue: allow setting empty values for edit count filters

https://gerrit.wikimedia.org/r/815695

@Urbanecm_WMF - please review the following:

(1)

The filtering dropdown interprets empty string as zeros, which can be confirmed by re-opening the dropdown again:

I assumed this as a reasonable default fallback for entering an empty string.

(2)

What should have happened instead?:
The filter dropdown interprets empty strings as "filter is not used". The module displays mentees with any editcount.

• some user with zero edits might have non-zero number of questions/reverts/blocks - and a mentor might be interested to see them on MentorDashboard. Right now there is no way to display such users, i.e. with zero edits + non-zero number of questions/reverts/blocks.
• when a mentor enters 0 (zero), there is no indication that it's non-valid input, but newcomers with zero edits won't be displayed. When filters are re-opened, the 0 (zero) input is not displayed, which might be confusing to users.

• my mentor test user (on enwiki betalabs ET402) won't see users who registered less two weeks ago and haven't done any edits.

(enwiki)>  select gemm_mentee_id, user_name, user_registration, user_editcount  from growthexperiments_mentor_mentee join user on gemm_mentee_id=user_id  where gemm_mentor_id=66675 and user_registration >='20220715162016'and user_editcount=0;
+----------------+------------------+-------------------+----------------+
| gemm_mentee_id | user_name        | user_registration | user_editcount |
+----------------+------------------+-------------------+----------------+
|          94633 | AdrianNxe1225    | 20220720145342    |              0 |
|          94681 | JacquelynCummins | 20220721135048    |              0 |
|          94751 | VWalters         | 20220722185422    |              0 |
|          94806 | WilburnManson11  | 20220724135512    |              0 |
|          94846 | SanfordBarge277  | 20220725002910    |              0 |
|          94892 | Michailmvx       | 20220725162817    |              0 |
|          94897 | IsobelBeak38162  | 20220725233523    |              0 |
+----------------+------------------+-------------------+----------------+
7 rows in set (0.002 sec)

The filter selection on Special:MentorDashboard is - 0-500 edits for last 6 months. There will be one exception - JacquelynCummins because that user was claimed via Special:ClaimMentee.

In T313263#8113551, @Etonkovidova wrote:

@Urbanecm_WMF - please review the following:

(1)

The filtering dropdown interprets empty string as zeros, which can be confirmed by re-opening the dropdown again:

I assumed this as a reasonable default fallback for entering an empty string.

I'd agree, but IMO, users should be able to remove a filter. It'd be possible to use 0-9999999999999999999999999999 (or similarly high number), but I think removing the filter by inserting empty strings is more obvious.

(2)

What should have happened instead?:
The filter dropdown interprets empty strings as "filter is not used". The module displays mentees with any editcount.

• some user with zero edits might have non-zero number of questions/reverts/blocks - and a mentor might be interested to see them on MentorDashboard. Right now there is no way to display such users, i.e. with zero edits + non-zero number of questions/reverts/blocks.

Per below, I think the module should support a zero in the edit count filter.

• when a mentor enters 0 (zero), there is no indication that it's non-valid input, but newcomers with zero edits won't be displayed. When filters are re-opened, the 0 (zero) input is not displayed, which might be confusing to users.

Thanks for catching this! I think zero should be a valid input, to let mentors to see information about their newcomers who are yet to make their first edit. I think the behavior should be:

• User enters zero for the filter: Only users with zero edits are shown (filter gets applied, zero is the value)
• User enters an empty string for the filter: No limit is applied (filter does not kick in)

In practice, the difference will be only meaningful for the "Maximum" field (as editcounts are at least 0), to let you represent both "mentees with any number of edits" and "mentees with exactly zero edits.

@Sgs I'm not sure if this is caused by the fix you uploaded as part of this task, or a new bug. Happy to fill a new task if that'd make things easier!

• my mentor test user (on enwiki betalabs ET402) won't see users who registered less two weeks ago and haven't done any edits.

(enwiki)>  select gemm_mentee_id, user_name, user_registration, user_editcount  from growthexperiments_mentor_mentee join user on gemm_mentee_id=user_id  where gemm_mentor_id=66675 and user_registration >='20220715162016'and user_editcount=0;
+----------------+------------------+-------------------+----------------+
| gemm_mentee_id | user_name        | user_registration | user_editcount |
+----------------+------------------+-------------------+----------------+
|          94633 | AdrianNxe1225    | 20220720145342    |              0 |
|          94681 | JacquelynCummins | 20220721135048    |              0 |
|          94751 | VWalters         | 20220722185422    |              0 |
|          94806 | WilburnManson11  | 20220724135512    |              0 |
|          94846 | SanfordBarge277  | 20220725002910    |              0 |
|          94892 | Michailmvx       | 20220725162817    |              0 |
|          94897 | IsobelBeak38162  | 20220725233523    |              0 |
+----------------+------------------+-------------------+----------------+
7 rows in set (0.002 sec)

The filter selection on Special:MentorDashboard is - 0-500 edits for last 6 months. There will be one exception - JacquelynCummins because that user was claimed via Special:ClaimMentee.

Unfortunately, I don't see this image (F35359355), and it looks like it has some context to understand this issue better.

In T313263#8119918, @Urbanecm_WMF wrote:

In T313263#8113551, @Etonkovidova wrote:

Unfortunately, I don't see this image (F35359355), and it looks like it has some context to understand this issue better.

The image seems to be public, let me know if it's still not visible. With the image I was trying to illustrate the discrepancy between what a query for mentees with zero edits and the registration date fetches and what the UI displays (i.e. no users with zero edits). The only user with zero edits - JacquelynCummins - is displayed because that user was claimed.

(enwiki)>  select gemm_mentee_id, user_name, user_registration, user_editcount  from growthexperiments_mentor_mentee join user on gemm_mentee_id=user_id  where gemm_mentor_id=66675 and user_registration >='20220715162016'and user_editcount=0;

Testing notes (what to test after the fix(es)):

What should have happened instead?:

The filter dropdown interprets empty strings as "filter is not used". The module displays mentees with any editcount.

(1)

The filter drop-down interprets empty strings as "filter is not used".

• there are default values for filters: Minimum: 1 Maximum: 500 - should they be displayed? Or the previous user's filter?
• the filter preserves 0 as a user's input

Current behavior on testwiki wmf.23:

• user Etonkoviova (a mentor) sees two users with default Min/Max: (1-500)
  
• the mentor changes Min/Max (16-500) - one user is displayed:

• the mentor removes any values for Min/Max fields - the fields are empty; clicks Update - three users are displayed (Maximum field is set to unlimited value?)

• the user re-opens the filters - the Min/Max fields are empty:

(2)

The module displays mentees with any edit count.

• make sure that automatically assigned users with zero counts are displayed

Change 821271 had a related patch set uploaded (by Sergio Gimeno; author: Sergio Gimeno):

[mediawiki/extensions/GrowthExperiments@master] Mentee overview(vue): allow zero as edit count filter value

https://gerrit.wikimedia.org/r/821271

Change 821271 merged by jenkins-bot:

[mediawiki/extensions/GrowthExperiments@master] Mentee overview(vue): allow zero as edit count filter value

https://gerrit.wikimedia.org/r/821271

Change 819647 had a related patch set uploaded (by Sergio Gimeno; author: Sergio Gimeno):

[mediawiki/extensions/GrowthExperiments@master] Mentee overview(vue): use number input for the edit count filters

https://gerrit.wikimedia.org/r/819647

Change 819647 merged by jenkins-bot:

[mediawiki/extensions/GrowthExperiments@master] Mentee overview(vue): use number input for the edit count filters

https://gerrit.wikimedia.org/r/819647