Skip to content

feat(config-ui): support plugin azure devops#4504

Merged
likyh merged 6 commits intoapache:mainfrom
mintsweet:feat-3321
Mar 31, 2023
Merged

feat(config-ui): support plugin azure devops#4504
likyh merged 6 commits intoapache:mainfrom
mintsweet:feat-3321

Conversation

@mintsweet
Copy link
Copy Markdown
Member

@mintsweet mintsweet commented Feb 23, 2023

Summary

support plugin azure devops.

Does this close any open issues?

Closes #3321

Screenshots

screenshot-20230331-173745

screenshot-20230331-174014

screenshot-20230331-174024

screenshot-20230331-174048

screenshot-20230331-174054

screenshot-20230331-174104

screenshot-20230331-173730

@mintsweet mintsweet marked this pull request as draft February 23, 2023 06:21
@keon94 keon94 added this to the v0.17.0 milestone Mar 29, 2023
@keon94 keon94 added the component/config-ui This issue or PR relates to config-ui label Mar 29, 2023
@mintsweet mintsweet marked this pull request as ready for review March 31, 2023 09:45
@mintsweet mintsweet requested a review from likyh March 31, 2023 09:50
@likyh likyh merged commit 4c07a81 into apache:main Mar 31, 2023
if err != nil {
return &CallResult{
Err: errors.Default.Wrap(err, fmt.Sprintf("failed to invoke remote function \"%s\"", methodName)),
Err: errors.Default.Wrap(err, fmt.Sprintf("get error when invoking remote function %s", methodName)),
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

get -> got

Message: err.Error(),
}
return &plugin.ApiResourceOutput{Body: body, Status: 401}, nil
return &plugin.ApiResourceOutput{Body: body, Status: 500}, nil
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is true that the error can be a server error, but it is more regularly an authorization error from the tool API.

@mintsweet mintsweet deleted the feat-3321 branch March 31, 2023 11:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component/config-ui This issue or PR relates to config-ui

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature][Config UI] Config UI needs to support Azure DevOps

4 participants