Skip to content

277 Enclose Describe Results#278

Merged
simonoakesepimorphics merged 2 commits intomasterfrom
277-describe-filter
Jan 21, 2026
Merged

277 Enclose Describe Results#278
simonoakesepimorphics merged 2 commits intomasterfrom
277-describe-filter

Conversation

@simonoakesepimorphics
Copy link
Contributor

Fixes #277

Add API spec property (elda:encloseDescribe -> boolean) to determine whether to enclose describe query results. Disabled by default.

Copy link
Member

@der der left a comment

Choose a reason for hiding this comment

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

Thanks Simon code looks good but can't get it to work in practice. Deployed this branch as a snapshot, built an image from it with configs modified to add the elda:encloseDescribe true annotation to each api instance but having no effect.

Haven't yet managed to find a way to run it under a debugger to figure out what's going on.

Copy link
Member

@der der left a comment

Choose a reason for hiding this comment

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

Ignore last comment, trying again more carefully today it is working fine. Many thanks.

@simonoakesepimorphics simonoakesepimorphics merged commit 041f204 into master Jan 21, 2026
2 checks passed
@simonoakesepimorphics simonoakesepimorphics deleted the 277-describe-filter branch January 21, 2026 11:48
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.

Option to filter DESCRIBE query results

2 participants