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.send_verification_email() method #201

Merged

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 22, 2023 18:30
@jonatascastro12 jonatascastro12 requested review from danerwilliams and removed request for a team August 22, 2023 18:30
@linear
Copy link

linear bot commented Aug 22, 2023

@codecov-commenter
Copy link

Codecov Report

Merging #201 (0a88555) into feature/usrld-756-add-completepasswordreset-to-python (37959fe) will increase 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                                    @@
##           feature/usrld-756-add-completepasswordreset-to-python     #201      +/-   ##
=========================================================================================
+ Coverage                                                  88.17%   88.22%   +0.04%     
=========================================================================================
  Files                                                         33       33              
  Lines                                                       1201     1206       +5     
=========================================================================================
+ Hits                                                        1059     1064       +5     
  Misses                                                       142      142              
Files Changed Coverage Δ
workos/users.py 94.28% <100.00%> (+0.28%) ⬆️

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

@jonatascastro12 jonatascastro12 self-assigned this Aug 22, 2023
@jonatascastro12 jonatascastro12 requested a review from a team August 22, 2023 18:46
@jonatascastro12 jonatascastro12 force-pushed the feature/usrld-756-add-completepasswordreset-to-python branch from 37959fe to aa9c641 Compare August 23, 2023 20:00
@jonatascastro12 jonatascastro12 force-pushed the feature/usrld-739-add-sendverificationemail-to-python branch from 245de37 to defb660 Compare August 23, 2023 20:00
Base automatically changed from feature/usrld-756-add-completepasswordreset-to-python to users August 23, 2023 20:02
@jonatascastro12 jonatascastro12 force-pushed the feature/usrld-739-add-sendverificationemail-to-python branch from defb660 to 8dd3bf0 Compare August 23, 2023 20:03
@jonatascastro12 jonatascastro12 merged commit 7b70e0c into users Aug 24, 2023
6 checks passed
@jonatascastro12 jonatascastro12 deleted the feature/usrld-739-add-sendverificationemail-to-python branch August 24, 2023 14:28
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