Inconsistent rendering of errors in List components
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 25/100
- Tipo de issue
- Error
- Claridad
- Necesita aclaración
- Estado de actividad
- Estancado
- Stack tecnológico
- react, typescript
- Área
- frontend
Línea de trabajo
Abre la story de Storybook “List / Error in Fetch” y reproduce los fallos en List, ReferenceManyField, Datagrid, SimpleList y SingleFieldList. Revisa la división de responsabilidades propuesta en el issue y, después, define y valida un estado de error coherente y no duplicado, con un renderizado de errores de List que se pueda sobrescribir; el issue necesita un comportamiento decidido antes de que la implementación pueda considerarse terminada.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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
- Lenguaje dominante
- TypeScript
- Estrellas
- 27k
- Forks
- 5.5k
- Merge medio
- 1 d 3 min
- PR fusionados (30 d)
- 32
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de marmelab/react-admin
-
Blank page after logout when the app re-renders `<Admin>` during `authProvider.logout()`Posiblemente ocupada @uchiha-suraj la tomó hace 1 día. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
marmelab/react-admin#11431 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Pagination: page button aria-labels are not translated on small screensPosiblemente ocupada @Singmin la tomó hace 3 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
marmelab/react-admin#11427 ·
Los mantenedores suelen responder en 1 día
-
Add emptyText property to SelectArrayInputPosiblemente ocupada @kwitykwity la tomó hace 61 días. Abiertoenhancement good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
marmelab/react-admin#10454 · 7 comentarios ·
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 Más de una semana Aptitud para principiantes 38/100
marmelab/react-admin#11426 · 1 reacción ·
Los mantenedores suelen responder en 1 día
-
ra-data-graphql-simple: outdated graphql-ast-types-browser dependency causes npm peer warningsQuizá libre de nuevo @bipul724 la tomó hace 14 días y no hay ningún pull request abierto. Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
marmelab/react-admin#11395 · 1 comentario ·
Los mantenedores suelen responder en 1 día
Todos los issues de marmelab/react-admin
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
Slaymish/HealthAgent#7 ·
-
area: engine topic: math type: bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 80/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
Get-Concord-AI/concord-mcp#310 ·
Los mantenedores suelen responder en 1 día
-
Correct install_claude: 'marketplace add' over a different source is refused, not replaced in placeAbiertotype:bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 80/100
nightgauge/nightgauge#2804 ·
Los mantenedores suelen responder en 1 día