Skip to content

fix: suppress $batch path for REST services - #194

Open
daogrady wants to merge 6 commits into
mainfrom
fix/rest-batch
Open

daogrady wants to merge 6 commits into
mainfrom
fix/rest-batch

Conversation

@daogrady

@daogrady daogrady commented Sep 23, 2026 •

Copy link
Copy Markdown
Contributor

Fixes #192

The field is specifc for odata and should therefore not be produced for other protocols (REST, specifically):
https://docs.oasis-open.org/odata/odata/v4.0/errata03/os/complete/part1-protocol/odata-v4.0-errata03-os-part1-protocol-complete.html#_Toc453752313

@daogrady daogrady changed the title fix: suppress $batch path for REST services fix: suppress $batch path for REST services Sep 23, 2026
@daogrady
daogrady requested a review from tim-sh September 29, 2026 07:23

@tim-sh tim-sh left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good to me.

I just found one gap: in REST-collection GETs we still have the OData query options $filter, $top, $skip, $select, $count, $search, $orderby, whereas CAP's REST adapter doesn't support them. So docs still show unsupported features (as with /$batch).

@daogrady

Copy link
Copy Markdown
Contributor Author

Fair observation. I adjusted the output accordingly.

This branch has not been deployed

No deployments
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.

[BUG] Compiler generates $batch request for REST service

2 participants