Review

Author: fzaninotto

docs/CRUD.md

@@ -53,7 +53,7 @@ Here, the `<List>` component will call `dataProvider.getList('posts')` to fetch
 
 ## Page Context
 
-`<List>` and other page component don't just fetch data; they provide a way to update the page settings:
+`<List>` and other page components don't just fetch data; they provide a way to update the page settings:
 
 - Sort field and order
 - Current page & page size
@@ -123,10 +123,12 @@ This is the equivalent of the following react-router configuration:
 
 ```jsx
 <ResourceContextProvider value="posts">
-    <Route path="/posts" element={<PostList />} />
-    <Route path="/posts/:id/show" element={<PostShow />} />
-    <Route path="/posts/:id" element={<PostEdit />} />
-    <Route path="/posts/create" element={<PostCreate />} />
+    <Routes>
+        <Route path="/posts" element={<PostList />} />
+        <Route path="/posts/:id/show" element={<PostShow />} />
+        <Route path="/posts/:id" element={<PostEdit />} />
+        <Route path="/posts/create" element={<PostCreate />} />
+    </Routes>
 </ResourceContextProvider>
 ```
 

docs/DataFetchingGuide.md

@@ -192,14 +192,14 @@ POST /api/user/123/ban
 
 The react-admin way to expose these endpoints to the app components is to add a custom method in the `dataProvider`:
 
-```jsx
+```tsx
 import simpleRestDataProvider from 'ra-data-simple-rest';
 
 const baseDataProvider = simpleRestDataProvider('http://path.to.my.api/');
 
 export const dataProvider = {
     ...baseDataProvider,
-    banUser: (userId) => {
+    banUser: (userId: string) => {
         return fetch(`/api/user/${userId}/ban`, { method: 'POST' })
             .then(response => response.json());
     },
@@ -208,11 +208,11 @@ export const dataProvider = {
 
 Then you can use react-query's `useMutation` hook to call the `dataProvider.banUser()` method:
 
-```jsx
+```tsx
 import { useDataProvider } from 'react-admin';
 import { useMutation } from '@tanstack/react-query';
 
-const BanUserButton = ({ userId }) => {
+const BanUserButton = ({ userId }: { userId: string }) => {
     const dataProvider = useDataProvider();
     const { mutate, isPending } = useMutation({
         mutationFn: () => dataProvider.banUser(userId)