Page MenuHomePhabricator

How should reading lists work for temporary account users?
Open, LowPublicBUG REPORT

Description

In T344723 we disabled the reading list API and storage for IP masked users to be consistent with the existing behaviour for anonymous users.
Currently reading lists are only stored for logged in users.

Questions to answer

  • What does the workflow look like for IP masked users?
  • Is there a reason you would want to store reading lists in the database for temporary users instead of local to the device?

Things to consider

  • What happens to a user's reading list when the temporary account expires or the user "logs out"?
  • Are there any benefits for storing in the database over locally to the device?
  • Any privacy concerns to think about?
  • Is there bandwidth for the content transform team to make any required and desired changes to the API?
  • Are there any additional moderation requirements?

Event Timeline

Jdlrobson updated the task description. (Show Details)

Change 973333 had a related patch set uploaded (by Jdlrobson; author: Seddon):

[mediawiki/extensions/ReadingLists@master] Revert "Reading lists should not be available to IP masked users"

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

Change 973334 had a related patch set uploaded (by Jdlrobson; author: Seddon):

[mediawiki/extensions/ReadingLists@master] Revert "Disallow temporary users from using the APIs"

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

LGoto triaged this task as Low priority.Nov 14 2023, 5:07 PM
LGoto moved this task from Needs Triage to Product Backlog on the Wikipedia-Android-App-Backlog board.
LGoto added subscribers: JTannerWMF, Seddon.
kostajh renamed this task from How should reading lists work for IP masked users? to How should reading lists work for temporary account users?.Nov 16 2023, 11:50 AM

What does the workflow look like for IP masked users?

AIUI the workflow would be the same as for anonymous users: reading lists would be stored locally, and the user would be encouraged to create a (permanent) account so that their lists could be synced.

What happens to a user's reading list when the temporary account expires or the user "logs out"?

The reading list would persist on the device. (This is the current behavior when a user logs out of their account - their reading lists are still present locally). If the user creates an account, their reading list would be synced automatically from the device to the db.

Welcome back :)
My understanding is that will require some changes in the app but not MediaWiki core. Is that correct?

My understanding is that will require some changes in the app but not MediaWiki core. Is that correct?

Yep, you're correct - all of the above logic will be done on the client side.

Change 973334 abandoned by Jdlrobson:

[mediawiki/extensions/ReadingLists@master] Revert "Disallow temporary users from using the APIs"

Reason:

Per ticket it doesn't sound like this will be needed. Please reach out if I've misunderstood anything.

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

Change #973333 abandoned by Jdlrobson:

[mediawiki/extensions/ReadingLists@master] Revert "Reading lists should not be available to IP masked users"

Reason:

I think this can safely be abandoned? I'm out next week but let's chat in 3 weeks time if not.

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