Skip to content

Conversation

va3093
Copy link
Contributor

@va3093 va3093 commented Aug 15, 2025

Fixes typos. See #770 for context

const client = new DuneClient(DUNE_API_KEY ?? "");
const queryID = 1215383;
const params = {
query_parameters = [
Copy link
Contributor

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

Copy link
Contributor Author

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.

Copy link
Contributor

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
Copy link
Contributor

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}}\"}")
Copy link
Contributor

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

Copy link
Contributor Author

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.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

might not be needed

Copy link
Contributor Author

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.

Copy link

@cursor cursor bot left a 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'
Copy link

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.

Fix in Cursor Fix in Web

"X-DUNE-API-KEY": "<x-dune-api-key>"
}

response = requests.request("POST", url, data=data, headers=headers)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

keep as is

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants