orlyapps / nova-belongsto-depend

Larave Nova BelongsTo Field with Dependcy
MIT License
182 stars 65 forks source link

add withoutGlobalScopes to controller #92

Open MichailSuroven opened 3 years ago

MichailSuroven commented 3 years ago

Global scopes like soft delete breakes controller

Must be somthing like this:

$model = $request->modelClass::withoutGlobalScopes()->find($request->dependKey);

or need to define some closure to prepare model query

orlyapps commented 3 years ago

@MichailSuroven please make a PR and i will merge ist asap

MichailSuroven commented 3 years ago

seems my composer take renovate branch instead of master

but in master branch i guess field.js compiled from old resource, so any request not contain dependOn prop and fails, i cant check newer version

orlyapps commented 3 years ago

@MichailSuroven ah yes. The master is currently in development and not stable. Let us release v3 and then you can make a pr. Sorry for the delay D: