Skip to content
New issue

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

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

Already on GitHub? # to your account

fix(commandbus): disallow having two #[CommandHandler] for the same command #706

Merged

Conversation

Bapawe
Copy link
Contributor

@Bapawe Bapawe commented Nov 9, 2024

Fixes #683

@innocenzi innocenzi changed the title fix(commandbus): double command handlers fix(commandbus): disallow having two #[CommandHandler] for the same command Nov 9, 2024
@brendt brendt merged commit f3054f9 into tempestphp:main Nov 9, 2024
58 checks passed
@brendt
Copy link
Member

brendt commented Nov 9, 2024

Thank you! Appreciate it!

@Bapawe Bapawe deleted the fix/command-bus/double-command-handlers branch November 11, 2024 12:26
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Throw exception for two or more command handlers that handle the same command
2 participants