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

dependsOn doesn't work for attributes in settings #138

Open
om0x1ar opened this issue Oct 6, 2022 · 6 comments
Open

dependsOn doesn't work for attributes in settings #138

om0x1ar opened this issue Oct 6, 2022 · 6 comments
Labels
help wanted Extra attention is needed

Comments

@om0x1ar
Copy link

om0x1ar commented Oct 6, 2022

Hello, the dependsOn method doesn't work for any of the fields inside the settings resource as the update-field request returns 404.

image

@KasparRosin
Copy link
Member

KasparRosin commented Oct 7, 2022

As an quick alternative, I think we should have support for outl1ne/nova-dependency-container
I personally haven't used it in a while inside nova-settings, so it might need testing to see whether some nova release could have broken it.

@chrillep
Copy link
Contributor

chrillep commented Nov 4, 2022

second this. not even getting any error logs :)
sends
: PATCH to
/nova-api/nova-settings/general/update-fields

https://nova.laravel.com/docs/4.0/resources/fields.html#dependent-fields

@chrillep
Copy link
Contributor

chrillep commented Nov 19, 2022

https://nova.laravel.com/docs/4.0/customization/fields.html#dependent-form-field

Route::delete('/nova-api/nova-settings/{path}/field/{fieldName}', 'SettingsController@deleteImage');

add something like

    Route::patch('/nova-api/nova-settings/{path}/update-fields', [SettingsController::class, 'sync']);

and add sync functionality to SettingsController

@chrillep
Copy link
Contributor

chrillep commented Mar 10, 2023

https://github.com/stepanenko3/nova-settings#using-dependson-with-out-package now has dependsOn fields support. Just FYI

@Tarpsvo Tarpsvo added the help wanted Extra attention is needed label Mar 16, 2023
@shane-smith
Copy link

Just to confirm, this is not implemented yet?

I'm attempting to make a Text field visible only when a Boolean field is true, but dependsOn seems to be ignored.

@devskio
Copy link

devskio commented Feb 14, 2024

Hello, any update on this topic? Is there a plan to fix this?
Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

6 participants