Skip to content
This repository has been archived by the owner on Apr 14, 2023. It is now read-only.

Set Aggregate in Command (simplify dispatcher) #51

Closed
lucabriguglia opened this issue Mar 28, 2019 · 4 comments
Closed

Set Aggregate in Command (simplify dispatcher) #51

lucabriguglia opened this issue Mar 28, 2019 · 4 comments

Comments

@lucabriguglia
Copy link
Owner

Add IDomainCommand as suggested by @andreujuanc

@andreujuanc
Copy link
Contributor

Is this for v7? :)

@lucabriguglia
Copy link
Owner Author

Well, it may be. I'm playing around a little bit and it looks like it would need a lot of refactoring at every level.

@lucabriguglia
Copy link
Owner Author

lucabriguglia commented Mar 28, 2019

And also, I don't see a way of achieving it without using reflection and therefore affecting performance.

@lucabriguglia lucabriguglia changed the title Set Aggregate in Command Set Aggregate in Command (simply dispatcher) Oct 15, 2019
@lucabriguglia lucabriguglia added this to the Kledex 1.3 milestone Oct 15, 2019
@lucabriguglia lucabriguglia changed the title Set Aggregate in Command (simply dispatcher) Set Aggregate in Command (simplify dispatcher) Oct 16, 2019
@lucabriguglia lucabriguglia modified the milestones: Kledex 1.3, Kledex 2.0 Oct 23, 2019
lucabriguglia pushed a commit that referenced this issue Oct 29, 2019
@lucabriguglia
Copy link
Owner Author

Merged to branch release/2.0

lucabriguglia pushed a commit that referenced this issue Nov 3, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants