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

Make CommandHandlers less specific #1284

Open
phavekes opened this issue Oct 31, 2024 · 0 comments
Open

Make CommandHandlers less specific #1284

phavekes opened this issue Oct 31, 2024 · 0 comments

Comments

@phavekes
Copy link
Member

This issue is imported from pivotal - Originaly created at Nov 28, 2018 by Michiel Kodde

The command handlers often perform very specific tasks on the domain. Sometimes command handlers invoke new commands on the command bus. This is considered bad practice and might prove problematic in the future.

Proposal:
Fix the command handlers an move much of their logic to services.
In the case where CommandHandlers invoked commands on the bus. Review the shared logic between the handlers that need to perform the same task on the domain. It should be possible to move that logic to a service and let the handlers call that instead.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: New
Development

No branches or pull requests

1 participant