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

True Laravel 11 support #71

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

dorgan
Copy link

@dorgan dorgan commented Dec 23, 2024

Since kernel.php no longer exists we needed to get the artisan commands another way. Also it seems during setup that it tries to do some scheduling before the schedules table exists, so added some code for that.

Copy link

what-the-diff bot commented Dec 23, 2024

PR Summary

  • Enhancement to Permission Sets
    Introducing expanded command permissions within database-schedule.php. This enhancement gives a broader access control and functionality, including commands for installation, channel control, about, documentation, model management, storage unlinking, language publishing, and completion.

  • Preventive Check in Database Scheduler
    A checking mechanism has been incorporated in DatabaseSchedulingServiceProvider.php. This update confirms the existence of 'schedules' table before executing the schedule, thereby reducing potential errors or exceptions.

  • Upgraded Command Retrieval
    Updated CommandService.php replaces the use of Kernel with the Artisan facade for retrieving commands. This change enhances the process of fetch all commands, improving efficiency and the overall performance.

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

Successfully merging this pull request may close these issues.

1 participant