Page MenuHomePhabricator

[4.0.7.5] Lead Image & Image Gallery features broken for migrated articles
Closed, ResolvedPublic

Description

Wikimedia Mobile TestFlight Beta 4.0.7.5

Steps to reproduce

  1. Download app from the app store
    1. Or, check out the old version of the project at git SHA: 056211487f3f4e41726c0e301f34b3c5ba4cf13e
  2. Save & visit pages
  3. Note the saved & visited pages
  4. Download the Wikimedia Mobile TF Beta 4.0.7.5
  5. !! Double check saved & recent pages are accurate
  6. Click on one of the saved/recent pages to view it

Expected Results:
Lead image is present for the article (if it has one). Image gallery functions as normal (opens the selected image and shows the rest of the article's images).

Actual results:
Lead image does not appear, and image gallery only shows the first image, w/o any other images. Requires a refresh via pull-to-refresh to restore functionality.

Event Timeline

BGerstle-WMF raised the priority of this task from to Needs Triage.
BGerstle-WMF updated the task description. (Show Details)

It seems that opening saved/recent pages the second etc time displays the leading image (iPad mini 8.2 with Beta 4.0.7.5 ).

@Etonkovidova that sounds inconsistent with what I noticed, which is that no matter how many times I opened the article from saved/recent that the lead image & gallery were broken. Only after refreshing the article did I see the lead image & gallery function properly. Admittedly, the refresh ''could'' happen as a result of opening the article, but I don't remember exactly how that's supposed to work.

Deskana lowered the priority of this task from Medium to Low.

It's not the biggest deal that saved pages are not showing lead images if they're migrated, because they didn't have lead images before.

@BGerstle-WMF: Are you still working on this?

@Deskana haven't started working on it yet. Was planning on picking it up soon, though.

BGerstle-WMF raised the priority of this task from Low to High.
BGerstle-WMF set Security to None.
BGerstle-WMF moved this task from To Do to Code Review on the Mobile-App-Sprint-53-iOS board.

Change 198432 had a related patch set uploaded (by Bgerstle):
set article imageURL and save image list

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

Change 198432 merged by Dr0ptp4kt:
set article imageURL and save image list

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