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

Return User instead of MagicAuthChallenge response #204

Conversation

jonatascastro12
Copy link
Contributor

Description

Documentation

Does this require changes to the WorkOS Docs? E.g. the API Reference or code snippets need updates.

[ ] Yes

If yes, link a related docs PR and add a docs maintainer as a reviewer. Their approval is required.

@jonatascastro12 jonatascastro12 requested a review from a team as a code owner August 24, 2023 14:25
@jonatascastro12 jonatascastro12 requested review from maxdeviant and removed request for a team August 24, 2023 14:25
@linear
Copy link

linear bot commented Aug 24, 2023

@jonatascastro12 jonatascastro12 changed the title Magic Auth and Email Verification response returns User Response instead of MagicAuthChallenge Response Returns User Response instead of MagicAuthChallenge Response Aug 24, 2023
@jonatascastro12 jonatascastro12 changed the title Returns User Response instead of MagicAuthChallenge Response Return User Response instead of MagicAuthChallenge Response Aug 24, 2023
@jonatascastro12 jonatascastro12 self-assigned this Aug 24, 2023
@jonatascastro12 jonatascastro12 changed the title Return User Response instead of MagicAuthChallenge Response Return User Response instead of MagicAuthChallenge eesponse Aug 24, 2023
@jonatascastro12 jonatascastro12 changed the title Return User Response instead of MagicAuthChallenge eesponse Return User instead of MagicAuthChallenge response Aug 24, 2023
@codecov-commenter
Copy link

codecov-commenter commented Aug 24, 2023

Codecov Report

Merging #204 (51802ea) into users (9d32ada) will decrease coverage by 0.04%.
The diff coverage is 100.00%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@            Coverage Diff             @@
##            users     #204      +/-   ##
==========================================
- Coverage   88.37%   88.34%   -0.04%     
==========================================
  Files          34       33       -1     
  Lines        1222     1218       -4     
==========================================
- Hits         1080     1076       -4     
  Misses        142      142              
Files Changed Coverage Δ
workos/users.py 94.87% <100.00%> (-0.05%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@jonatascastro12 jonatascastro12 force-pushed the feature/usrld-734-add-sendmagicauthcode-to-python branch from bb1932e to 73bb1f5 Compare August 24, 2023 14:28
@jonatascastro12 jonatascastro12 force-pushed the feature/usrld-921-magic-auth-and-email-verification-response-returns-user branch from a9a9d39 to 68e224f Compare August 24, 2023 14:28
@jonatascastro12 jonatascastro12 force-pushed the feature/usrld-734-add-sendmagicauthcode-to-python branch from 73bb1f5 to 4a389d9 Compare August 24, 2023 14:45
@jonatascastro12 jonatascastro12 force-pushed the feature/usrld-921-magic-auth-and-email-verification-response-returns-user branch from 68e224f to d85277e Compare August 24, 2023 14:45
Base automatically changed from feature/usrld-734-add-sendmagicauthcode-to-python to users August 24, 2023 15:56
@jonatascastro12 jonatascastro12 force-pushed the feature/usrld-921-magic-auth-and-email-verification-response-returns-user branch from d85277e to 5653606 Compare August 24, 2023 15:56
@jonatascastro12 jonatascastro12 requested a review from a team August 25, 2023 14:25
@jonatascastro12 jonatascastro12 merged commit 800fbb9 into users Aug 25, 2023
6 checks passed
@jonatascastro12 jonatascastro12 deleted the feature/usrld-921-magic-auth-and-email-verification-response-returns-user branch August 25, 2023 19:43
blairlunceford added a commit that referenced this pull request Nov 14, 2023
* Add `Users` client and `Users.create_user()` method (#191)

* Add Users client and Users.create_user() method

* Fix comment and format.

* Get user. (#192)

* Add `Users.list_users()` (#193)

* Add `Users.list_users()`

* Do not use f-string

* Fix comment

* Fix format

* Remove User type, and other fields

* Add add_user_to_organization and remove_user_to_organization methods (#195)

* Add `Users.authenticate_with_magic_auth()` (#196)

* Add authenticate_with_magic_auth

* Update comment

* Add `Users.authenticate_with_password()` (#197)

* Add `Users.authenticate_with_password()`

* Fix method

* Update comment

* Update test

* Add `Users.authenticate_with_code()` (#198)

* Add `Users.authenticate_with_token()` method

* Rename method

* Fix test

* Add `Users.create_password_challenge()` method (#199)

* Add `Users.complete_password_reset()` method (#200)

* Add `Users.send_verification_email()` method (#201)

* Add `Users.send_verification_email()` method

* Fix Magic Auth challenge

* Fix test

* Add `Users.verify_email()` method (#202)

* Add `Users.send_magic_auth_code()` method (#203)

* Add `Users.send_magic_auth_code()` method

* Fix test

* Fix test 2

* Return `User` instead of `MagicAuthChallenge` response (#204)

* Magic Auth and Email Verification response returns User Response instead of MagicAuthChallenge Response

* Replace other magic_auth_challenge_id references

* Add users.delete_user() method (#205)

* Fix authenticate methods (#206)

* Add `users.update_user_password()` method (#207)

* Add `users.update_user()` method (#208)

* Fix Verify Email Code method (#209)

---------

Co-authored-by: Jônatas Santos <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants