-
Notifications
You must be signed in to change notification settings - Fork 16
feat: add missing automatic layout params #88
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I apologize for the slow response. Adding these attributes looks good to me.
Actually, I took a look at the Java and the API definition and these two attributes are not part of the |
but i able to get it from the UI by inspecting the network tab
|
@simonbrowndotje can you comment on what the intention for |
API does not currently support the added attributes.
@WLun001 I guess this first needs to be a feature request on the Structurizr API then before we can make changes here on the Python SDK. |
@Midnighter how about this? already on the Java SDK |
The Python package only interacts with the web facing API. So we can only support what is possible through this API. |
@Midnighter I see. So we have to wait the web facing APIs to update |
I will close this as part of a clean up. |
New to python, please help me to review @Midnighter