8000 [FEAT] `getAccessToken` by itailevi98 · Pull Request #67 · PropelAuth/react · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

[FEAT] getAccessToken #67

New issue

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

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Oct 17, 2024
Merged

[FEAT] getAccessToken #67

merged 1 commit into from
Oct 17, 2024

Conversation

itailevi98
Copy link
Contributor
@itailevi98 itailevi98 commented Oct 8, 2024 •
edited
Loading

Background

This PR adds a new function to the auth context, getAccessToken(). This allows for a specific call to get the latest valid access token. This can help with avoiding bugs where we closed in a local usage of the out of date access token, and now we can explicitly get the newest one every time.

Smoke Testing

  • In the dashboard, where the env. config bug would happen, this function was used, and solved the issue of saving the access token in a use ref.
  • In an example app, this function was used and called in a useEffect with a useState saving the access token.

@itailevi98 itailevi98 self-assigned this Oct 8, 2024
@andrew-propelauth andrew-propelauth merged commit 57fc542 into main Oct 17, 2024
1 check passed
@andrew-propelauth andrew-propelauth deleted the feat/access-token-updates branch October 17, 2024 22:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@andrew-propelauth andrew-propelauth andrew-propelauth approved these changes

@pfvatterott pfvatterott Awaiting requested review from pfvatterott

Assignees

@itailevi98 itailevi98

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@itailevi98 @andrew-propelauth
0