Inconsistent rendering of errors in List components
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 5/5
- 見積もり時間
- 1週間以上
- 初心者へのやさしさ
- 25/100
- issue の種類
- バグ
- 明瞭さ
- 説明が足りない
- 活発さ
- 停滞
- 技術スタック
- react, typescript
- 領域
- frontend
調査の方向性
Storybook の「List / Error in Fetch」ストーリーを開き、List、ReferenceManyField、Datagrid、SimpleList、SingleFieldList で発生する失敗を再現してください。Issue で提案されている責務分担を確認し、一貫性があり重複しないエラー状態と、上書き可能な List のエラーレンダリングを定義して検証してください。実装を完了と見なすには、Issue で動作を決定する必要があります。
索引モデルが issue の本文から書いたものです。
説明
What you were expecting:
When the dataProvider.getList() call fails, components rendering a list should render something meaningful and consistent.
What happened instead:
- For fetcher components
<List>renders the error messages twice (once in the notification, once in the<Error>component), and if the content component renders it, too, that's three times<ReferenceManyField>renders the notification (no error component)
- For content components
<Datagrid>and<SimpleList>render<ListNoResults>, which is misleading<SingleFieldList>fails.
Besides, in the <List> component, it is not possible to override / hide the <Error> render (cf https://github.com/marmelab/react-admin/issues/8528#issuecomment-1369117499).
Steps to reproduce:
Check the Storybook for List / Error in Fetch : https://react-admin-storybook.vercel.app/?path=/story/ra-ui-materialui-list-list--error-in-fetch
Other information:
Which component should handle the error case? In my opinion, just like for the loading case, it's the content components' responsibility (<Datagrid>, <SimpleList>, etc), not the fetcher component (<List>, etc). But I'm open to discussion on this one.
Environment
- React-admin version: 4.x
- Last version that did not exhibit the issue (if applicable): n/a
- React version: 17
- Browser: Chrome
- 主要言語
- TypeScript
- スター
- 27k
- フォーク
- 5.5k
- 平均マージ
- 1日 1時間
- マージ済み PR（30日）
- 33
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
marmelab/react-admin のほかの issue
-
Add emptyText property to SelectArrayInput対応中かも @kwitykwity が 65 日前に担当しました。 オープンenhancement good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
marmelab/react-admin#10454 · コメント 7 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 4/5 1週間以上 初心者へのやさしさ 38/100
marmelab/react-admin#11426 · リアクション 1 件 ·
メンテナーはふだん 1 日以内に返信
-
ra-data-graphql-simple: outdated graphql-ast-types-browser dependency causes npm peer warnings再び着手できるかも @bipul724 が 19 日前に担当しましたが、オープン中のプルリクエストはありません。 オープン
難易度 4/5 3〜5日 初心者へのやさしさ 25/100
marmelab/react-admin#11395 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
Fix decode-uri-component DoS (CVE-2026-45822) by dropping query-string再び着手できるかも このイシューのプルリクエストはマージされずにクローズされました。 オープン
難易度 3/5 1〜2日 初心者へのやさしさ 68/100
marmelab/react-admin#11380 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
useCreate > onError returns callback function where error is "unknown"対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープン
難易度 3/5 1〜2日 初心者へのやさしさ 65/100
marmelab/react-admin#11262 · コメント 3 件 ·
メンテナーはふだん 1 日以内に返信
marmelab/react-admin の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 66/100
メンテナーはふだん 2 日以内に返信
-
reproduced
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
anomalyco/opencode#62052 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
strapi/design-system#2084 ·
メンテナーはふだん 3 日以内に返信
-
check:passed streams:remove
難易度 1/5 1時間未満 初心者へのやさしさ 67/100
メンテナーはふだん 1 日以内に返信
-
OAuth documentation for GET /api/v2/users/me/ appears inconsistent with required global read scopeオープン
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
メンテナーはふだん 1 日以内に返信