chris-ware / nova-breadcrumbs

Breadcrumbs for Laravel Nova
152 stars 44 forks source link

Class ChrisWare\NovaBreadcrumbs\NovaBreadcrumbs doesn't implement abstract method Laravel\Nova\Tool::menu #133

Closed VGirol closed 2 years ago

VGirol commented 2 years ago

I've got this error : Class ChrisWare\NovaBreadcrumbs\NovaBreadcrumbs contains 1 abstract method and must therefore be declared abstract or implement the remaining methods (Laravel\Nova\Tool::menu)

To Reproduce Steps to reproduce the behavior:

  1. Install last version (2.3.1)
  2. Follow the installation guide
  3. See error
chris-ware commented 2 years ago

You're trying to use it with nova 4 which isn't supported.

VGirol commented 2 years ago

Ah, ok. I thank you for your quick answer.

If Nova 4 isn't supported, maybe you should change the dependencies in the composer.json : replace "laravel/nova": ">=3" by "laravel/nova": "^3"

chris-ware commented 2 years ago

@VGirol That won't work, as if I were to release a new version with that constraint, composer would just install the version that it believes does support Nova 4. I'd have to delete every tagged version prior to the latest one.

VGirol commented 2 years ago

I did not think about that ... ;-) So, let's leave it as it is.