Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

More documentation #5

Open
undjike opened this issue Jan 6, 2021 · 6 comments
Open

More documentation #5

undjike opened this issue Jan 6, 2021 · 6 comments

Comments

@undjike
Copy link

undjike commented Jan 6, 2021

Can you please provide better documentation and more examples for this package?

@therezor
Copy link
Owner

Nice catch. Will expand readme file

@undjike
Copy link
Author

undjike commented Oct 10, 2022

Hello, just wanted to know what changes should be applied in the app/Console/Kernel.php to be able to interact with this package?

@therezor
Copy link
Owner

therezor commented Oct 10, 2022

@undjike
Add command, it will override the default scheduler
protected $commands = [ TheRezor\DatabaseSchedule\Models\Schedule::class, ];

@WildernessStars
Copy link

WildernessStars commented May 29, 2024

Hello, i just added command to app/Console/Kernel.php
protected $commands = [ \TheRezor\DatabaseSchedule\Models\Schedule::class // \App\Console\Commands\InventorySync::class, ];
There is a TypeError:Illuminate\Console\Application::add(): Argument #1 ($command) must be of type Symfony\Component\Console\Command\Command, TheRezor\DatabaseSchedule\Models\Schedule given

is there anything i can do to deal with it? Thanks in advance.

@therezor
Copy link
Owner

Hi @WildernessStars
What laravel version are you suing?

@WildernessStars
Copy link

Hi @WildernessStars What laravel version are you suing?

8.x Thank you! I find everything goes well even if I don't add command to kernel.php.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants