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 Users.list_users() #193

Merged
merged 5 commits into from
Aug 21, 2023

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.

@linear
Copy link

linear bot commented Aug 17, 2023

@jonatascastro12 jonatascastro12 force-pushed the feature/usrld-689-add-getuser-to-python branch from eebff30 to 4e26f5a Compare August 17, 2023 14:26
@jonatascastro12 jonatascastro12 force-pushed the feature/usrld-694-add-listusers-to-python branch from 2c05622 to 63f0515 Compare August 17, 2023 14:26
Base automatically changed from feature/usrld-689-add-getuser-to-python to users August 17, 2023 14:30
@jonatascastro12 jonatascastro12 force-pushed the feature/usrld-694-add-listusers-to-python branch from 63f0515 to 9e12ed5 Compare August 17, 2023 17:35
@jonatascastro12 jonatascastro12 self-assigned this Aug 17, 2023
@jonatascastro12 jonatascastro12 marked this pull request as ready for review August 17, 2023 17:37
@jonatascastro12 jonatascastro12 requested a review from a team as a code owner August 17, 2023 17:37
@jonatascastro12 jonatascastro12 requested review from danerwilliams and removed request for a team August 17, 2023 17:37
workos/users.py Outdated Show resolved Hide resolved
@codecov-commenter
Copy link

Codecov Report

Merging #193 (0223e33) into users (acf5cea) will decrease coverage by 0.37%.
The diff coverage is 74.19%.

❗ 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     #193      +/-   ##
==========================================
- Coverage   86.80%   86.44%   -0.37%     
==========================================
  Files          30       31       +1     
  Lines        1031     1062      +31     
==========================================
+ Hits          895      918      +23     
- Misses        136      144       +8     
Files Changed Coverage Δ
workos/users.py 83.67% <70.37%> (-16.33%) ⬇️
workos/utils/user_types.py 100.00% <100.00%> (ø)

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

@jonatascastro12 jonatascastro12 changed the title Add Users.list_users() Add Users.list_users() Aug 21, 2023
@jonatascastro12 jonatascastro12 merged commit a9df256 into users Aug 21, 2023
9 checks passed
@jonatascastro12 jonatascastro12 deleted the feature/usrld-694-add-listusers-to-python branch August 21, 2023 21:33
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.

3 participants