[GitHub] [apisix-dashboard] Jaycean commented on pull request #1245: feat(BE): Export route from OpenAPI Specification3.0

2021-01-26 Thread GitBox
Jaycean commented on pull request #1245: URL: https://github.com/apache/apisix-dashboard/pull/1245#issuecomment-767936462 @starsz @nic-chen PTAL. Thks. This is an automated message from the Apache Git Service. To respond

[GitHub] [apisix-dashboard] Jaycean commented on pull request #1245: feat(BE): Export route from OpenAPI Specification3.0

2021-01-25 Thread GitBox
Jaycean commented on pull request #1245: URL: https://github.com/apache/apisix-dashboard/pull/1245#issuecomment-766539166 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [apisix-dashboard] Jaycean commented on pull request #1245: feat(BE): Export route from OpenAPI Specification3.0

2021-01-24 Thread GitBox
Jaycean commented on pull request #1245: URL: https://github.com/apache/apisix-dashboard/pull/1245#issuecomment-766610459 In my opinion, from the perspective of apimix platform, I choose scheme a to satisfy the users of apimix platform. The function that URI can be renamed is more open

[GitHub] [apisix-dashboard] Jaycean commented on pull request #1245: feat(BE): Export route from OpenAPI Specification3.0

2021-01-24 Thread GitBox
Jaycean commented on pull request #1245: URL: https://github.com/apache/apisix-dashboard/pull/1245#issuecomment-766539166 @liuxiran @nic-chen The unit test and test cases have been supplemented. In the process of testing, I found a problem: - At present, the path data structure of

[GitHub] [apisix-dashboard] Jaycean commented on pull request #1245: feat(BE): Export route from OpenAPI Specification3.0

2021-01-22 Thread GitBox
Jaycean commented on pull request #1245: URL: https://github.com/apache/apisix-dashboard/pull/1245#issuecomment-765474916 > > > > > > Hi, @Jaycean. Need unit test. > > > > > > > > > > > > > > > At present, the function interface needs to export the existing route data and ensure

[GitHub] [apisix-dashboard] Jaycean commented on pull request #1245: feat(BE): Export route from OpenAPI Specification3.0

2021-01-21 Thread GitBox
Jaycean commented on pull request #1245: URL: https://github.com/apache/apisix-dashboard/pull/1245#issuecomment-764447946 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [apisix-dashboard] Jaycean commented on pull request #1245: feat(BE): Export route from OpenAPI Specification3.0

2021-01-21 Thread GitBox
Jaycean commented on pull request #1245: URL: https://github.com/apache/apisix-dashboard/pull/1245#issuecomment-765077174 > > > > Hi, @Jaycean. Need unit test. > > > > > > > > > At present, the function interface needs to export the existing route data and ensure the correctness

[GitHub] [apisix-dashboard] Jaycean commented on pull request #1245: feat(BE): Export route from OpenAPI Specification3.0

2021-01-21 Thread GitBox
Jaycean commented on pull request #1245: URL: https://github.com/apache/apisix-dashboard/pull/1245#issuecomment-765076940 > > > Hi, @Jaycean. Need unit test. > > > > > > At present, the function interface needs to export the existing route data and ensure the correctness of the

[GitHub] [apisix-dashboard] Jaycean commented on pull request #1245: feat(BE): Export route from OpenAPI Specification3.0

2021-01-20 Thread GitBox
Jaycean commented on pull request #1245: URL: https://github.com/apache/apisix-dashboard/pull/1245#issuecomment-764447946 > it would be better to add three more test cases: > > * export a route created with upstream id > * export a route created with service which is created with

[GitHub] [apisix-dashboard] Jaycean commented on pull request #1245: feat(BE): Export route from OpenAPI Specification3.0

2021-01-10 Thread GitBox
Jaycean commented on pull request #1245: URL: https://github.com/apache/apisix-dashboard/pull/1245#issuecomment-757631777 > > > Hi, @Jaycean. Need unit test. > > > > > > At present, the function interface needs to export the existing route data and ensure the correctness of the

[GitHub] [apisix-dashboard] Jaycean commented on pull request #1245: feat(BE): Export route from OpenAPI Specification3.0

2021-01-10 Thread GitBox
Jaycean commented on pull request #1245: URL: https://github.com/apache/apisix-dashboard/pull/1245#issuecomment-757598022 > Hi, @Jaycean. Need unit test. At present, the function interface needs to export the existing route data and ensure the correctness of the exported data, so I