You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
@daryl Collins, we created a PR in the NEPs for Injected Wallets based on the call we had and based on the comment left on the ticket for adding signInMulti: SQC-540: NEP-428 Wallet Selector Interface
PAGODA REVIEW
The content you are editing has changed. Please copy your edits and refresh the page.
Based on the latest discussions we've had in the last meeting related to this issue it was mentioned that the change looked fine for Daryl and he informed us that he had a talk with Andy who is a reviewer of the PR and explained that the PR in the NEP is good to be approved and merged, after that we've not had any more discussions about this change, it was left for a comment to be added on the PR for visibility with the context why this change should be merged.
Description
Make account optional for SignInParams and SignInMultiParams on Injected Wallets
Kujtim Prenku
May 25, 2023 at 1:47 AM
@daryl Collins, we created a PR in the NEPs for Injected Wallets based on the call we had and based on the comment left on the ticket for adding signInMulti: SQC-540: NEP-428 Wallet Selector Interface
PAGODA REVIEW
Related Issues
The text was updated successfully, but these errors were encountered: