Page MenuHomePhabricator

[BUG] Turn on reading list syncing? panel is presented when sync is already enabled
Closed, ResolvedPublic

Description

NOTE: Similar to T189015 but steps to reproduce are different so I'm creating another ticket.

How many times were you able to reproduce it?

5/5

Steps to reproduce

  1. Open the app, log in and enable sync
  2. Re-install the app
  3. Open the app

Expected results

I'm not prompted to enable sync because I'm already logged in and sync is enabled

Actual results

I'm prompted to enable sync

Screenshots

Environments observed

App version: 1370
OS versions: 11.2
Device model: iPhone X
Device language: EN

Event Timeline

NHarateh_WMF added a subscriber: JMinor.

@JMinor after eng sync, we agreed that we could solve this and T190308 if on fresh install we don't re-authenticate user credentials - would you be ok with not re-authenticating user credentials on fresh install, unless user attempts to log in via the UI?

cmadeo subscribed.

Hurrah!! Thanks @NHarateh_WMF! Imagine a cute gif of a dog dancing here, the internet is being weird at work so I can't get to Giphy 😝