Comments (4)
chris-pardy
commented on May 20, 2024
1
@mauroservienti I think this approach makes sense. Giving some thought to our use case it feels like this idea works really well with CQRS so you should never have a case of needing to read data from 1 service before you dispatch a command to another service.
from all-our-aggregates-are-wrong-demos.
mauroservienti
commented on May 20, 2024
Hi @chris-pardy. Yeah you’re right, I was in hurry with the demo and that was the first solution that came to mind. CartIdAppender doesn’t sound like the solution I was looking for, ensuring that a Cart ID is probably someone else responsibility. Not to mention that it’s a technical requirement, I’ll give it more thoughts and as soon as I come up with a better design I’ll update this issue. Thanks.
from all-our-aggregates-are-wrong-demos.
mauroservienti
commented on May 20, 2024
@chris-pardy I raised #2 as a different approach to solve the problem. I haven't changed the logic, but given that now the CartId generation is completely disconnected from the ViewModel Composition process, it can be changed to whatever different requirement.
from all-our-aggregates-are-wrong-demos.
mauroservienti
commented on May 20, 2024
Agreed. I’ll merge the PR and close this one then, feel free to reopen or raise other issues. Thanks!
from all-our-aggregates-are-wrong-demos.
Related Issues (20)
- Stop using HTTP from IHandleRequests implementations HOT 3
- Show how to remove items from the cart
- Show how to create a new product to sell
- Move to generic hosting
- Update to .NET Core 3.1
- [QUESTION] How to implement an archiving service for order history? HOT 6
- How to handle change of cart item quantity? HOT 7
- Do the demo's service API projects require NSB endpoints? HOT 3
- Can't find an example with CleanupFailedCartRequest HOT 1
- Integration testing
- Update to new ServiceComposer API
- Question: If Shipping had a cost requirement which service owns that price? HOT 2
- Update to use the new NServiceBus.TransactionalSession
- README is outdate
- Move to PostgreSQL
- Move to RabbitMQ
- Use docker-compose to allow creating the required containers for RabbitMQ and PostgreSQL
- Move to Dev Containers to enable Codespaces support
- Craft some documentation that explains the role of each project
- add item to cart error handling IHandleRequestsErrors HOT 6
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from all-our-aggregates-are-wrong-demos.