-
Notifications
You must be signed in to change notification settings - Fork 52
Typo fixes #4 #774
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?
Typo fixes #4 #774
Conversation
const client = new DuneClient(DUNE_API_KEY ?? ""); | ||
const queryID = 1215383; | ||
const params = { | ||
query_parameters = [ |
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.
don't think this needs to be changed
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.
this would be invalid javascript then.
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.
don't think this needs to be changed
icon="road-bridge" | ||
href="/api-reference/projects/endpoint/linea_lxp" | ||
> | ||
Get LXP balance for wallet, along with social and onchain behavior stats |
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.
keep
|
||
url := "https://api.dune.com/api/v1/query/{queryId}" | ||
|
||
payload := strings.NewReader("{\n \"query_id\": 1252207,\n ,\n \"query_sql\": \"{{blockchain}}.transactions WHERE to = {{address}} AND block_number > {{blocknumber}}\"}") |
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.
i think we can keep the \n and also add the SELECT * FROM
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.
The ,\n
makes this invalid json. But yes the query_sql should have select * from.
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.
might not be needed
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.
yeah. But it is prob fine to leave. Should be non-consequential and would take more effort to undo.
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
--- | ||
title: 'Private Query' | ||
openapi: 'POST /v1/query/{queryId}/private' | ||
openapi: 'GET /v1/query/{queryId}/private' |
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: Inconsistent HTTP Method for State Change
The OpenAPI specification for the private query endpoint is changed from POST to GET. Making a query private is a state-changing operation that should use POST. This change is inconsistent with other similar endpoints (archive, unarchive, unprivate) which are correctly defined as POST, and are even being updated to POST in this same diff. This could lead to incorrect API documentation or failed API calls.
"X-DUNE-API-KEY": "<x-dune-api-key>" | ||
} | ||
|
||
response = requests.request("POST", url, data=data, headers=headers) |
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.
keep as is
Fixes typos. See #770 for context