Skip to content

Conversation

ruijialin-avalabs
Copy link
Contributor

@ruijialin-avalabs ruijialin-avalabs commented Oct 14, 2025

Description

Ticket: CP-12207

Please provide:

  • update balance listener to fetch xp networks every 5mins (prod)
  • update redux state balance.status from a boolean type to Record<QueryType, QueryStatus>
  • save xp account balance by key of address

Screenshots/Videos

Include relevant screenshots or screen recordings of iOS and Android.

  • modified fetching interval to 5 seconds for testing purpose in this video
Screen.Recording.2025-10-14.at.6.42.41.PM.mov

Testing

Dev Testing (if applicable)

  • verify xp balance are updated every 5 mins by checking the logger fetching xp balances
  • or checkout the branch and modify the interval to something like 10 seconds to make sure it gets updated every 10 seconds

QA Testing (if applicable)

  • Provide instructions for QA to test this feature thoroughly
  • State expected behavior / acceptance criteria

Checklist

Please check all that apply (if applicable)

  • I have performed a self-review of my code
  • I have verified the code works
  • I have included screenshots / videos of android and ios
  • I have added testing steps
  • I have added/updated necessary unit tests
  • I have updated the documentation

error: Error | null
}

export type BalancesForXpAddress = Omit<BalancesForAccount, 'accountId'>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks like it still contains accountAddress. shouldn't this just contain the walletId?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

use the accountAddress as key in Balances, so later on we can use the address to render balance by each address.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants