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

Front Controller: Command Detection #52

Open
nibra opened this issue Oct 12, 2015 · 6 comments
Open

Front Controller: Command Detection #52

nibra opened this issue Oct 12, 2015 · 6 comments
Assignees
Milestone

Comments

@nibra
Copy link
Member

nibra commented Oct 12, 2015

All CRUD operations are (will be) covered by the core system. If a component provides business logic, this should go into (business) Models. We need a way to identify or declare the public methods (and their parameters) to wrap them into Commands.

@nibra nibra self-assigned this Oct 12, 2015
@nibra nibra added this to the 4.0 milestone Oct 12, 2015
@laoneo laoneo changed the title [priority1] Command bus Command bus Oct 12, 2015
@nibra nibra modified the milestones: CDH, 4.0 Jun 2, 2016
@laoneo
Copy link
Contributor

laoneo commented Jun 2, 2016

Ported from the service project https://github.com/chrisdavenport/service.

@laoneo laoneo closed this as completed Jun 2, 2016
@nibra
Copy link
Member Author

nibra commented Jun 3, 2016

It#s only partially done, since some issues with the detection of Command(Handler)s are not solved yet.

@nibra nibra reopened this Jun 3, 2016
@chrisdavenport
Copy link
Contributor

What issues have you come across?

@nibra
Copy link
Member Author

nibra commented Jun 3, 2016

I'm looking for a convenient way to autodetect commands (and their required parameters) from controllers/models, if their public methods are treated as command handlers. Haven't spent much time on this yet.

@laoneo
Copy link
Contributor

laoneo commented Jun 4, 2016

But then I would open a new issue which addresses exactly that problem. Having a generic issue which addresses all Command Bus problems is confusing and difficult to trace.

@nibra
Copy link
Member Author

nibra commented Jun 4, 2016

That's one of the next steps.

@nibra nibra removed the in progress label Jun 8, 2016
@nibra nibra changed the title Command bus Front Controller: Command Bus Jun 16, 2016
@nibra nibra modified the milestones: X.0, CDH Jun 29, 2016
@nibra nibra changed the title Front Controller: Command Bus Front Controller: Command Detection Oct 7, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants