-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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 SwiftKanban connector #691
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.
Could this please be put into the certified-connectors
directory, rather than the custom-connectors
directory? Additionally, please include a readme.md
. You can take a look at other connectors located in the certified-connectors
directory for some examples.
Thanks!
"/restapi/comment-operations/boards/{boardId}/cards/{cardType}:{cardId}/comments": { | ||
"post": { | ||
"responses": { | ||
"default": { |
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.
@mx012 We would recommend you to replace ALL the "default" responses with expected status codes, otherwise users may see errors about incorrect schema instead of real error responses from your service in OpenAPI mode.
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 have added the status codes now. Please check. Thanks!
Please check the following conditions for your PR.
apiDefinition.swagger.json
is validated usingpaconn validate
command.apiProperties.json
has a valid brand color. Invalid brand colors are#007ee5
and#ffffff
.