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

config update for laravel 11 #324

Merged
merged 1 commit into from
Aug 2, 2024

Conversation

abhihyder
Copy link
Contributor

Update Event Listener Registration Documentation for Laravel 11


Description

This PR updates the event listener registration documentation to provide clarity on the changes in Laravel version 11 and later regarding automatic registration of event listeners. Specifically, the following updates have been made:

  • Added a detailed comment block explaining that event listeners located in the App\Listeners directory are automatically registered in Laravel 11 and later, eliminating the need for manual registration.
  • Included a note that if listeners are registered manually in this configuration file, they will be called twice.
  • Provided guidance for users who plan to store their listeners in different or multiple directories, specifying that they should register such listeners in this configuration file.
  • Added a note for users of Laravel version 10 or earlier, indicating that corresponding listeners must be registered manually in this configuration file.

Changes

  • Updated the comment block in the event listener configuration file with detailed explanations and instructions.

Documentation

For more details on event discovery in Laravel 11, please refer to the official documentation.

Testing

No code changes were made, so no testing is required. This PR only updates the documentation within the configuration file.

Copy link
Owner

@Kyon147 Kyon147 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great idea adding the additional context

@Kyon147 Kyon147 merged commit 9231d76 into Kyon147:master Aug 2, 2024
7 of 8 checks passed
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.

2 participants