Skip to content
This repository was archived by the owner on Mar 31, 2023. It is now read-only.

Conversation

@skdong
Copy link
Contributor

@skdong skdong commented May 8, 2020

List of Change

  • Add swagger dependencies
  • Move swagger config from test to src

NOTE

  • Need to execute microservices and access the UI of each microservices via /swagger-ui.html

2. mv swagger config from test to src
@skdong
Copy link
Contributor Author

skdong commented May 8, 2020

@xieus Dr. xie this branch is to Setup Swagger UI

@xieus
Copy link
Contributor

xieus commented May 8, 2020

Awesome! Does this support API gateway as it has the complete set of customer-facing APIs?

@xieus xieus added the UI Frontend UI and CLI label May 8, 2020
@xieus xieus added this to the Version 0.5.2020.05.30 milestone May 8, 2020
@xieus xieus requested a review from centaurus-cloud-bot May 8, 2020 06:34
@xieus xieus changed the title Set up a Swagger UI [Alcor Tools] Set up a Swagger UI May 8, 2020
@skdong
Copy link
Contributor Author

skdong commented May 9, 2020

API-Gateway's Swagger documentation is not supported at this time. The Spring Swagger generated documentation depends on some Spring annotations such as: RestController RequestMapping

@xieus
Copy link
Contributor

xieus commented May 9, 2020

Got it. Let us merge this PR first and continue to check other options.

@xieus xieus self-requested a review May 9, 2020 04:05
Copy link
Contributor

@xieus xieus left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks, @skdong

@xieus xieus merged commit dec9dd9 into futurewei-cloud:master May 9, 2020
xieus pushed a commit to xieus/alcor that referenced this pull request May 9, 2020
commit dec9dd9
Author: kaidong <shikaidong@126.com>
Date:   Sat May 9 12:06:38 2020 +0800

    [Alcor Tools] Set up a Swagger UI (futurewei-cloud#187)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

UI Frontend UI and CLI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants