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

Add PlantUML output option #9

Open
naddison36 opened this issue Nov 7, 2019 · 3 comments
Open

Add PlantUML output option #9

naddison36 opened this issue Nov 7, 2019 · 3 comments
Labels
enhancement New feature or request

Comments

@naddison36
Copy link
Owner

Here's an example PlantUML file https://gitlab.com/oax/asset-gateway-poc/blob/master/doc/dappsys.puml
Here's what it renders as https://miro.medium.com/max/3072/1*JsZItNqHas8XV4zWar7U1Q.png

@maurelian
Copy link

I would really like to have this in order to be able to manually edit, rearchitect, and change the level of detail in the generated image.

I looked into the code, and was surprised to see that at no point does it actually pass through the uml language. Where would be the right place to generate the uml? I'm thinking ./src/ts/converter.ts, in a similar manner to convertUmlClasses2Dot()?

@naddison36
Copy link
Owner Author

The code to convert the parsed Solidity classes to dot format is in the dotUmlClass function on the UmlClass class. In hindsight, I should have separated the generation of the dot format from the in-memory representation of the Solidity classes.
I'll give it a go this weekend.

@aspiers
Copy link

aspiers commented Jan 6, 2022

I would love to see this too. PlantUML output quality is a lot higher, even the added colour makes the result visually a lot easier to digest.

@naddison36 naddison36 added the enhancement New feature or request label Jul 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants
@aspiers @naddison36 @maurelian and others