outl1ne / nova-simple-repeatable

A Laravel Nova simple repeatable rows field.
MIT License
73 stars 44 forks source link

Fix SimpleRepeatable is not work in Action fields() due to missing ha… #34

Closed deckchan closed 2 years ago

deckchan commented 2 years ago

…ndling in resolveForAction

KasparRosin commented 2 years ago

Let's group the logic in a separate function.

I don't think we can call this->resolve($resource, $attribute) since that will call parent::resolve() which might have unwanted effects.

Also the two functions are very much different in terms of arguments/parameters they accept.

KasparRosin commented 2 years ago

Seems to be abandoned. Feel free to open a new pull request with the fixes.