-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Optimize: add plantuml diagram #152
base: master
Are you sure you want to change the base?
Conversation
I think this UML diagram is a good addition, but I don't want it to replace the ER diagram, since the two types of diagram serve different purposes (software architecture for UML and database structure for ER). Can you put this image below the Trello board link in the readme and revert the removal of the ER diagram? |
Sounds good for me, I did it. |
What is the purpose of the diagram? Class diagram, context map or something in between? |
The diagram is a class diagram to show the architecture on domain implementation.
|
I’m not sure I follow you on the bounded contexts.Do you mean that cargo, voyage, handling and location are four different bounded contexts? To me it looks more like a class diagram of classes in the domain layer. |
I was meaning that that the cargo, voyage, handling and location are in the same bounded contexts. |
@mackapappa cloud you help to review it? |
Why
The existing diagram is created many years ago, and is lack of the package (boundary context) and properties.
So I added the UML diagram with plantuml.
pros:
What