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
Conversation
? asset?.networks | ||
? { | ||
...ethereumUtils.formatGenericAsset(genericAsset, nativeCurrency), | ||
type: asset.type, |
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
@@ -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 comment
The 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
@@ -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 comment
The 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
Fixes APP-1090
What changed (plus any additional context for devs)
this exposed a few regressions that came from us allowing search of L2 only assets, before search ran under the assumption that all results were on mainnet which is no longer the case. working on a few smol tweaks rn
there's some more work that can be done but its contingent on us moving off out zerion websockets first
Screen recordings / screenshots
https://cloud.skylarbarrera.com/Screen-Recording-2024-01-22-12-57-56.mp4
What to test