-
Notifications
You must be signed in to change notification settings - Fork 2
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
chore: Remove wallet connect v1 #100
Conversation
#if UNITY_ANDROID | ||
await this.Connect(); | ||
#endif | ||
await this.Connect(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
#if UNITY_ANDROID 일때를 묵은 이유가 있을까용? 이렇게 빠지면 Connect가 두번 호출될것 같은데 어떤 이슈가 있었나용?
#if UNITY_IOS | ||
// await this._walletConnectClient.Connect(); | ||
#endif |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
지워도 될것같아요!
제가 만든 PR이라 approve가 안되네요. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
No description provided.