-
Notifications
You must be signed in to change notification settings - Fork 0
Conversation
src/main/resources/openapi/specs/api.kuflow.com/v1/openapi.yaml
Outdated
Show resolved
Hide resolved
src/main/resources/openapi/specs/api.kuflow.com/v1/openapi.yaml
Outdated
Show resolved
Hide resolved
| default: | ||
| $ref: "#/components/responses/DefaultError" | ||
|
|
||
| /processes/{id}/~actions/save-user-action-value-document: |
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'm not sure, but maybe here we should use: /processes/{id}/~actions/save-user-action-document:
🤔
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 don't know, I left it like that for homogeneity with that of element...
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.
save-element-value-document:
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.
yap, but a user action can't have more than one value, isn't it?
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.
just thinking
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.
nop, only one value
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.
That's why I commented that 'value' is redundant, 🤔
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'm not sure, but maybe is better left as is, as you have it, to avoid confusion.
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 don't know about the naming, but now is the right moment to make any change because later is a new API version :S
No description provided.