feat: Progressbar and group name and image at series card#492
Conversation
Confidence Score: 5/5Safe to merge; changes are well-scoped to display logic and data parsing with no impact on write paths or auth flows. The data-layer changes (entity fields, model parsing, cache versioning, optional auth forwarding) are additive and backward-compatible. The widget refactoring replaces inline duplicates with shared components without altering any navigation, state-mutation, or enrollment logic. The two findings are style-level observations with no runtime impact. No files require special attention. Reviews (2): Last reviewed commit: "Update lib/features/home/presentation/wi..." | Re-trigger Greptile |
| String featuredSeriesKey(String language, int limit) => | ||
| 'featured_series:$language:$limit'; | ||
| 'featured_series:v3:$language:$limit'; | ||
| String seriesListKey(String language) => 'series_list:$language'; |
There was a problem hiding this comment.
seriesListKey was not given a version suffix, unlike featuredSeriesKey (→ v3) and seriesByIdKey (→ v2). The same staleness issue applies: a device with a series_list:$language entry from before auth was wired to the endpoint will serve the cached models (missing progressJson/partnerJson) on the initial render, so the progress bar and partner row won't appear until the background refresh completes. Adding a version token (e.g. 'series_list:v2:$language') would invalidate those entries on first launch.
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
Uh oh!
There was an error while loading. Please reload this page.