Skip to content

Fix/multiple wallets - #443

Merged
Agilulfo1820 merged 3 commits into
mainfrom
fix/multiple-wallets
Sep 10, 2025
Merged

Agilulfo1820 merged 3 commits into
mainfrom
fix/multiple-wallets

Conversation

@Agilulfo1820

Copy link
Copy Markdown
Member

Description

There was an issue with the oauth login, where the onComplete was triggered multiple times creating many wallets.

Added also 2 example buttons for oauth login in the homepage.

Closes #(issue)

Updated packages (if any):

  • next-template
  • homepage
  • vechain-kit
  • contracts

@Agilulfo1820
Agilulfo1820 requested a review from a team September 9, 2025 14:11
@socket-security

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updated@​privy-io/​react-auth@​2.21.0 ⏵ 2.24.0100100100100100

View full report

@mikeredmond mikeredmond left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM 🚀

@Agilulfo1820
Agilulfo1820 merged commit 8e23142 into main Sep 10, 2025
9 of 11 checks passed
@Agilulfo1820
Agilulfo1820 deleted the fix/multiple-wallets branch September 10, 2025 11:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants