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

goframe swagger generate not necessary summary info #2911

Closed
webignorant opened this issue Aug 29, 2023 · 2 comments
Closed

goframe swagger generate not necessary summary info #2911

webignorant opened this issue Aug 29, 2023 · 2 comments
Labels
done This issue is done, which may be release in next version. enhancement

Comments

@webignorant
Copy link

1. What version of Go and system type/arch are you using?

go version go1.20.4 windows/amd64

2. What version of GoFrame are you using?

v2.5.1

3. Can this issue be re-produced with the latest release?

yes

4. What did you do?

if same route has multiple method, then swagger.json will generate incorrect structure.

lool this picture
5acfea48c54c7f47f5bba6e9fcc7355

5. What did you expect to see?

one route, the children node only has http method.
you sure compared apifox swagger.json

look this picture
5261e65e3afe287784703b9c5a778cd

6. What did you see instead?

so use swagger-typescript-api tool,generated api module use Fetch Api or Axios to make requests.

if use goframe swagger.json, has following error occurred.
f948c92e1819fc461fe36195faf2526

beause the summary node is an illegal node
image

so use swagger-typescript-api, result is not expected
167daa54c5775db7708ecf16d43d516

@cjhyy
Copy link

cjhyy commented Sep 13, 2023

i meet the same problem, i find the gf cause this. So i do some change with the swagger node.js setting to solve it. drop the wrong route which method is summary.

@gqcn
Copy link
Member

gqcn commented Oct 9, 2023

@webignorant @cjhyy It was fixed in 4791581

@gqcn gqcn closed this as completed Oct 9, 2023
@gqcn gqcn added enhancement done This issue is done, which may be release in next version. labels Oct 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
done This issue is done, which may be release in next version. enhancement
Projects
None yet
Development

No branches or pull requests

3 participants