Skip to content
This repository has been archived by the owner on Apr 17, 2019. It is now read-only.

React does not recognize the dependsOnValue #23

Closed
llioor opened this issue Jan 14, 2018 · 5 comments
Closed

React does not recognize the dependsOnValue #23

llioor opened this issue Jan 14, 2018 · 5 comments

Comments

@llioor
Copy link

llioor commented Jan 14, 2018

Hi,
I think it is an issue with React 16 but I'm not sure.
I have this code:

React does not recognize the dependsOnValue prop on a DOM element

 <DependentInput dependsOn="country_id">
              <StatesSelectInput source="state_id"/>
            </DependentInput>

In my console I have this error:

Warning: React does not recognize the `dependsOnValue` prop on a DOM element. If you intentionally want it to appear in the DOM as a custom attribute, spell it as lowercase `dependsonvalue` instead. If you accidentally passed it from a parent component, remove it from the DOM element.

The error is clear, it is not urgent but just annoying.
Thanks.

@afilp
Copy link

afilp commented Mar 3, 2018

@djhi Does it need sanitizeProps I assume?

@afilp
Copy link

afilp commented Mar 4, 2018

@djhi Is it known that dependentInput is not functional in next? I am not sure if I should create an issue here or in admin-on-rest. I thought it is more suitable there, as everything is there now:

https://github.com/marmelab/admin-on-rest/issues/1607

If you want me to create the issue here, please let me know, I am really very stuck with this...

@fzaninotto
Copy link
Member

Fixed by https://github.com/marmelab/admin-on-rest/pull/1675, let's track the issue there.

@khukeke
Copy link

khukeke commented Mar 14, 2019

Fixed by https://github.com/marmelab/admin-on-rest/pull/1675, let's track the issue there.

@fzaninotto this url is 404.

@fzaninotto
Copy link
Member

try marmelab/react-admin#1675

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

No branches or pull requests

4 participants