Skip to content

feat(witnesses): align witness voting with legacy#301

Merged
ety001 merged 5 commits into
nextfrom
feat/witness-voting
May 28, 2026
Merged

feat(witnesses): align witness voting with legacy#301
ety001 merged 5 commits into
nextfrom
feat/witness-voting

Conversation

@ety001

@ety001 ety001 commented May 28, 2026

Copy link
Copy Markdown
Member

Summary

  • Align /witnesses voting UI with legacy (quick vote by account, proxy set/clear, vote list ordering).
  • Improve UX: avoid list refresh on actions, add success toasts, and stabilize optimistic vote state.
  • Surface chain broadcast error details to the user.

Test plan

  • pnpm verify
  • Open /witnesses
    • Vote/unvote witness (buttons update without flicker)
    • Vote outside top 100 via input
    • Set/clear witness proxy
    • If proxy set, witness voting should show chain error details

ety001 added 5 commits May 28, 2026 02:16
Add quick vote by username input, witness proxy broadcast, rank by vote
order, and login dialog for auth-required actions.
Split list vs action loading, add success toasts, surface broadcast details, and keep witness-vote UI stable with optimistic updates while refetching fresh account state.
Add unit tests for degradation-state to keep coverage thresholds passing.
Use a typed global cast to replace fetch in tests without triggering unused ts-expect-error warnings.
- Remove unnecessary double type assertion on currentProxy (SteemAccount
  already has proxy? field)
- Add signingKey guard in pendingAction effect run() to handle async
  key clearing
- Extract DISABLED_SIGNING_KEY to src/lib/steem/constants.ts for reuse
- Add afterEach reset in degradation-state test to prevent cross-test
  leakage
@ety001 ety001 merged commit 58b3a76 into next May 28, 2026
2 of 3 checks passed
@ety001 ety001 deleted the feat/witness-voting branch May 28, 2026 06:25
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