Skip to content

feat: add ae swagger support#2451

Merged
mindlesscloud merged 2 commits into
apache:mainfrom
merico-ai:kw-2450-ae-swag
Jul 12, 2022
Merged

feat: add ae swagger support#2451
mindlesscloud merged 2 commits into
apache:mainfrom
merico-ai:kw-2450-ae-swag

Conversation

@klesh
Copy link
Copy Markdown
Contributor

@klesh klesh commented Jul 8, 2022

Summary

As title suggested

Does this close any open issues?

Closes #2450

Screenshots

image

Comment thread api/docs/stub.go
@keon94
Copy link
Copy Markdown
Contributor

keon94 commented Jul 11, 2022

There's also a mistake in the swagger docs here.
The parameter type should be models.NewPipeline, not string

@klesh
Copy link
Copy Markdown
Contributor Author

klesh commented Jul 12, 2022

There's also a mistake in the swagger docs here. The parameter type should be models.NewPipeline, not string

Nice catch, but it is out of the scope, would you file a bug for it?

@mindlesscloud mindlesscloud merged commit ca8ede7 into apache:main Jul 12, 2022
@warren830 warren830 deleted the kw-2450-ae-swag branch August 3, 2022 10:59
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.

[Doc][ae] swagger support for AE plugin

4 participants