Skip to content

fix: unit tests for deriveEvents, type=button, aria-label, aria-pressed (#433 #438 #439 #440) - #451

Open
observerr411 wants to merge 1 commit into
FlowwStar:mainfrom
observerr411:main
Open

fix: unit tests for deriveEvents, type=button, aria-label, aria-pressed (#433 #438 #439 #440)#451
observerr411 wants to merge 1 commit into
FlowwStar:mainfrom
observerr411:main

Conversation

@observerr411

Copy link
Copy Markdown

Summary

Test plan

  • Run npm test — new deriveEvents tests should all pass
  • Inspect streams page in browser: token/status/clear filter buttons no longer lack type
  • Use a screen reader on the dashboard with a stale price: emoji is announced as "Price may be outdated"
  • Use a screen reader on the create form: toggling amount mode announces aria-pressed state changes

…aria-label on stale-price emoji, aria-pressed on amount-mode toggle

closes FlowwStar#433
closes FlowwStar#438
closes FlowwStar#439
closes FlowwStar#440
@drips-wave

drips-wave Bot commented Jul 29, 2026

Copy link
Copy Markdown

@observerr411 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment