Skip to content

Show per-item translated error messages on /reader operations#14

Open
MojRezakhani wants to merge 1 commit into
lukaslerche:mainfrom
MojRezakhani:fix/reader-page-error-display
Open

Show per-item translated error messages on /reader operations#14
MojRezakhani wants to merge 1 commit into
lukaslerche:mainfrom
MojRezakhani:fix/reader-page-error-display

Conversation

@MojRezakhani

Copy link
Copy Markdown
Contributor

Previously, when secure/unsecure/edit/clear operations failed, errors were only logged to the browser console — the user saw no indication that something went wrong.

Now:

  • Each reader item shows its own error inline within its card
  • Same alert style as /tagging (icon + text)
  • Auto-clears after 5 seconds
  • Cleared immediately on retry
  • Pattern-matches common errors: wrong password, tag not found, reader unreachable, tag locked, authentication failed
  • Paraglide support (EN + DE)

Also fixes a few translation inconsistencies discovered along the way:

  • multiple_tags_detected_message: en.json had German text; added missing German version
  • reader_initialisation_failure_message: de.json had wrong spelling (with 'z' instead of 's') that didn't match the code; renamed and fixed a typo (Lesegräte -> Lesegeräts)

Root causes fixed:

  1. reader.secure/unsecure/edit/clear return { success, message } instead of throwing, so the existing catch blocks never fired
  2. Backend returns errors in the 'error' field but feig.ts only read 'message', so error text was lost

Previously, when secure/unsecure/edit/clear operations failed, errors
were only logged to the browser console — the user saw no indication
that something went wrong.

Now:
- Each reader item shows its own error inline within its card
- Same alert style as /tagging (icon + text)
- Auto-clears after 5 seconds
- Cleared immediately on retry
- Pattern-matches common errors: wrong password, tag not found,
reader unreachable, tag locked, authentication failed
- Paraglide support (EN + DE)

Also fixes a few translation inconsistencies discovered along the way:
- multiple_tags_detected_message: en.json had German text; added missing
German version
- reader_initialisation_failure_message: de.json had wrong spelling
(with 'z' instead of 's') that didn't match the code; renamed and
fixed a typo (Lesegräte -> Lesegeräts)

Root causes fixed:
1. reader.secure/unsecure/edit/clear return { success, message } instead
    of throwing, so the existing catch blocks never fired
2. Backend returns errors in the 'error' field but feig.ts only read
    'message', so error text was lost
@vercel

vercel Bot commented Jul 16, 2026

Copy link
Copy Markdown

@MojRezakhani is attempting to deploy a commit to the lerche's projects Team on Vercel.

A member of the Team first needs to authorize it.

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