Skip to content

Query Archive command APP-7348#5

Merged
ivpusic merged 1 commit into
update_queryfrom
archive_query
Mar 3, 2026
Merged

Query Archive command APP-7348#5
ivpusic merged 1 commit into
update_queryfrom
archive_query

Conversation

@ivpusic
Copy link
Copy Markdown
Member

@ivpusic ivpusic commented Mar 2, 2026

dune query archive 4125432

Copy link
Copy Markdown
Member Author

ivpusic commented Mar 2, 2026

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Mar 2, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Comment @coderabbitai help to get the list of available commands and usage tips.

@ivpusic ivpusic changed the title cleanup Query Archive command APP-7348 Mar 3, 2026
@linear
Copy link
Copy Markdown

linear Bot commented Mar 3, 2026

APP-7348 Step 7: `dune query archive`

cmd/query/archive.go — positional arg: query ID. Calls api.UpdateQuery(id, {IsArchived: true}).

Output: Archived query 4125432

Acceptance criteria:

  • Sends PATCH with is_archived: true
  • Missing ID errors
  • API errors handled

Tests:

  • Correct PATCH body
  • Missing argument errors
  • 404 handled

@ivpusic ivpusic requested a review from va3093 March 3, 2026 08:45
@ivpusic ivpusic marked this pull request as ready for review March 3, 2026 08:45
@cursor
Copy link
Copy Markdown

cursor Bot commented Mar 3, 2026

PR Summary

Cursor Bugbot is generating a summary for commit 79cffaf. Configure here.

@ivpusic ivpusic merged commit b865b14 into update_query Mar 3, 2026
2 checks passed
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