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

[Unity] fix unity wallet connect #950

Merged
merged 3 commits into from
May 1, 2023
Merged

Conversation

joaquim-verges
Copy link
Member

No description provided.

@joaquim-verges joaquim-verges requested a review from a team as a code owner May 1, 2023 17:56
@changeset-bot
Copy link

changeset-bot bot commented May 1, 2023

🦋 Changeset detected

Latest commit: 88036fa

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@thirdweb-dev/unity-js-bridge Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@codecov
Copy link

codecov bot commented May 1, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (5a67d5d) 69.49% compared to head (88036fa) 69.49%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #950   +/-   ##
=======================================
  Coverage   69.49%   69.49%           
=======================================
  Files         233      233           
  Lines       11213    11213           
  Branches     1381     1381           
=======================================
  Hits         7793     7793           
  Misses       2678     2678           
  Partials      742      742           

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@joaquim-verges joaquim-verges added this pull request to the merge queue May 1, 2023
case "walletConnect":
walletInstance = new WalletConnect({
case "walletConnectV1":
walletInstance = new WalletConnectV1({
Copy link
Contributor

Choose a reason for hiding this comment

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

The reason for this is to support the most popular wallets correct? This is not fixing any bugs

Copy link
Member Author

Choose a reason for hiding this comment

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

yep

Merged via the queue into main with commit 7e7a507 May 1, 2023
@joaquim-verges joaquim-verges deleted the joaquim/unity_fixes branch May 1, 2023 18:50
@github-actions github-actions bot mentioned this pull request May 1, 2023
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