This class isn't strictly a data client yet but we think that renaming it will help change the way we think about it and eventually get there.
Description
Description
Details
Details
Subject | Repo | Branch | Lines +/- | |
---|---|---|---|---|
Hygiene: Rename JsonPageLoadStrategy to PageDataClient | apps/android/wikipedia | master | +21 -21 |
Status | Subtype | Assigned | Task | ||
---|---|---|---|---|---|
Resolved | • Mholloway | T136322 [Dev] Rename JsonPageLoadStrategy to PageDataClient | |||
Resolved | None | T136321 [Dev] Transition JsonPageLoadStrategy caching to Retrofit and OkHttp | |||
Resolved | • Niedzielski | T148675 Replace PageCache with Retrofit OkHttp caching | |||
Invalid | None | T156829 Remove must-revalidate cache-control header from mobile-sections-lead and mobile-sections-remaining responses |
Event Timeline
Comment Actions
@Dbrant, this is unpointed but I like to remove it from the sprint in light of the extra cards coming in for screenshot testing unless someone wants to work on it.
Comment Actions
Change 313624 had a related patch set uploaded (by Mholloway):
Rename JsonPageLoadStrategy to PageDataClient
Comment Actions
Change 313624 merged by jenkins-bot:
Hygiene: Rename JsonPageLoadStrategy to PageDataClient