Page MenuHomePhabricator

[native] ColdStartTracker
ClosedPublic

Authored by varun on Feb 19 2025, 7:47 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Mar 27, 2:55 PM
Unknown Object (File)
Wed, Mar 26, 2:02 AM
Unknown Object (File)
Tue, Mar 25, 1:44 PM
Unknown Object (File)
Tue, Mar 25, 12:47 AM
Unknown Object (File)
Mon, Mar 24, 3:31 AM
Unknown Object (File)
Sat, Mar 22, 7:07 AM
Unknown Object (File)
Wed, Mar 19, 9:59 PM
Unknown Object (File)
Sat, Mar 15, 7:21 AM
Subscribers

Details

Summary

I'll use this component in DisplayCommunityDirectoryPrompt to get the coldStartCount. We'll use this count to determine when to display the prompt

Test Plan

Added logs and confirmed that the count resets on logout and increments on cold start, but not when the app is just backgrounded.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

varun requested review of this revision.Feb 19 2025, 8:08 PM
ashoat added inline comments.
lib/reducers/alert-reducer.js
23–25 ↗(On Diff #47166)

Let's do else if to match the other reducers

30 ↗(On Diff #47166)

We need to reset the state on logout. Can you add a condition for logOutActionTypes.started before landing?

This revision is now accepted and ready to land.Feb 19 2025, 8:22 PM
lib/reducers/alert-reducer.js
30 ↗(On Diff #47166)

resetUserSpecificState should clear the alertStore on logout, so we don't need a condition for logOutActionTypes.started

This revision was automatically updated to reflect the committed changes.