Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

[TypeScript] Fix regression in type of <FunctionField> render #8964

Merged
merged 1 commit into from
Jun 1, 2023

Conversation

slax57
Copy link
Contributor

@slax57 slax57 commented Jun 1, 2023

Fixes #8960

I originally intended to fix the types properly, by setting the type to be Record<string, any> in case RecordType was not provided, but couldn't find a way to do so. 馃槮
So I had to resort to using any as the default type.

Nevertheless, autocompletion works when setting the type either to <FunctionField> or to render's record param, and TS no longer throws an error. 馃檪

@slax57 slax57 added the RFR Ready For Review label Jun 1, 2023
@fzaninotto fzaninotto merged commit bf84341 into master Jun 1, 2023
10 checks passed
@fzaninotto fzaninotto deleted the FunctionField-TS-regression branch June 1, 2023 11:21
@fzaninotto fzaninotto added this to the 4.11.1 milestone Jun 1, 2023
elstgav added a commit to elstgav/react-admin that referenced this pull request Jun 1, 2023
elstgav added a commit to elstgav/react-admin that referenced this pull request Jun 1, 2023
TurtIeSocks pushed a commit to TurtIeSocks/react-admin that referenced this pull request Jun 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RFR Ready For Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Regression in FunctionField when using TypeScript
2 participants