Skip to content
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 throwing NFT filters panel #3513

Merged
merged 3 commits into from
Jul 3, 2023
Merged

Fix throwing NFT filters panel #3513

merged 3 commits into from
Jul 3, 2023

Conversation

jagodarybacka
Copy link
Contributor

@jagodarybacka jagodarybacka commented Jun 30, 2023

What

Follow up for #3486

Add fallback from name to address in NFT filters. To avoid NFT filters throwing error when the account is not yet loaded let's add a fallback from name to address to put in the SharedToggle label.

Testing

  • install extension and throttle network so it is super slow 🐌
  • add account and while it is loading quickly go to NFTs and try to click on the filters panel as soon as it is unblocked
  • filters panel should not throw

Latest build: extension-builds-3513 (as of Mon, 03 Jul 2023 16:12:23 GMT).

To avoid NFT filters throwing error when account is not yet loaded
let's add fallback from name to address to put in the SharedToggle label.
Copy link
Contributor

@Shadowfiend Shadowfiend left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, shipping it 🚢

@Shadowfiend Shadowfiend merged commit 756cbcf into main Jul 3, 2023
@Shadowfiend Shadowfiend deleted the nft-filters-fallback-name branch July 3, 2023 16:07
@Shadowfiend Shadowfiend mentioned this pull request Jul 6, 2023
kkosiorowska pushed a commit that referenced this pull request Jul 10, 2023
## Highlights

- An updated Taho token list! This will start auto-updating Soon™.

## What's Changed
* Update Taho token list reference by @kkosiorowska in
#3506
* Don't run e2e tests when workflow is triggered by push of tag by
@michalinacienciala in #3514
* Fix throwing NFT filters panel by @jagodarybacka in
#3513
* Use gas price from 0x quote to setup swap transaction by
@jagodarybacka in #3516
* v0.40.0 by @hyphenized in
#3507
* Add cron job to sync pledge signers by @hyphenized in
#3505
* Fix invalid workflow file by @hyphenized in
#3519
* Remove workflow trigger used for testing by @hyphenized in
#3520
* Fix migration for cleaning account balances by @jagodarybacka in
#3527
* v0.40.1 with op swaps gas price hotfix by @Shadowfiend in
#3518
* v0.40.2 - accounts migration fix by @jagodarybacka in
#3529


**Full Changelog**:
v0.40.2...v0.41.0

Latest build:
[extension-builds-3531](https://github.com/tahowallet/extension/suites/14093546910/artifacts/788185513)
(as of Thu, 06 Jul 2023 02:05:27 GMT).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants