Track a user's last activity #5781
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
stnguyen90
force-pushed
the
feat-last-user-activity
branch
4 times, most recently
from
2d3a4f3 to
98ce832
Compare
stnguyen90
force-pushed
the
feat-last-user-activity
branch
from
98ce832 to
31e8e81
Compare
stnguyen90
force-pushed
the
feat-last-user-activity
branch
from
31e8e81 to
c8f09f4
Compare
stnguyen90
commented
Jul 13, 2023
eldadfux
approved these changes
Jul 13, 2023
stnguyen90
force-pushed
the
feat-last-user-activity
branch
from
c8f09f4 to
0ce3969
Compare
stnguyen90
force-pushed
the
feat-last-user-activity
branch
2 times, most recently
from
733b6e5 to
8e5a554
Compare
eldadfux
approved these changes
Jul 19, 2023
stnguyen90
force-pushed
the
feat-last-user-activity
branch
5 times, most recently
from
515b959 to
9aa168e
Compare
A user will have an accessedAt timestamp that will update at most once per day if they make some API call. This timestamp can then be used find active users and calculate daily, weekly, and monthly active users. To ensure consistent updates to the user the $user from the resource is always updated to the user making the request, including requests like Create Account, Update Team Membership Status, and Create Phone Session (confirmation). This ensures the shutdown can update the accessedAt timestamp if there was a $user set.
stnguyen90
force-pushed
the
feat-last-user-activity
branch
from
9aa168e to
2befa60
Compare
2 tasks
2 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What does this PR do?
A user will have an accessedAt timestamp that will update at most once per day if they make some API call. This timestamp can then be used find active users and calculate daily, weekly, and monthly active users.
Test Plan
Automated tests
Related PRs and Issues
Blocks:
Depends on:
Checklist