fix token address searches on discover screen

rainbow-me / rainbow

🌈‒ the Ethereum wallet that lives in your pocket

https://rainbow.me

GNU General Public License v3.0

3.96k stars 628 forks source link

Closed maxbbb closed 4 days ago

maxbbb commented 1 week ago

Fixes APP-1674

Incorrectly named variable "params" was replaced with correctly named variable "queryParams", which fixed mainnet address searches throwing an error.
tokenSearch function logic was changed to more closely match search logic in src/__swaps__/screens/Swap/resources/search/search.ts which fixed cross-chain address search.

Search for an exact address on the discover screen (ex "0x6982508145454Ce325dDbE47a25d4ec3d2311933")
Search normally

linear[bot] commented 1 week ago

brunobar79 commented 1 week ago

Launch in simulator or device for a01664d0861a8c054f678c5565f79d24fc08bcc5

brunobar79 commented 6 days ago

Switching reviewers from @walmat to @derHowie since Matthew is OOO

sentry-io[bot] commented 3 days ago

This pull request was deployed and Sentry observed the following issues:

‼️ Error: [handleDeeplink]: invalid or unknown deeplink getImportedAsset(src/hooks/useSearchCurrencyList) View Issue
‼️ Error: [handleDeeplink]: invalid or unknown deeplink getImportedAsset(src/hooks/useSearchCurrencyList) View Issue
‼️ Error: [handleDeeplink]: invalid or unknown deeplink getImportedAsset(src/hooks/useSearchCurrencyList) View Issue

_{Did you find this useful? React with a 👍 or 👎}