Editor load errors display raw exception text to users
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 55/100
調査の方向性
ios/Sources/GutenbergKit/Sources/EditorViewController.swift の displayError と、android/Gutenberg/src/main/java/org/wordpress/gutenberg/GutenbergView.kt の EditorErrorView/showErrorPhase から始めます。EditorHttpClient.kt の 85–88 行目と、#603 で提案されている型付き EditorLoadFailure との関係を確認します。完了の条件は、両プラットフォームでユーザーに適した auth または再試行可能なメッセージが表示され、raw の詳細はログとテレメトリーで引き続き利用できることです。
索引モデルが issue の本文から書いたものです。
説明
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.
- 主要言語
- JavaScript
- スター
- 29
- フォーク
- 7
- 平均マージ
- 3日 26分
- マージ済み PR（30日）
- 95
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
wordpress-mobile/GutenbergKit のほかの issue
-
[Type] Bug [Type] Code Quality iOS
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
wordpress-mobile/GutenbergKit#668 ·
メンテナーはふだん 1 日以内に返信
-
[Type] Bug [Type] Performance iOS
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
wordpress-mobile/GutenbergKit#666 ·
メンテナーはふだん 1 日以内に返信
-
[Type] Bug iOS
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
wordpress-mobile/GutenbergKit#664 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
wordpress-mobile/GutenbergKit#522 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
wordpress-mobile/GutenbergKit#443 ·
メンテナーはふだん 1 日以内に返信
wordpress-mobile/GutenbergKit の issue をすべて見る
似ている issue
-
t0ggles
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
nolimits4web/swiper#8240 · コメント 1 件 ·
-
netbox status: backlog topic: testing type: housekeeping
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
netbox-community/netbox#23395 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
メンテナーはふだん 1 日以内に返信
-
[scanner] check:spelling skips blog/ — posts and author metadata are never spell-checked対応中かも @hivecommons-hive が今日担当しました。 オープンagent/scanner bug hive/covered-by-pr hive/hosted-available-lke648397-260827-5n31
難易度 1/5 1時間未満 初心者へのやさしさ 72/100
cncf/endusers#1452 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
Edit: HTVKey.vnオープンchannels:edit check:passed
難易度 1/5 1時間未満 初心者へのやさしさ 72/100
メンテナーはふだん 6 日以内に返信