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.update_user() method #208

Merged
merged 1 commit into from
Aug 25, 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.

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

linear bot commented Aug 25, 2023

USRLD-958 Add updateUser

Use Node SDK as reference

@codecov-commenter
Copy link

codecov-commenter commented Aug 25, 2023

Codecov Report

Merging #208 (25e5040) into users (25e5040) will not change coverage.
The diff coverage is n/a.

❗ Current head 25e5040 differs from pull request most recent head e9cc9f5. Consider uploading reports for the commit e9cc9f5 to get more accurate results

❗ 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     #208   +/-   ##
=======================================
  Coverage   87.69%   87.69%           
=======================================
  Files          32       32           
  Lines        1154     1154           
=======================================
  Hits         1012     1012           
  Misses        142      142           

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

@jonatascastro12 jonatascastro12 force-pushed the feature/usrld-952-add-updateuserpassword branch from 5e50222 to 69d588e Compare August 25, 2023 17:06
@jonatascastro12 jonatascastro12 force-pushed the feature/usrld-952-add-updateuserpassword branch from 69d588e to 288b7e0 Compare August 25, 2023 19:43
@jonatascastro12 jonatascastro12 force-pushed the feature/usrld-952-add-updateuserpassword branch from 288b7e0 to 9383258 Compare August 25, 2023 21:12
@jonatascastro12 jonatascastro12 force-pushed the feature/usrld-952-add-updateuserpassword branch from 9383258 to ee72a6c Compare August 25, 2023 21:20
Base automatically changed from feature/usrld-952-add-updateuserpassword to users August 25, 2023 21:21
@jonatascastro12 jonatascastro12 merged commit cbc9f99 into users Aug 25, 2023
6 checks passed
@jonatascastro12 jonatascastro12 deleted the feature/usrld-958-add-updateuser branch August 25, 2023 21:24
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