[Bug] Sync error requires force quitting app to resolve
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	cmadeo
	Mar 7 2018, 11:19 PM

Description

Steps to reproduce

Prerequisite: User is logged in to two devices with sync active on their account.

On device#1 , login to Wikipedia app and enable syncing with reading list “Alpha” saved.
On device #1 , log out and choose to delete saved articles from the device.
On device #1, delete the Wikipedia app.
On device #2, Create a new reading list “Beta”.
On device #1, re-install the Wikipedia app and login.

Expected behavior

Reading lists “Alpha” and “Beta” should appear on device #1 after re-downloading and logging-in to Wikipedia.

Actual behavior

On device 2 there is an error under the "Beta" reading list, which is also not synced with device 1.
This error is not fixed by pulling down to refresh on either Reading lists view. It can only be resolved by force quitting device 2 (not device 1).

Proposed solution

Do a full sync when the user pulls down to refresh on either list in the Saved tab

Related Objects
Search...

Status	Assigned	Task
Resolved	None	T176357 [Epic] As a user of 'Save for later' I would like a way to create multiple reading lists and sync them across my devices
Resolved	• JMinor	T177564 Exploratory testing when users with multiple devices use synced reading lists
Resolved	Mhurd	T189171 [Bug] Sync error requires force quitting app to resolve

Event Timeline

cmadeo triaged this task as Medium priority.Mar 7 2018, 11:19 PM

cmadeo created this task.

cmadeo edited parent tasks, added: T177564: Exploratory testing when users with multiple devices use synced reading lists; removed: T189170: [Bug] As a user of synced reading lists, I'd like my reading lists to load in a timely manner.

• JMinor updated the task description. (Show Details)Mar 8 2018, 7:45 PM

• JMinor moved this task from Tasks from Product Backlog to Needs Engineering Sync on the iOS-app-v5.8.0-Manatee-On-A-Skateboard board.

cmadeo updated the task description. (Show Details)Mar 8 2018, 7:45 PM

LGoto added a project: Wikipedia-iOS-App-Backlog.Mar 27 2018, 5:47 PM

• JMinor moved this task from Needs Engineering Sync to Ready for Development on the iOS-app-v5.8.0-Manatee-On-A-Skateboard board.Mar 27 2018, 6:45 PM

Mhurd claimed this task.Apr 5 2018, 3:18 AM

Mhurd moved this task from Ready for Development to Doing on the iOS-app-v5.8.0-Manatee-On-A-Skateboard board.

https://github.com/wikimedia/wikipedia-ios/pull/2181

Edit: ^ fixes a crash, but not specifically related to this ticket, which is about an error vs a crash. This ticket needs the pull to refresh added in more places as specified in the description.

• NHarateh_WMF mentioned this in rAPIOSceb5d635e24a: Merge pull request #2181 from wikimedia/fix/startup-crash/T189171.Apr 6 2018, 12:48 PM

https://github.com/wikimedia/wikipedia-ios/pull/2183

Mhurd moved this task from Doing to Needs Code Review on the iOS-app-v5.8.0-Manatee-On-A-Skateboard board.Apr 6 2018, 10:07 PM

• NHarateh_WMF mentioned this in rAPIOS274009f2a13e: Merge pull request #2183 from wikimedia/fix/pull-to-refresh/T189171.Apr 6 2018, 11:39 PM

@cmadeo When goes out in a build can you retry your repro steps and let me know if it's still funky? I wasn't able to repro the error but we did fix the pull to refresh to work as outlined in the description.

• NHarateh_WMF moved this task from Waiting for Build to Needs Design Review on the iOS-app-v5.8.0-Manatee-On-A-Skateboard board.Apr 8 2018, 6:03 PM

@Mhurd, will do!

I can't repo the error either @Mhurd. Going to move this over to needs QA

Device #1: iOS 10.3.3 (iPhone 6S)
Device #2: iOS 11 (iPhone 7)
App Version 5.8.0 (1377)

This is fixed now as shown in screencaps from both devices.

Working for me on 5.8.0 (1388)

LGoto moved this task from Needs Triage to Bug Backlog on the Wikipedia-iOS-App-Backlog board.Apr 17 2018, 5:44 PM

• JMinor closed this task as Resolved.Apr 17 2018, 11:11 PM

	F16920673: Device 1.PNG
	Apr 11 2018, 7:41 PM

	F16920672: Device 2.jpg
	Apr 11 2018, 7:41 PM

[Bug] Sync error requires force quitting app to resolve Closed, ResolvedPublicActions