Skip to content

feat(api): Allow to filter the API by app and object - #3348

Merged
nickvergessen merged 3 commits into
masterfrom
feat/noid/allow-filter-api-by-app-and-object
Sep 15, 2026
Merged

nickvergessen merged 3 commits into
masterfrom
feat/noid/allow-filter-api-by-app-and-object

Conversation

@nickvergessen

Copy link
Copy Markdown
Member

Usecases

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI

Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Joas Schilling <coding@schilljs.com>
@github-actions

github-actions Bot commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

🐢 Performance warning.
It looks like the query count of the integration tests increased with this PR.
Database query count is now 9158 was 9041 (+1.29%)
Please check your code again. If you added a new test this can be expected and the base value in tests/Integration/base-query-count.txt can be increased.

Assisted-by: ClaudeCode:claude-opus-5
Signed-off-by: Joas Schilling <coding@schilljs.com>
@github-actions

github-actions Bot commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

🐢 Performance warning.
It looks like the query count of the integration tests increased with this PR.
Database query count is now 9158 was 9041 (+1.29%)
Please check your code again. If you added a new test this can be expected and the base value in tests/Integration/base-query-count.txt can be increased.

@Antreesy Antreesy left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@nickvergessen
nickvergessen merged commit dd75d2a into master Sep 15, 2026
91 checks passed
@nickvergessen
nickvergessen deleted the feat/noid/allow-filter-api-by-app-and-object branch September 15, 2026 05:37
@nickvergessen

Copy link
Copy Markdown
Member Author

/backport to stable35

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants