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

Allow fetching metadata by any property of AssetID #719

Conversation

jessepinho
Copy link
Contributor

No description provided.

@jessepinho jessepinho force-pushed the jessepinho/metadata-fix branch 3 times, most recently from d5c39ec to 88312f0 Compare March 12, 2024 18:27
@jessepinho jessepinho force-pushed the jessepinho/metadata-fix branch from 88312f0 to d5346ed Compare March 12, 2024 18:29
@jessepinho jessepinho marked this pull request as ready for review March 12, 2024 19:33
@grod220
Copy link
Contributor

grod220 commented Mar 12, 2024

Pairing w/ Jesse. Merging!

@grod220 grod220 merged commit c9e1291 into jessepinho/slow-results-from-extension-web-701 Mar 12, 2024
6 checks passed
@grod220 grod220 deleted the jessepinho/metadata-fix branch March 12, 2024 19:37
jessepinho added a commit that referenced this pull request Mar 12, 2024
* Render all validator infos at once instead of streaming

* Flush to state periodically

* Tweak how we load delegations

* Pass down more props to avoid unnecessary rerenders

* Move Delegations to its own component to avoid unnecessary rerenders

* Fix tests

* Use more granular selectors instead of prop drilling

* Use useShallow

* Revert name change

* Tweak syntax

* Simplify use of state in Delegations; remove unused props

* Remove unneeded fragment

* Tweak logic; add comments

* Add performance learnings to docs

* Create a useStoreShallow hook

* Tweak interval

* Allow fetching metadata by any property of AssetID (#719)

* Use throttle()

* Fix test
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