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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ArrayInput Validation Inside TabbedFrom Throw Error #5844

Closed
nckenn opened this issue Feb 1, 2021 · 3 comments · Fixed by #5850
Closed

ArrayInput Validation Inside TabbedFrom Throw Error #5844

nckenn opened this issue Feb 1, 2021 · 3 comments · Fixed by #5850
Labels

Comments

@nckenn
Copy link

nckenn commented Feb 1, 2021

What you were expecting:
Before upgrading to latest version 3.12.0 all components in my applications is running without error.

What happened instead:
After upgrading to latest version i found this bug in one of my components :
KUJ9Tbe9Ej

So i try to remove my validation on ArrayInput and it works, but without validation : (
OzR4ZCAGq1

Steps to reproduce:
I think this is related to latest features of RA, specifically #5752

Other information:

Environment

  • React-admin version: 3.12.0
  • Last version that did not exhibit the issue (if applicable): 3.11.4
  • React version: ^17.0.1
  • Browser: Chrome
@djhi
Copy link
Collaborator

djhi commented Feb 2, 2021

Thanks for reporting this. I do not manage to reproduce the issue. Please provide a sample application showing the issue by forking the following CodeSandbox (https://codesandbox.io/s/github/marmelab/react-admin/tree/master/examples/simple).

@nckenn
Copy link
Author

nckenn commented Feb 2, 2021

Thanks for reporting this. I do not manage to reproduce the issue. Please provide a sample application showing the issue by forking the following CodeSandbox (https://codesandbox.io/s/github/marmelab/react-admin/tree/master/examples/simple).

Kindly check :) i added validation on backlinks and it throws error.
https://codesandbox.io/s/kind-moser-k6id1?file=/src/posts/PostEdit.js

@djhi
Copy link
Collaborator

djhi commented Feb 2, 2021

Indeed, I must have missed something in my tests. Thanks for the report!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants