Skip to content

fix: use meaningful fallback in delete announcement#8

Merged
jonathan-beebe merged 2 commits into
mainfrom
fix/005-stale-announce-after-delete
Mar 29, 2026
Merged

fix: use meaningful fallback in delete announcement#8
jonathan-beebe merged 2 commits into
mainfrom
fix/005-stale-announce-after-delete

Conversation

@jonathan-beebe
Copy link
Copy Markdown
Owner

Summary

  • Change empty string fallback to 'item' in delete announcement when item is not found (race condition)
  • Prevents unhelpful Item "" deleted from ... screen reader messages
  • Resolves BUG-005 (stale-framework-in-announce-after-delete)

Test plan

  • New test: verifies delete announcement includes the item text ("Task A")
  • Full test suite passes (213 tests)

🤖 Generated with Claude Code

When an item is not found during deletion (race condition), the
announcement now says "item" instead of an empty string, avoiding
unhelpful "Item "" deleted from ..." messages for screen reader users.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 29, 2026

Deploy Preview for golden-scone-6bfeb1 ready!

Name Link
🔨 Latest commit 899c094
🔍 Latest deploy log https://app.netlify.com/projects/golden-scone-6bfeb1/deploys/69c9508b8c8bec0008e5f755
😎 Deploy Preview https://deploy-preview-8--golden-scone-6bfeb1.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

When the item is missing (race condition), announce "Item deleted from
..." cleanly instead of including awkward placeholder text.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@jonathan-beebe jonathan-beebe merged commit fe433d6 into main Mar 29, 2026
5 checks passed
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