Skip to content

fix(dashboard): keep validator logos aligned - #157

Merged
Albermonte merged 1 commit into
mainfrom
albermonte/fix-validator-logo-hydration
Oct 7, 2026
Merged

Albermonte merged 1 commit into
mainfrom
albermonte/fix-validator-logo-hydration

Conversation

@Albermonte

Copy link
Copy Markdown
Member

A saved 'All validators' preference could make the client hydrate different rows than the server, leaving logos attached to the previous validator. Enable the toggle and reload the dashboard to reproduce.

Read the saved filter after hydration. Unknown validators use address identicons; known validators keep their listed logos.

Checks:

  • 208 tests and typecheck pass.
  • ESLint passes excluding .superpowers/**. Full lint still reports 86 existing archive errors.
  • Playwright SSR fixture passes saved true/false reloads, toggle, and logo/name/address matching. Search/sort checks remain limited: both original and fixed fixtures leave stale rows.

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
validators-api-test 704fe55 Commit Preview URL

Branch Preview URL
Oct 07 2026, 06:36 PM

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
validators-api-main 704fe55 Commit Preview URL

Branch Preview URL
Oct 07 2026, 06:36 PM

@Albermonte
Albermonte merged commit 6d36416 into main Oct 7, 2026
3 checks passed
@Albermonte
Albermonte deleted the albermonte/fix-validator-logo-hydration branch October 7, 2026 18:39

This branch was successfully deployed

1 active deployment
testnet — 704fe55e Deployed Oct 7, 2026 by Albermonte via ci #818
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.

1 participant