Page MenuHomePhabricator

Mentor dashboard: Implement "empty screen" window
Closed, ResolvedPublic

Description

Figma (under "Alternative V1") specifies how the dashboard should look like with zero mentees.

That screen is not currently implemented.

See [Figma](https://www.figma.com/file/C24jqGCSzlaQ4k0WXGjIbb/V1-mobile-desktop?node-id=889:26019 for details.

Initial description

(1)

  • On Special:MentorDashboard click Add filter
  • Add a value to From field that will be less than the values added to To field, e.g. From = 2, To =1
  • Click to submit

The pagination looks confusing: 1 of 0 :

Screen Shot 2021-08-26 at 3.01.23 PM.png (1×1 px, 174 KB)

(2)

  • If a user was on a different page (before applying filters that would return zero results) , than the page number will be different than 1:

Screen Shot 2021-08-26 at 3.29.21 PM.png (678×1 px, 83 KB)

(3) It's even possible make the final page number in [the initial page number] of [the final page number] look different from zero even though that number of results is zero.

With default filters go to some page - in the screenshot below, a user is on the page 4 with default filtersA user applies time filterA user applies "Only show starred mentees" - the pagination maintains the initial page number (the page 4), but somehow calculates that there is another page (the number 1)
Screen Shot 2021-08-26 at 3.35.33 PM.png (1×1 px, 296 KB)
Screen Shot 2021-08-26 at 3.36.11 PM.png (1×1 px, 228 KB)
Screen Shot 2021-08-26 at 3.36.27 PM.png (904×1 px, 112 KB)

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript
Urbanecm_WMF renamed this task from Mentor dashboard: Pagination issues when zero results are returned to Mentor dashboard: Implement "empty screen" window.Aug 27 2021, 1:36 PM
Urbanecm_WMF triaged this task as Low priority.
Urbanecm_WMF updated the task description. (Show Details)

Change 715239 had a related patch set uploaded (by Urbanecm; author: Urbanecm):

[mediawiki/extensions/GrowthExperiments@master] Mentee overview: Implement empty screen window

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

Thanks for filling this @Etonkovidova. This should likely be fixed by just following this Figma page.

@OTichonova I think that the screen should be different when mentor just specifies filters that don't match anything, right? Unfortunately, I don't see screen for that in the mockup.

How useful is this message given that there never should be a time when mentors don't see mentees because the data is being collected, except the first day after deployment? OTOH it's not impossible for a mentor to just not have mentees, in which case this message is misleading.

How useful is this message given that there never should be a time when mentors don't see mentees because the data is being collected, except the first day after deployment?

I don't think that's true. Any mentor who visits their dashboard immediately after signing up (or at least, before the next update finishes) will see zero mentees, because the updating script only processes mentors who are on the list of mentors.

OTOH it's not impossible for a mentor to just not have mentees, in which case this message is misleading.

Not impossible, but I'd say it's unlikely. I just ran the following query on all Growth wikis to get an idea:

WITH mentees_by_mentor AS (
    SELECT
        gemm_mentor_id,
        COUNT(*) AS mentees
    FROM growthexperiments_mentor_mentee
    GROUP BY gemm_mentor_id
)

SELECT
    DATABASE() AS wiki,
    MIN(mentees) AS min_mentees,
    SUM(mentees) AS total_mentees
FROM mentees_by_mentor

This is what I got:

wikimin_menteestotal_mentees
arwiki1233138.0
azwiki184.0
bnwiki111928.0
cswiki130778.0
testwiki1585.0
enwiki1406241.0
kowiki146586.0
hewiki121497.0
ckbwiki211.0
tewiki3659.0
kuwiki36.0
idwiki722316.0
ukwiki1226239.0
viwiki2076530.0
svwiki2312450.0
rowiki245082.0
ptwiki27113530.0
fawiki2873420.0
trwiki5057830.0
frwiki55188638.0
frwiktionary813148.0
ruwiki101117824.0
hrwiki1021287.0
hywiki1294071.0
plwiki21926610.0
srwiki3946955.0
dewiki3972899.0
huwiki40615876.0
skwiki513513.0
euwiki5303410.0
elwiki6041853.0
nlwiki6282942.0
afwiki00.0
alswiki00.0
anwiki00.0
arzwiki00.0
astwiki00.0
aswiki00.0
azbwiki00.0
barwiki00.0
bawiki00.0
be_x_oldwiki00.0
bewiki00.0
bgwiki00.0
brwiki00.0
bswiki00.0
cawiki00.0
cebwiki00.0
cywiki00.0
dawiki00.0
eowiki00.0
eswiki00.0
etwiki00.0
fiwiki00.0
gawiki00.0
glwiki00.0
guwiki00.0
hiwiki00.0
iswiki00.0
jawiki00.0
jvwiki00.0
kawiki00.0
kkwiki00.0
kmwiki00.0
knwiki00.0
kywiki00.0
lawiki00.0
lbwiki00.0
ltwiki00.0
lvwiki00.0
minwiki00.0
mkwiki00.0
mlwiki00.0
mnwiki00.0
mrwiki00.0
mswiki00.0
mywiki00.0
newiki00.0
nnwiki00.0
nowiki00.0
ocwiki00.0
pawiki00.0
scowiki00.0
shwiki00.0
simplewiki00.0
siwiki00.0
slwiki00.0
sowiki00.0
sqwiki00.0
suwiki00.0
swwiki00.0
tawiki00.0
thwiki00.0
tlwiki00.0
ttwiki00.0
urwiki00.0
warwiki00.0
zh_classicalwiki00.0
zh_min_nanwiki00.0
zh_yuewiki00.0

The zeros are very likely caused by mentorship module not being enabled. Apart from that, the lowest number is one. I spot checked that, and it is caused by me mentoring myself via my WMF account.

I expect numbers will get smaller once we deploy to the long tail of small wikis. Good point about new mentors' first day though.

Maybe the text could be rephrased to say something like "You have no mentees. This might be because it takes up to a day for mentee data to be updated"?

It would be possible to differentiate between no results vs. an empty result set (e.g. store last update timestamp as a user preference), but it seems more effort than worth it.

Urbanecm_WMF raised the priority of this task from Low to Medium.Sep 2 2021, 10:42 AM

Change 715239 merged by jenkins-bot:

[mediawiki/extensions/GrowthExperiments@master] Mentee overview: Implement empty screen window

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

Change 717588 had a related patch set uploaded (by Urbanecm; author: Urbanecm):

[mediawiki/extensions/GrowthExperiments@master] Mentor dashboard: Update wording for empty screen

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

Change 717588 merged by jenkins-bot:

[mediawiki/extensions/GrowthExperiments@master] Mentor dashboard: Update wording for empty screen

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