Skip to content

fix(notifications): fall back to live device lookup for contact name and photo#845

Merged
bmc08gt merged 1 commit into
code/cashfrom
fix/notification-contact-freshness
Jun 4, 2026
Merged

fix(notifications): fall back to live device lookup for contact name and photo#845
bmc08gt merged 1 commit into
code/cashfrom
fix/notification-contact-freshness

Conversation

@bmc08gt

@bmc08gt bmc08gt commented Jun 4, 2026

Copy link
Copy Markdown
Collaborator

When a cached contact photo URI or display name is missing from the DB, resolve directly from the device contacts provider via PhoneLookup and cache the result for subsequent lookups.

  • Add lookupPhotoUri to DeviceContactLookup interface and implementation
  • ContactResolver.resolvePhotoUri falls back to device on DB cache miss
  • ContactResolver.resolveName prefers fresh device name over stale cache
  • Only write back to DB when the value actually changed
  • Add DAO/DataSource methods for targeted name and photo URI updates

…and photo

When a cached contact photo URI or display name is missing from the DB,
resolve directly from the device contacts provider via PhoneLookup and
cache the result for subsequent lookups.

- Add lookupPhotoUri to DeviceContactLookup interface and implementation
- ContactResolver.resolvePhotoUri falls back to device on DB cache miss
- ContactResolver.resolveName prefers fresh device name over stale cache
- Only write back to DB when the value actually changed
- Add DAO/DataSource methods for targeted name and photo URI updates

Signed-off-by: Brandon McAnsh <git@bmcreations.dev>
@bmc08gt bmc08gt self-assigned this Jun 4, 2026
@github-actions github-actions Bot added the type: fix Bug fix label Jun 4, 2026
@bmc08gt bmc08gt merged commit 69eb2a3 into code/cash Jun 4, 2026
3 checks passed
@bmc08gt bmc08gt deleted the fix/notification-contact-freshness branch June 4, 2026 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: fix Bug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant