Skip to content

Add cql fields for 'type' and 'parent' - #30

Merged
kurtnordstrom merged 2 commits into
mainfrom
expand_cql_fields
Jul 1, 2026
Merged

Add cql fields for 'type' and 'parent'#30
kurtnordstrom merged 2 commits into
mainfrom
expand_cql_fields

Conversation

@kurtnordstrom

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds support for filtering /entries using CQL fields for type and parent, and extends the integration test suite to validate the new query behavior against fixture responses.

Changes:

  • Extend handleEntryCQL to support CQL fields type and parent.
  • Add integration test cases for GET /entries?q=parent=... and GET /entries?q=type=....
  • Add new API fixture responses for the above queries.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
api/entries.go Registers new CQL fields (type, parent) for entries query parsing.
test/entries_test.go Adds new integration test cases covering CQL queries by parent and type.
test/apifixtures/entries-cql-parent.get.res.json Expected response fixture for filtering entries by parent.
test/apifixtures/entries-cql-type.get.res.json Expected response fixture for filtering entries by type.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread api/entries.go
@kurtnordstrom
kurtnordstrom marked this pull request as ready for review July 1, 2026 20:02
@kurtnordstrom
kurtnordstrom merged commit 9def533 into main Jul 1, 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

Development

Successfully merging this pull request may close these issues.

2 participants