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

Added directSignIn parameter to client signIn method #62

Closed
wants to merge 2 commits into from

Conversation

jonsamwell
Copy link

Summary

Added directSignIn parameter to client signIn method. This parameter allows you to skip the first screen and invoke the sign-in process directly. ** Introduces a breaking change by adding optional named method parameters **

Testing

Unit and manual testing

Checklist

  • .changelog
  • unit tests
  • integration tests
  • necessary TSDoc comments

Copy link
Contributor

@simeng-li simeng-li left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution. LGTM

@simeng-li
Copy link
Contributor

simeng-li commented Jul 16, 2024

@jonsamwell cloud you add signatures and re-submit the commit? We require all the commits to be signed in the Logto code base. Also, there was a new version bumped PR just merged to the master, you might need to update your version to 2.0.3 and resolve the conflicts. Thanks.

@simeng-li
Copy link
Contributor

This PR's updates are included in the #68. Will close this one, once #68 is merged.

@simeng-li simeng-li closed this Sep 6, 2024
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