-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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: Support dynamic native token name on gas component #28048
Conversation
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
48e9f09
to
1a0b87b
Compare
1a0b87b
to
dcd8152
Compare
Builds ready [dcd8152]
Page Load Metrics (2057 ± 124 ms)
Bundle size diffs
|
@@ -32,14 +32,18 @@ export function useFeeCalculations(transactionMeta: TransactionMeta) { | |||
const conversionRate = useSelector(getConversionRate); | |||
const fiatFormatter = useFiatFormatter(); | |||
|
|||
const multichainNetwork = useSelector(getMultichainNetwork); |
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.
This is okay in the context of a fix, but long-term this is ultimately still using getProviderConfig
where as we'll want to pass in our transaction chain ID somehow.
Missing release label release-12.6.0 on PR. Adding release label release-12.6.0 on PR and removing other release labels(release-12.7.0), as PR was cherry-picked in branch 12.6.0. |
Description
Uses the multinetwork ticker.
Related issues
Fixes: #28001
Manual testing steps
See original ticket linked above.
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist