Repository navigation
Friendly error messages + Developer Mode (pattern 06) - #37
Merged
Merged
Conversation
) Production users no longer see raw HTTP detail/stacktraces in toasts: - utils/friendlyError.ts maps an ApiError by status class to a calm message (via ui.errors.* keys with English fallbacks, so it works with zero i18n setup). utils/devMode.ts + a Settings > About toggle (off by default) flip to the raw detail for debugging. - notify.error shows the friendly message; the 'Report issue' action still carries the full raw detail + apiError. - Non-hook translate() accessor on useI18n for module-level localization. tsc clean; eslint clean; Vitest 1207 passed; build OK; complexity gate OK. Closes #36
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #36
Finishes pattern 06. Production users no longer see raw HTTP
detail/stacktraces in error toasts.utils/friendlyError.tsmaps anApiErrorby status class (404/4xx/409/413/5xx/network) to a calm message, resolved viaui.errors.*keys with English fallbacks (works before an app adds catalog entries -> no i18n-parity churn).utils/devMode.ts+ a Settings > About toggle (off by default) flip to the raw technical detail for debugging.notify.errorshows the friendly message; the Report issue action still carries the full raw detail +apiError.translate()accessor onuseI18nlets module-level code (notify/friendlyError) localize without a hook.Green
tsc clean. ESLint clean. Vitest 124 files / 1207 passed (+8; no component test broke). Build OK. Complexity gate OK.