Review

marmelab · Jun 16, 2022 · b8c3c2f · b8c3c2f
1 parent ceb3050
commit b8c3c2f
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 3 deletions.
diff --git a/docs/ReferenceArrayField.md b/docs/ReferenceArrayField.md
@@ -71,7 +71,7 @@ export const PostList = () => (
 | ------------ | -------- | ------------------- | -------- | ------------------------------------------------------------------------------------------------------------ |
 | `source`     | Required | `string`            | -        | Name of the property to display                                                                              |
 | `reference`  | Required | `string`            | -        | The name of the resource for the referenced records, e.g. 'tags'                                             |
-| `children`   | Required | `Element`           | -        | The Field element used to render the referenced records                                                      |
+| `children`   | Required | `Element`           | -        | One or several elements that render a list of records based on a `ListContext`                               |
 | `sortBy`     | Optional | `string | Function` | `source` | When used in a `List`, name of the field to use for sorting when the user clicks on the column header.       |
 | `filter`     | Optional | `Object`            | -        | Filters to use when fetching the related records (the filtering is done client-side)                         |
 | `pagination` | Optional | `Element`           | -        | Pagination element to display pagination controls. empty by default (no pagination)                          |

diff --git a/packages/ra-ui-materialui/src/field/ReferenceManyField.tsx b/packages/ra-ui-materialui/src/field/ReferenceManyField.tsx
@@ -139,7 +139,11 @@ ReferenceManyField.defaultProps = {
 // FIXME kept for backwards compatibility, unused, to be removed in v5
 export const ReferenceManyFieldView: FC<ReferenceManyFieldViewProps> = props => {
     const { children, pagination } = props;
-
+    if (process.env.NODE_ENV !== 'production') {
+        console.error(
+            '<ReferenceManyFieldView> is deprecated, use <ReferenceManyField> directly'
+        );
+    }
     return (
         <>
             {children}

diff --git a/packages/ra-ui-materialui/src/field/ReferenceOneField.tsx b/packages/ra-ui-materialui/src/field/ReferenceOneField.tsx
@@ -1,4 +1,4 @@
-import React, { ReactElement, ReactNode } from 'react';
+import React, { ReactNode } from 'react';
 import PropTypes from 'prop-types';
 import { Typography } from '@mui/material';
 import {