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

Add MFA support #116

Merged
merged 11 commits into from
Mar 24, 2022
Merged

Add MFA support #116

merged 11 commits into from
Mar 24, 2022

Conversation

awolfden
Copy link
Contributor

@awolfden awolfden commented Mar 8, 2022

Add MFA to Python SDK

@awolfden awolfden requested a review from a team as a code owner March 8, 2022 23:58
@linear
Copy link

linear bot commented Mar 8, 2022

@pliao28 pliao28 requested a review from robframpton March 9, 2022 17:06
@robframpton
Copy link
Contributor

Method signatures look good to me, would love someone with more Python prowess to give a look as well 👀

workos/mfa.py Outdated Show resolved Hide resolved
tests/test_mfa.py Outdated Show resolved Hide resolved
@pliao28
Copy link
Contributor

pliao28 commented Mar 23, 2022

@awolfden Can you change the version for a pre-release version (believe it would be 1.11.0.a1 https://pythonpackaging.info/07-Package-Release.html)
@robframpton could you check if the semaphore publish run needs any adjustment

@maxdeviant maxdeviant changed the title Add MFA to Python SDK Add MFA support Mar 23, 2022
@codecov-commenter
Copy link

Codecov Report

Merging #116 (6dc7e76) into master (81e86c2) will decrease coverage by 0.19%.
The diff coverage is 92.68%.

@@            Coverage Diff             @@
##           master     #116      +/-   ##
==========================================
- Coverage   95.08%   94.88%   -0.20%     
==========================================
  Files          18       19       +1     
  Lines         488      528      +40     
==========================================
+ Hits          464      501      +37     
- Misses         24       27       +3     
Impacted Files Coverage Δ
workos/client.py 85.71% <40.00%> (-6.18%) ⬇️
workos/__about__.py 100.00% <100.00%> (ø)
workos/mfa.py 100.00% <100.00%> (ø)
workos/utils/validation.py 100.00% <100.00%> (ø)

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@pliao28
Copy link
Contributor

pliao28 commented Mar 23, 2022

@awolfden @robframpton , published can be retrieved by pip install 'workos==1.11.0.a1'

workos/__about__.py Outdated Show resolved Hide resolved
Copy link
Contributor

@robframpton robframpton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@awolfden once the __version__ change I suggested is committed this will be good to go.

Co-authored-by: Robert Frampton <rob@workos.com>
@awolfden awolfden merged commit 743a038 into master Mar 24, 2022
@awolfden awolfden deleted the feature/sup-258-add-mfa-to-python-sdk branch March 24, 2022 19:53
@robframpton robframpton mentioned this pull request Mar 28, 2022
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.

None yet

5 participants