Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

remove user lastActiveTimestamp property #365

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

vincanger
Copy link
Collaborator

@vincanger vincanger commented Feb 14, 2025

Description

removes user.lastActiveTimestamp property entirely since we're not really using it in a meaningful way.

Renames updateCurrentUser to a more generic function name in the authorization docs.

Contributor Checklist

Make sure to do the following steps if they are applicable to your PR:

@vincanger vincanger changed the title return user id and timestamp only from updateUserTimestamp remove user lastActiveTimestamp property Feb 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant