search

antonioribeiro / health

Laravel Health Panel
BSD 3-Clause "New" or "Revised" License
1.94k stars 198 forks source link

Broken scheduler config #218

Closed truesteps closed 2 years ago

truesteps commented 3 years ago

Hey, it seems that the path for the scheduler config is wrong in the service provider in the registerTasks method. It should be config('health.notifications.scheduler.enabled') and ($frequency = config('health.notifications.scheduler.frequency')) &&.

As a hotfix, simply move the scheduler key out of the notifications wrapper part of the config, but that seems to break the whole app on Laravel 8. I just manually registered the health checks for now, but I think you should fix it :).

If you need more info/context, i'll be happy to help!

antonioribeiro commented 3 years ago

Should be fixed now.

truesteps commented 3 years ago

@antonioribeiro hey! sorry but please reopen the issue. When I configure the scheduler from the app config, despite it registering properly I get an exception saying "The class [cahce] does not exist".

antonioribeiro commented 3 years ago

@truesteps , what Laravel version?

truesteps commented 3 years ago

@antonioribeiro, I am on laravel 8 currently, more specifically v8.10.0