Skip to content

Java - draft - delta and delete - #2956

Merged
renejeglinsky merged 7 commits into
mainfrom
Java---draft---delta-and-delete
Oct 23, 2026
Merged

renejeglinsky merged 7 commits into
mainfrom
Java---draft---delta-and-delete

Conversation

@agoerler

Copy link
Copy Markdown
Contributor

No description provided.

@hyperspace-pr-bot

hyperspace-pr-bot Bot commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

Control Panel

Hi, I'm an AI-powered Review Bot that helps you with summarizing and reviewing pull requests.
To interact with me, just use the following actions:

  • 📝 Summarize PR
  • 🔍 Review
  • 🗑️ Delete all bot comments and reviews

@smahati

smahati commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor
  • Fix build errors:
Found 2 link issue(s) in 1 source(s):
 
in: /java/fiori-drafts.md
  Not found: ./../working-with-cql/query-api#deep-update-delta (/java/fiori-drafts.md)
  Not found: ./../working-with-cql/query-api#deep-update-delta-processing (/java/fiori-drafts.md)

Comment thread java/fiori-drafts.md Outdated
MattSchur
MattSchur previously approved these changes Oct 21, 2026
Co-authored-by: Matthias Schur <107557548+MattSchur@users.noreply.github.com>
@agoerler

Copy link
Copy Markdown
Contributor Author
  • Fix build errors:
Found 2 link issue(s) in 1 source(s):
 
in: /java/fiori-drafts.md
  Not found: ./../working-with-cql/query-api#deep-update-delta (/java/fiori-drafts.md)
  Not found: ./../working-with-cql/query-api#deep-update-delta-processing (/java/fiori-drafts.md)

done

MattSchur
MattSchur previously approved these changes Oct 22, 2026
@smahati

smahati commented Oct 22, 2026

Copy link
Copy Markdown
Contributor

there's overlap between this PR and Robin's which was merged. The content is mostly compatible but there was a beta tag that isn't present here, should we keep it out?

@agoerler

Copy link
Copy Markdown
Contributor Author

I was not aware of Robin's PR. I'll merge main into this branch and push the merge. The beta flag is not necessary.

@agoerler

Copy link
Copy Markdown
Contributor Author

I was not aware of Robin's PR. I'll merge main into this branch and push the merge. The beta flag is not necessary.

done

hyperspace-pr-bot[bot]

This comment was marked as outdated.

@hyperspace-pr-bot hyperspace-pr-bot Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The new draft deletion and delta-save guidance is comprehensive and consistent with the configuration reference. I left two clarity suggestions; the existing review comments should also be addressed.

PR Bot Information

Version: 1.33.12

Comment thread java/fiori-drafts.md Outdated
Comment thread java/fiori-drafts.md Outdated
Co-authored-by: hyperspace-pr-bot[bot] <209611008+hyperspace-pr-bot[bot]@users.noreply.github.com>
@renejeglinsky
renejeglinsky merged commit cdeea24 into main Oct 23, 2026
4 checks passed
@renejeglinsky
renejeglinsky deleted the Java---draft---delta-and-delete branch October 23, 2026 09:30
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.

4 participants