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 SwiftKanban connector #691

Merged
merged 3 commits into from
Feb 19, 2021
Merged

Add SwiftKanban connector #691

merged 3 commits into from
Feb 19, 2021

Conversation

mx012
Copy link
Contributor

@mx012 mx012 commented Feb 11, 2021


Please check the following conditions for your PR.

  • apiDefinition.swagger.json is validated using paconn validate command.
  • apiProperties.json has a valid brand color. Invalid brand colors are #007ee5 and #ffffff.

@mx012 mx012 requested a review from a team as a code owner February 11, 2021 18:46
@ghost
Copy link

ghost commented Feb 11, 2021

CLA assistant check
All CLA requirements met.

Copy link
Contributor

@dougelliott dougelliott left a 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!

@mx012 mx012 requested a review from dougelliott February 12, 2021 07:24
"/restapi/comment-operations/boards/{boardId}/cards/{cardType}:{cardId}/comments": {
"post": {
"responses": {
"default": {
Copy link
Contributor

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.

Copy link
Contributor Author

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!

@mx012 mx012 requested a review from sriyen-msft February 17, 2021 13:17
@mx012 mx012 changed the title Pull request for connector certification Add SwiftKanban connector Feb 17, 2021
@sriyen-msft sriyen-msft merged commit c6665bc into microsoft:dev Feb 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants