Skip to content

expose refresh token on DescopeClient #39

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.

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
Aug 16, 2022
Merged

expose refresh token on DescopeClient #39

merged 1 commit into from
Aug 16, 2022

Conversation

asafshen
Copy link
Member

  • expose refresh token on DescopeClient
  • add default value None to project_id

@asafshen asafshen requested a review from aviadl August 16, 2022 12:01
@asafshen asafshen enabled auto-merge (squash) August 16, 2022 12:01
@asafshen asafshen requested a review from guyp-descope August 16, 2022 12:01
@github-actions
Copy link

github-actions bot commented Aug 16, 2022 •
edited
Loading

Coverage

Coverage Report
FileStmtsMissCoverMissing
descope
   auth.py210498%55, 384, 414–415
   descope_client.py46198%111
TOTAL719599% 

Tests Skipped Failures Errors Time
63 0 💤 0 ❌ 0 🔥 2.659s ⏱️

aviadl
aviadl approved these changes Aug 16, 2022
View reviewed changes
@asafshen asafshen merged commit 91e673c into main Aug 16, 2022
@asafshen asafshen deleted the expose-refresh-token branch August 16, 2022 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aviadl aviadl aviadl approved these changes

@guyp-descope guyp-descope Awaiting requested review from guyp-descope

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@asafshen @aviadl