Skip to content

feat: implementing account balance refresh [PERA-3407]#235

Open
fmsouza wants to merge 2 commits intomainfrom
fmsouza/pera-3407
Open

feat: implementing account balance refresh [PERA-3407]#235
fmsouza wants to merge 2 commits intomainfrom
fmsouza/pera-3407

Conversation

@fmsouza
Copy link
Contributor

@fmsouza fmsouza commented Mar 23, 2026

Pull Request Template

Description

  • Implements account balance refresh
    • calls the "should refresh" for all the imported walled addresses
    • only invalidate and refreshes the currently active account

Related Issues

Checklist

  • Have you tested your changes locally?
  • Have you reviewed the code for any potential issues?
  • Have you documented any necessary changes in the project's documentation?
  • Have you added any necessary tests for your changes?
  • Have you updated any relevant dependencies?

Additional Notes

  • Add any additional notes or comments that may be helpful for reviewers.

@fmsouza fmsouza requested review from wjbeau and yasincaliskan March 23, 2026 14:41
@fmsouza fmsouza self-assigned this Mar 23, 2026
onRefresh?: () => void
}

export const usePolling = (options?: UsePollingOptions) => {
Copy link
Contributor

Choose a reason for hiding this comment

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

Currently useShouldRefreshMutation will fetch the refresh status for all accounts. I don't know exactly what we get back there but one other thing we need to hang off of this is rekeys. It might be worth a conversation with the Android or iOS guys about what exactly they do with this refresh loop. It's quite a complicated mechanic on the native app and worth delving into, to make sure we don't miss anything. At the very least though, we should probably update the refresh query to only fetch what we actually care about refreshing.

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.

2 participants