feature/version-gated-updates #1

Merged

fredo merged 5 commits from feature/version-gated-updates into main

2026-09-09 21:59:41 +02:00

fredo commented

2026-09-09 20:15:18 +02:00

Owner

Changed the update model

fredo added 4 commits

2026-09-09 20:15:18 +02:00

Delete dead-code duplicates (NetworkRepository, Version, VersionInfo) 5fe22c67b4

NetworkRepository contained a working version-check implementation but
was never called. Version.kt and VersionInfo.kt in data/model are
duplicate definitions of VersionInfo already declared in
MedicationApiService.kt. All three are referenced nowhere outside
their own files.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Harden MedicationDao: atomic replaceAll via @Transaction 4c5eb4f10d

Convert interface to abstract class so Room can generate a concrete
@Transaction method. replaceAll() wraps deleteAll() + insertAll() in a
single transaction, ensuring a mid-insert failure leaves the DB
unchanged rather than empty.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Wire version-gated refresh into MedicationRepository d24d64a8ac

Refresh sequence now:
  1. GET /version.json, validate explicitly (no trusting Gson non-nulls)
  2. Compare server version to stored version for current language
  3. If equal: update last_checked_at only, skip the ~1.1 MB download
  4. If newer: download, validate (non-empty and >= 80% of current
     count), replace transactionally via MedicationDao.replaceAll(),
     then persist dataVersion + dataPublishedAt + lastCheckedAt

Single OkHttp client with 5 MB cache replaces the two separate
Retrofit instances that existed before. MedicationApiService.create()
now takes a cacheDir to enable ETag/If-None-Match on the
nginx-served static files.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Expose DataStatus in ViewModel and show data age in list screen dfdd246e80

DataStatus carries dataVersion, dataPublishedAt, and lastCheckedAt.
The ViewModel loads the initial status from SharedPreferences on init
and refreshes it after every refresh/language-switch operation.

MedicationListScreen shows a status row at the top of the list:
- amber warning when never checked or last check > 30 days ago
- quiet text line otherwise: "Données du <date> · vérifié <when>"

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

fredo added 1 commit

2026-09-09 20:16:35 +02:00

Keep list visible during refresh and language switch 4f3a2518af

Split _isLoading into two states:
- _isInitializing: true only during first-time DB population, shows
  the full-screen spinner (list has nothing to show yet)
- _isRefreshing: true during network refresh, language switch, and
  export, shows only a thin LinearProgressIndicator below the search
  bar — the medication list stays visible throughout

The menu and search field are disabled during both states to prevent
double-taps. Language switch benefits from the @Transaction replaceAll
added earlier: Room only emits a new list once the transaction commits,
so the old language's data stays visible until the new data is ready.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>