search

mmajis / plantuml-serverless

Serverless PlantUML diagram rendering
GNU General Public License v3.0
48 stars 22 forks source link

Multiplicities #4

Closed andswitch closed 6 years ago

andswitch commented 6 years ago

Multiplicities are not being handled correctly.

Test case:

@startuml
Foo *-- "2" bar
@enduml

Result on https://plantuml.nitorio.us: 39396575-be98f3f8-4af0-11e8-9253-c93d27181566

Direct link to nitorio.us case: https://plantuml.nitorio.us/png/SoWkIImgAStDuNBBprVGqjLLK39IKd1ABE9oICrB0Oe30000

Result on http://plantuml.com/plantuml: 39396579-d182db14-4af0-11e8-8b17-54d9a882f177

Direct link to plantuml.com case: http://www.plantuml.com/plantuml/uml/SoWkIImgAStDuNBBprVGqjLLK39IKd1ABE9oICrB0Oe30000

mmajis commented 6 years ago

Yes, in fact any arrow captions on diagrams other than sequences are not showing up. Looks like I need to find a better way to compile graphviz dot statically with its dependencies...

mmajis commented 6 years ago

I’ll try to release a fixed version tomorrow.

mmajis commented 6 years ago

Ok @negativedensity, the fix is out in v0.0.5 available in the AWS Serverless Repository here: https://serverlessrepo.aws.amazon.com/applications/arn:aws:serverlessrepo:us-east-1:293246570391:applications~plantuml-render

Let me know if it works for you!

Thanks!

andswitch commented 6 years ago

Seems to work now! Tried a few examples from plantuml.com and no problems so far. :)