-
Notifications
You must be signed in to change notification settings - Fork 579
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
search: l2 clean up #5333
search: l2 clean up #5333
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -199,7 +199,13 @@ export default function ChartExpandedState({ asset }) { | |
return hasBalance | ||
? { ...asset } | ||
: genericAsset | ||
? ethereumUtils.formatGenericAsset(genericAsset, nativeCurrency) | ||
? asset?.networks | ||
? { | ||
...ethereumUtils.formatGenericAsset(genericAsset, nativeCurrency), | ||
type: asset.type, | ||
colors: asset?.colors, | ||
} | ||
: ethereumUtils.formatGenericAsset(genericAsset, nativeCurrency) | ||
: { ...asset }; | ||
}, [asset, genericAsset, hasBalance, nativeCurrency]); | ||
|
||
|
@@ -415,7 +421,7 @@ export default function ChartExpandedState({ asset }) { | |
<BuyActionButton color={color} /> | ||
</SheetActionButtonRow> | ||
) : null} | ||
{isL2 && ( | ||
{!networks && isL2 && ( | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. just making sure we done show 2 network disclaimers if coming from search |
||
<L2Disclaimer | ||
assetType={assetWithPrice.type} | ||
colors={colors} | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -545,7 +545,7 @@ | |
"read_more_button": "Read more", | ||
"available_networks": "Available on %{availableNetworks} networks", | ||
"available_network": "Available on the %{availableNetwork} network", | ||
"available_networkv2": "Also available on %{availableNetwork}", | ||
"available_networkv2": "Only available on %{availableNetwork}", | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. copy change to support if the asset is only on 1 network, which previously was not possible |
||
"l2_disclaimer": "This %{symbol} is on the %{network} network", | ||
"l2_disclaimer_send": "Sending on the %{network} network", | ||
"l2_disclaimer_dapp": "This app on the %{network} network", | ||
|
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.
networks only exists if coming from search, this overrides types to allow for l2 only assets for this screen