Skip to content

Fix documentation generator #931

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.

Sign up for GitHub

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

Jump to bottom
Closed
delvedor opened this issue Aug 2, 2019 · 2 comments · Fixed by #969
Closed

Fix documentation generator #931

delvedor opened this issue Aug 2, 2019 · 2 comments · Fixed by #969
Assignees
@delvedor
Labels
good first issue

Comments

@delvedor
Copy link
Member

delvedor commented Aug 2, 2019

Some APIs support the same parameter both in the url and in the querystring. The current implementation of the documentation generator does not take into account this, and as result some APIs will have duplicated parameters in the description (see bulk or scroll for example).
This was referenced Aug 2, 2019
Closed
Closed
Closed
@alemagio
Copy link

Hi, i would like to work on this, any hint?

@delvedor
Copy link
Member Author

Hello @alemagio, sorry for the delay!
I'm in the process of updating the code generator to use the new spec adopted in the Elasticsearch repository, with that I'll probably fix also this issue as well.

@delvedor delvedor self-assigned this Sep 10, 2019
@delvedor delvedor mentioned this issue Sep 16, 2019
Merged
@delvedor delvedor mentioned this issue Oct 30, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@delvedor delvedor

Labels
good first issue
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update code generation elastic/elasticsearch-js
2 participants
@delvedor @alemagio