[Chore]: Update react-admin to use ReactNode instead of ReactElement where applicable#10996
[Chore]: Update react-admin to use ReactNode instead of ReactElement where applicable#10996
Conversation
There was a problem hiding this comment.
Although this is obviously the correct thing to do, I wonder if we should release this at least in a minor version. @slax57, @fzaninotto thoughts?
I tend to agree we should put this in a minor version. |
Worth mentionning that the enterprise edition compiles without any changes with this branch |
66a5156 to
7b12a15
Compare
djhi
left a comment
There was a problem hiding this comment.
I think the <InfiniteList> props also mention title and is incorrect (only mention string)
Problem
A lot of RA components expect a
ReactElementwhere aReactNodecould be used.Solution
Replace
ReactElementwithReactNodewhere applicable.How To Test
N/A
Additional Checks
masterfor a bugfix or a documentation fix, ornextfor a featureAlso, please make sure to read the contributing guidelines.