Editor load errors display raw exception text to users
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 55/100
Direzione di ricerca
Inizia da displayError in ios/Sources/GutenbergKit/Sources/EditorViewController.swift e da EditorErrorView/showErrorPhase in android/Gutenberg/src/main/java/org/wordpress/gutenberg/GutenbergView.kt. Esamina le righe 85–88 di EditorHttpClient.kt e la relazione con l’EditorLoadFailure tipizzato proposto in #603. Il lavoro è completato quando entrambe le piattaforme mostrano messaggi di auth o ritentabili appropriati per l’utente, mentre i dettagli grezzi restano disponibili per i log e la telemetria.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Description
When the editor fails to load, both platforms render the underlying exception message directly in the library's error view. These strings are written for developers, and they reach users of both production apps.
Examples seen in testing:
- iOS:
No route was found matching the URL and request method. - Android:
rest_no_route: No route was found matching the URL and request method. - Android:
Unknown error with status code: 500 - Android:
Failed to load editor; unable to resolve host…
Android additionally prefixes the machine-readable WP error code, because WPErrorResponse.message is built as "${error.code}: ${error.message}" (EditorHttpClient.kt#L85-L88).
This is not demo-only
The error views live in the library, and neither host app can override them:
- iOS:
displayErrorrenderserror.localizedDescription(EditorViewController.swift#L1015-L1025).didFailToLoadis documented as firing after the error is displayed, and WP-iOS's implementation is analytics-only. - Android:
GutenbergViewownsEditorErrorViewand callsshowErrorPhasedirectly (GutenbergView.kt#L373-L384). WP-Android embedsGutenbergViewand inherits the view as-is.
Proposed change
- Map failure classes to user-appropriate copy rather than echoing exception text
- Auth failures: something actionable, e.g. "Sign in again to continue editing"
- Server/network failures: a generic retryable message
- Keep the raw error for logs and telemetry, not the UI
- Android: stop prefixing the WP error code in user-visible text
- Consider a retry affordance where the failure is retryable
Notes
- #602 removes the
rest_no_routemessage from this view entirely, and #603 removes most transient 5xx cases under automatic fallback. What remains — auth failures, and everything under disabled fallback — still needs real copy, so this is worth doing independently and is lower priority than both. - Pairs naturally with the typed
EditorLoadFailureproposed in #603: the classification that drives host behavior can also drive the message.
- Lingua principale
- JavaScript
- Stelle
- 29
- Fork
- 7
- Merge medio
- 3g 3h
- PR unite (30g)
- 91
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di wordpress-mobile/GutenbergKit
-
[Type] Bug [Type] Code Quality iOS
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
wordpress-mobile/GutenbergKit#668 ·
I maintainer di solito rispondono entro 1 giorno
-
[Type] Bug [Type] Performance iOS
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
wordpress-mobile/GutenbergKit#666 ·
I maintainer di solito rispondono entro 1 giorno
-
[Type] Bug iOS
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
wordpress-mobile/GutenbergKit#664 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
wordpress-mobile/GutenbergKit#522 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
wordpress-mobile/GutenbergKit#443 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di wordpress-mobile/GutenbergKit
Issue simili
-
Notifications pagination accepts NaN and negative LIMIT/OFFSETForse già presa @Senthil455 l’ha presa oggi. ApertaBUGS MEDIUM
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
elixpo/blogs.elixpo#419 · 2 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
immich-app/immich#32426 ·
I maintainer di solito rispondono entro 1 giorno
-
automated issue report
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 70/100
lirantal/discoprint#45 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
scanaislop/aislop#482 ·
I maintainer di solito rispondono entro 1 giorno