-
Notifications
You must be signed in to change notification settings - Fork 52
[Automated] Update OpenAPI specifications #763
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
base: main
Are you sure you want to change the base?
Conversation
1eea9a7
to
55e1dde
Compare
55e1dde
to
12ec216
Compare
12ec216
to
7d663ce
Compare
7d663ce
to
4533e74
Compare
4533e74
to
716a16c
Compare
716a16c
to
71ba19c
Compare
71ba19c
to
6e52943
Compare
6e52943
to
b6cba49
Compare
b6cba49
to
53b7b78
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Comment @cursor review
or bugbot run
to trigger another review on this PR
"schema": { | ||
"type": "integer" | ||
} | ||
}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bug: Custom Endpoint QueryID Confusion
The /v1/endpoints/{handle}/{endpoint_slug}/results
endpoint incorrectly includes queryID
as a required parameter. Custom endpoints should derive their associated query from the handle
and endpoint_slug
path parameters, making a separate queryID
logically inconsistent for this endpoint.
This is an automated pull request to update the Dune API OpenAPI files.