search

orlyapps / nova-belongsto-depend

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

Laravel 5.8 issues #33

Closed fwartner closed 5 years ago

fwartner commented 5 years ago

Seems like it´s not working with L5.8 anymore. Something broke the Keys...

ghost commented 5 years ago

Hi! I just upgrade my Laravel to v5.8 and there is a change in the relations which is changing the getForeignKey method by getForeignKeyName method. I get this error when I try to run my v5.8 because Nova is using nova-belongsto-depend which is using getForeignKey method, so I change this call in vendor/orlyapps/nova-belongsto-depend/src/NovaBelongsToDepend.php's method "resolve" and now is working fine.

orlyapps commented 5 years ago

fixed with https://github.com/orlyapps/nova-belongsto-depend/commit/8082308de445a00cf1e38c990daa7b064c8acc10

Going to tag a v2.0 release