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.

Sign up for GitHub

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

Jump to bottom

Check accountId usage across the app #42

Open
dchernokur opened this issue May 22, 2024 · 0 comments
Open

Check accountId usage across the app #42

dchernokur opened this issue May 22, 2024 · 0 comments

Comments

@dchernokur
Copy link
Collaborator

dchernokur commented May 22, 2024 •
edited
Loading

We've introduced accountEvm in our wallet interface recently, but there are potentially places left with using the accountId still.

For MM/WC they both contain the evm address, but for hashpack/blade wallets accountId contains hedera id format address and will fail some calls.

This could be a good point to bring back and optimise some additional token info calls we used to have, like readTokenIsFrozen etc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@dchernokur