Skip to content
This repository was archived by the owner on Sep 8, 2025. It is now read-only.

Conversation

@silentworks
Copy link
Contributor

What kind of change does this PR introduce?

Bug fix

What is the current behavior?

Currently the should_create_user property is incorrectly named inside the sign_in_with_otp method

What is the new behavior?

The should_create_user property is correctly named inside the sign_in_with_otp method

Additional context

Add any other context or screenshots.

@sourcery-ai
Copy link
Contributor

sourcery-ai bot commented Jun 23, 2024

🧙 Sourcery has finished reviewing your pull request!


Tips
  • Trigger a new Sourcery review by commenting @sourcery-ai review on the pull request.
  • You can change your review settings at any time by accessing your dashboard:
    • Enable or disable the Sourcery-generated pull request summary or reviewer's guide;
    • Change the review language;
  • You can always contact us if you have any questions or feedback.

Copy link
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

We've reviewed this pull request using the Sourcery rules engine. If you would also like our AI-powered code review then let us know.

@silentworks silentworks merged commit 388fcba into main Jun 23, 2024
@silentworks silentworks deleted the silentworks/fix-sign-in-with-otp branch June 23, 2024 20:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants