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

[ID-1124][Passport]Add signIn function, changed connectEvm function to call slient login first #1057

Merged
merged 6 commits into from
Oct 27, 2023

Conversation

carmen0208
Copy link
Contributor

@carmen0208 carmen0208 commented Oct 24, 2023

Summary

Screen.Recording.2023-10-25.at.09.43.18.mov

Added

  • signIn function in Passport client
  • add the example to the sample app

Changed

Changed the connectImx function to call silent login before login

Depreicated

  • connectImxSilent function

Things worth calling out

Before submitting the PR, please consider the following:

  • Prefix your PR title with feat: , fix: , chore: , docs:, or refactor:.

@carmen0208 carmen0208 requested a review from a team as a code owner October 24, 2023 23:45
@carmen0208 carmen0208 force-pushed the feature/ID-1124-add-signin-method-to-passport branch from e03fe0e to 59ca524 Compare October 25, 2023 03:36
@carmen0208 carmen0208 force-pushed the feature/ID-1124-add-signin-method-to-passport branch from 59ca524 to b13466d Compare October 25, 2023 03:51
@carmen0208 carmen0208 changed the title [ID-1124][Passport]Add signIn function [ID-1124][Passport]Add signIn function, changed connectEvm function to call slient login first Oct 26, 2023
@carmen0208 carmen0208 force-pushed the feature/ID-1124-add-signin-method-to-passport branch from a9b5183 to 3e164f8 Compare October 26, 2023 04:38
@carmen0208 carmen0208 enabled auto-merge (squash) October 27, 2023 04:40
@carmen0208 carmen0208 merged commit 442c570 into main Oct 27, 2023
6 checks passed
@carmen0208 carmen0208 deleted the feature/ID-1124-add-signin-method-to-passport branch October 27, 2023 04:47
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.

None yet

3 participants