Skip to content

fix filtering#19

Merged
Many0nne merged 1 commit intomainfrom
fix/filtering
Mar 7, 2026
Merged

fix filtering#19
Many0nne merged 1 commit intomainfrom
fix/filtering

Conversation

@Many0nne
Copy link
Owner

@Many0nne Many0nne commented Mar 7, 2026

No description provided.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the list-endpoint filtering API by renaming filter suffixes and extending range filtering to support numeric fields, with corresponding updates to OpenAPI docs, README, and unit tests.

Changes:

  • Rename filtering conventions: _like_contains, _from/_to_gte/_lte.
  • Add numeric range filtering behavior (_gte/_lte) in the query processor.
  • Update Swagger parameter generation, README examples, and queryProcessor tests to match the new conventions.

Reviewed changes

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

Show a summary per file
File Description
src/core/queryProcessor.ts Renames filter buckets and implements _contains plus _gte/_lte logic (including numeric comparisons).
src/core/swagger.ts Updates generated query parameter names/descriptions and adds _gte/_lte for numeric fields.
tests/core/queryProcessor.test.ts Aligns tests with new filter names and adds coverage for numeric _gte/_lte.
README.md Updates documented filtering conventions and examples.
.mock-config.json Removes the tracked local config file.
review Removes an auto-generated/previous review artifact file.

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

@Many0nne Many0nne merged commit 17d4e31 into main Mar 7, 2026
6 checks passed
@Many0nne Many0nne deleted the fix/filtering branch March 7, 2026 19:30
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