Page MenuHomePhabricator

User education for reading lists and account creation: Alerts
Closed, ResolvedPublic

Assigned To
Authored By
cmadeo
Feb 14 2018, 11:34 PM
Referenced Files
F13937778: Article sync complete.png
Feb 21 2018, 11:50 PM
F13937775: Article limit reached.png
Feb 21 2018, 11:50 PM
F12210451: Splitting existing 1000+ lists.png
Feb 14 2018, 11:34 PM
F11793838: Syncing disrupted.png
Feb 14 2018, 11:34 PM
F10776485: Out of space.png
Feb 14 2018, 11:34 PM
F12210459: Saved list rename.png
Feb 14 2018, 11:34 PM
F10776359: 100 list limit.png
Feb 14 2018, 11:34 PM

Description

Why are we doing this?
User education will be an integral part of introducing Reading lists to users. User education will be especially necessary for explaining to users how to create or log into their account to enable reading list syncing across devices.

Alerts

LocationTypeMessagingWhen shown? Mock Zeplin
Saved tabAlertYou have reached the limit of 100 reading lists per accountNumber of lists reaches 100 list limit
100 list limit.png (375×667 px, 131 KB)
https://zpl.io/2p01yzo
AnyAlertYou do not have enough space on your device to save this articleUser attempts to save article when they are out of space on their device
Out of space.png (375×667 px, 192 KB)
https://zpl.io/25NEZKQ
AnyAlertSyncing will resume when internet connection is availableConnection dropped or disrupted during sync
Syncing disrupted.png (375×667 px, 191 KB)
https://zpl.io/a30kgex
User created 'Saved' list renamedAlertYour list ‘Saved’ has been renamed ‘Saved_[user created]’A user logs into an account with syncing turned on that is associated with an Android account with a user created (eg. non-default) reading list named 'Saved'
Saved list rename.png (375×667 px, 148 KB)
https://zpl.io/aM59Nq3
Splitting existing 5000+ article listsAlertThere is a limit of 5000 articles per reading list. Existing lists with more than this limit have been split into multiple listsA user logs into an account with syncing turned on that is associated with an Android account that has a pre-existing list with over 5000 articles in it
Splitting existing 1000+ lists.png (375×667 px, 151 KB)
https://zpl.io/aRoE76n
AnyAlertArticle cannot be added to this list. You have reached the limit of 5000 articles per list for list nameUser has reached the 5000 article limit for a reading list
Article limit reached.png (750×1,334 px, 512 KB)
https://zpl.io/V0Zq9Do
AnyAlertn articles and n reading lists synced from your accountSync taking longer than 5 seconds is completed
Article sync complete.png (750×1,334 px, 737 KB)
https://zpl.io/blNdrl0

Related Objects

Event Timeline

cmadeo triaged this task as Medium priority.
Mhurd removed Mhurd as the assignee of this task.Feb 20 2018, 9:22 PM
Mhurd subscribed.
cmadeo updated the task description. (Show Details)
cmadeo updated the task description. (Show Details)

@cmadeo should we somehow label the articles that weren't saved?

@cmadeo 👍 do you want different copy, like "Not saved" or something? 🤔

@NHarateh_WMF This is for when the user hits the User attempts to save article when they are out of space on their device scenario?
If we're still going to show the article in their saved list (eg. as a sort of stub) I think we should just use the same treatment as the article limit exceeded... alert. Perhaps the string should be Device limited exceeded, unable to sync article.

We could also just not fill in the save thumbnail and not 'save' the article to a list at all, which would alleviate this error case?

@cmadeo yes, that's the scenario - I was thinking about option B, i.e., not 'saving' the article to alleviate this error case but if user has a lot of articles that fit on device A and then tries to sync to device B, which doesn't have enough space, 'unsaving' articles on device B will result in data loss, so maybe the label would be a better solution? 🤔

@NHarateh_WMF, good point! Let's go with Option A then. I'll update the other ticket.

ABorbaWMF subscribed.

Tested this over the last few builds and the latest 5.8.0 (1388). I think I have every scenario covered except the device out of space warning, just because I don't have an easy way to simulate that. Looks good for me.