-
Notifications
You must be signed in to change notification settings - Fork 35
feat: migrate Safepal to use Hub #986
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
base: next
Are you sure you want to change the base?
Conversation
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.
Tnx for your contribution, I just added some minor comments. Also when I switch my account, it refreshes the page and the wallet gets disconnected. Please investigate this case and make sure we have implemented it the correct way and how safepal handles switch account.
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
f4397f2
to
cb4d193
Compare
cb4d193
to
4981b4e
Compare
79e54f6
to
da0e1ea
Compare
da0e1ea
to
fedbc01
Compare
a85571f
to
a663316
Compare
a663316
to
b6cffe5
Compare
Summary
Safepal has been migrated to Hub
Checklist: