Skip to content

Make id filter work for combined endpoint#238

Closed
daanvdk wants to merge 1 commit into
masterfrom
combined-id-filter
Closed

Make id filter work for combined endpoint#238
daanvdk wants to merge 1 commit into
masterfrom
combined-id-filter

Conversation

@daanvdk

@daanvdk daanvdk commented May 22, 2024

Copy link
Copy Markdown
Contributor

No description provided.

@daanvdk daanvdk force-pushed the combined-id-filter branch 3 times, most recently from b0a0418 to 2a7737e Compare May 22, 2024 10:09
@daanvdk daanvdk force-pushed the combined-id-filter branch from 2a7737e to 5e0e08b Compare May 22, 2024 10:13
@daanvdk daanvdk marked this pull request as ready for review May 22, 2024 12:19
@knokko

knokko commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Combining IDs like that looks more like a work-around than a feature to me. Since this also hasn't received any attention for 2+ years, I'm closing it.

If this feature is ever needed in the feature, we could reopen this, or try to achieve it using where clauses.

@knokko knokko closed this Jul 3, 2026
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