Skip to content

Comments

fix: fetch token balance after wallet registration#829

Merged
pedroferreira1 merged 7 commits intomasterfrom
fix/fetch-balance-after-token-register
Jan 28, 2026
Merged

fix: fetch token balance after wallet registration#829
pedroferreira1 merged 7 commits intomasterfrom
fix/fetch-balance-after-token-register

Conversation

@pedroferreira1
Copy link
Member

Motivation

After a token swap or a nano contract transaction in Reown that has a new token registration, the balance was not being fetched.

Acceptance Criteria

  • Create a util method for token registration and metadata fetch
  • Use these util methods in all places where we register tokens

Note: in the token creation register wouldn't be necessary to fetch the balance neither the metadata but I decided to handle all places the same for consistency.

Security Checklist

  • Make sure you do not include new dependencies in the project unless strictly necessary and do not include dev-dependencies as production ones. More dependencies increase the possibility of one of them being hijacked and affecting us.

@pedroferreira1 pedroferreira1 self-assigned this Jan 27, 2026
@pedroferreira1 pedroferreira1 moved this from Todo to In Progress (Done) in Hathor Network Jan 27, 2026
@pedroferreira1 pedroferreira1 moved this from In Progress (Done) to In Review (WIP) in Hathor Network Jan 28, 2026
@tuliomir tuliomir moved this from In Review (WIP) to In Review (Done) in Hathor Network Jan 28, 2026
@pedroferreira1 pedroferreira1 merged commit 11197e6 into master Jan 28, 2026
1 check passed
@pedroferreira1 pedroferreira1 deleted the fix/fetch-balance-after-token-register branch January 28, 2026 22:00
@github-project-automation github-project-automation bot moved this from In Review (Done) to Waiting to be deployed in Hathor Network Jan 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Waiting to be deployed

Development

Successfully merging this pull request may close these issues.

3 participants