Remove our caching code, including DiskLruCache (remove from About libraries too). Test that Retrofit picks up caching transparently, especially when you're offline, and that the article header logic didn't break.
Description
Description
| 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
Is this either a parent or a duplicate of T148675: Replace PageCache with Retrofit OkHttp caching?
Comment Actions
Good eye. I've marked it as a subtask because Retrofitting will be a lot easier without the custom caching