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.

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

"Try it out" for /assets-balances is incorrect in the docs. #847

Closed
TarikGul opened this issue Feb 15, 2022 · 1 comment
Closed

"Try it out" for /assets-balances is incorrect in the docs. #847

TarikGul opened this issue Feb 15, 2022 · 1 comment
Labels
I2 - Bug 🐜 Sidecar has a bug I5 - Documentation 📖 Documentation needs fixes or additions

Comments

@TarikGul
Copy link
Member

Specifically the query parameter section of the docs for /assets-balances is incorrect and should be fixed. I am not sure if the fix is obvious (depends on the swagger-ui support for express query params formatted by []).

I havent looked to deep into it yet, but it might be necessary to have PR #839 merged first.
@TarikGul TarikGul added I2 - Bug 🐜 Sidecar has a bug I5 - Documentation 📖 Documentation needs fixes or additions labels Jun 24, 2022
@TarikGul
Copy link
Member Author

This isn't supported in swagger, so there cant be a try it out feature for arrays. Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
I2 - Bug 🐜 Sidecar has a bug I5 - Documentation 📖 Documentation needs fixes or additions
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@TarikGul