-
Notifications
You must be signed in to change notification settings - Fork 86
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
Implement MagicAptosWallet #529
Implement MagicAptosWallet #529
Conversation
This pull request has been linked to Shortcut Story #78656: [aptos] implement MagicAptosWallet. |
3a4af64
to
9ddc5c3
Compare
87149ef
to
b4b9ec8
Compare
b4b9ec8
to
dc59435
Compare
6636206
to
1014638
Compare
readonly url = 'https://magic.link/'; | ||
readonly icon = ICON_BASE64; | ||
|
||
readonly providerName = 'magicWallet'; |
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.
can we change this to "magicWalletMA" (just in case we have to do something else for MC later)
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.
Nice catch.
fixed it a35b6b1
provider: Magic<[AptosExtension]>; | ||
private accountInfo: AccountInfo | null; | ||
|
||
constructor(magic: Magic<[AptosExtension]>) { |
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.
i think email should be passed in here, instead of us creating a different "connectWithMagicLink" function
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.
sounds good :)
fixed it. a35b6b1
π PR was released in |
π¦ Pull Request
[Provide a general summary of the pull request here.]
β Fixed Issues
https://app.shortcut.com/magic-labs/story/78656/aptos-implement-magicaptoswallet
https://app.shortcut.com/magic-labs/story/78763/aptos-add-support-for-react-adapter-react
π¨ Test instructions
[Describe any additional context required to test the PR/feature/bug fix.]
Please only add one label:
patch
: Bug Fix?minor
: New Feature?major
: Breaking Change?skip-release
: It's unnecessary to publish this change.