Open Bug 1907596 Opened 4 months ago Updated 28 days ago

Synced Tabs CFRs does not update the `lastCfrShownTimeInMillis` variable in `Settings.kt`

Categories

(Fenix :: Homepage, defect)

Product:
Fenix
Component:
Homepage
Platform:
All
Android
Type:
defect

Tracking

(Not tracked)

People

(Reporter: npoon, Unassigned, Mentored)

References

Details

(Keywords: good-first-bug)

All CFRs should update the lastCfrShownTimeInMillis variable in Settings.kt after being dismissed. As part of this ticket, add the following line to the onDismiss lambda for synced tabs CFRs, which is here:
context.settings().lastCfrShownTimeInMillis = System.currentTimeMillis()

See Also: → 1907571
See Also: → 1907599
Severity: -- → S3
Keywords: good-first-bug

hi Nick - thanks for raising this and providing details! I've added you as mentor, hope that is ok!

Mentor: npoon
Summary: Jump Back In and Synced Tabs CFRs do not update the `lastCfrShownTimeInMillis` variable in `Settings.kt` → Synced Tabs CFRs does not update the `lastCfrShownTimeInMillis` variable in `Settings.kt`
You need to log in before you can comment on or make changes to this bug.