-
Notifications
You must be signed in to change notification settings - Fork 35
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 custom networks assets #1186
Conversation
… @bruno/fix-custom-networks-assets
Here's the packed extension for this build: |
import { useCurrentCurrencyStore } from '~/core/state'; | ||
import { convertAmountToNativeDisplay } from '~/core/utils/numbers'; | ||
import { add, convertAmountToNativeDisplay } from '~/core/utils/numbers'; | ||
|
||
export function useUserAssetsBalance() { | ||
const { address } = useAccount(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wanna make this useCurrentAddressStore
since where here?
totalAssetsBalanceKnownNetworks as string, | ||
totalAssetsBalanceCustomNetworks as string, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
won't add
break if one of these are undefined
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The selector returns '0'
by default so we're good
… @bruno/fix-custom-networks-assets
Here's the packed extension for this build: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Here's the packed extension for this build: |
Fixes BX-####
Figma link (if any):
What changed (plus any additional context for devs)
There were a few bugs with custom networks
Screen recordings / screenshots
Ex. with Anvil as a custom RPC
What to test