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.

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

fix: stx available balance issues #6061

Merged
merged 1 commit into from
Jan 19, 2025
Merged

fix: stx available balance issues #6061

merged 1 commit into from
Jan 19, 2025

Conversation

alter-eggo
Copy link
Contributor

@alter-eggo alter-eggo commented Jan 16, 2025

Try out Leather build ec6d4e3Extension build, Test report, Storybook, Chromatic

This pr fixes part of stx balance issues. it sets:

  • total balance for balance on home page
  • available balance for balance in send form

@alter-eggo alter-eggo force-pushed the fix/stx-balance-issues branch from 35cd20c to 51e0a3e Compare January 17, 2025 10:34
@alter-eggo alter-eggo marked this pull request as ready for review January 17, 2025 10:52
@alter-eggo alter-eggo force-pushed the fix/stx-balance-issues branch from 51e0a3e to 3763d27 Compare January 17, 2025 10:52
@markmhendrickson
Copy link
Collaborator

Let's add tests here or are you planning to add them more holistically for cases in the underlying issue? (Please link the issue btw)

@alter-eggo
Copy link
Contributor Author

@markmhendrickson yeah, I'd like to add them separately, also there will be sep pr to support hiro api enhancement

@alter-eggo alter-eggo force-pushed the fix/stx-balance-issues branch from c6549b9 to 57808fc Compare January 17, 2025 13:44
@alter-eggo alter-eggo force-pushed the fix/stx-balance-issues branch from 57808fc to ec6d4e3 Compare January 17, 2025 21:50
@alter-eggo alter-eggo merged commit 6f054bb into dev Jan 19, 2025
32 checks passed
@alter-eggo alter-eggo deleted the fix/stx-balance-issues branch January 19, 2025 19:33
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