Skip to content

docs(java): update fiori-drafts table for draftNew / post-active semantics#2516

Merged
agoerler merged 8 commits intomainfrom
java/fiori-drafts-post-active-semantics
Apr 23, 2026
Merged

docs(java): update fiori-drafts table for draftNew / post-active semantics#2516
agoerler merged 8 commits intomainfrom
java/fiori-drafts-post-active-semantics

Conversation

@rjayasinghe
Copy link
Copy Markdown
Contributor

Summary

  • Splits the single POST row in the "Editing Drafts" table into three rows reflecting the new branching behavior: explicit IsActiveEntity=false creates a draft, plain POST on entities without a draftNew action keeps the classic draft behavior, and plain POST on entities with a draftNew action now creates an active entity
  • Adds a new row for POST with action draftNew (the new way to create a draft in direct CRUD mode)
  • Updates the "POST Behavior with Direct CRUD" tip to reflect that cds.drafts.post-active is now auto-detected from @Common.DraftRoot.NewAction presence rather than requiring explicit configuration

Test plan

  • Review rendered table in the VitePress dev server
  • Verify all three POST scenarios are clearly distinguishable
  • Check the tip callout reads correctly alongside the "Bypassing the SAP Fiori Draft Flow" section

…ntics

Reflect the new POST behavior introduced when an entity has a
`@Common.DraftRoot.NewAction` (draftNew action): a plain POST now creates
an active entity instead of a draft, while `POST /draftNew` creates the
draft. Also document that `cds.drafts.post-active` is auto-detected from
the annotation presence rather than requiring explicit configuration.
@rjayasinghe rjayasinghe requested a review from smahati as a code owner April 20, 2026 10:28
@rjayasinghe rjayasinghe requested a review from t-bonk April 20, 2026 10:29
Copy link
Copy Markdown
Contributor

@t-bonk t-bonk left a comment

Choose a reason for hiding this comment

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

LGTM

@rjayasinghe
Copy link
Copy Markdown
Contributor Author

@renejeglinsky @smahati please have a look! :)

Comment thread java/fiori-drafts.md Outdated
Comment thread java/fiori-drafts.md Outdated
Comment thread java/fiori-drafts.md Outdated
Comment thread java/fiori-drafts.md Outdated
Co-authored-by: Adrian Görler <adrian.goerler@sap.com>
@smahati smahati requested a review from renejeglinsky as a code owner April 23, 2026 13:32
Copy link
Copy Markdown
Contributor

@agoerler agoerler left a comment

Choose a reason for hiding this comment

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

see my comments

Co-authored-by: Adrian Görler <adrian.goerler@sap.com>
Comment thread java/fiori-drafts.md Outdated
Co-authored-by: Adrian Görler <adrian.goerler@sap.com>
@agoerler agoerler enabled auto-merge (squash) April 23, 2026 14:26
@agoerler agoerler merged commit e3ba609 into main Apr 23, 2026
7 checks passed
@agoerler agoerler deleted the java/fiori-drafts-post-active-semantics branch April 23, 2026 14:38
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.

4 participants