Skip to content

[java] mention limitation of @cds.search - #2942

Merged
renejeglinsky merged 3 commits into
mainfrom
java-deep-auth-cds-search
Oct 22, 2026
Merged

renejeglinsky merged 3 commits into
mainfrom
java-deep-auth-cds-search

Conversation

@vmikhailenko

Copy link
Copy Markdown
Contributor

Warn users that the @cds.search via associations should be used responsibly.

@hyperspace-pr-bot

Copy link
Copy Markdown
Contributor

👋 Hi — I'm PR Bot, your SAP code review assistant.

I'll automatically review your pull requests for code quality, security, and SAP compliance. Get an overview of what I do →

What I do

  • Summarize your pull request changes
  • Review code for quality, correctness, and reliability
  • Suggest fixes when a pipeline job fails

Key commands

Command Description
/review [--all] Trigger a code review. Add --all to include files excluded by excluded_paths.
/summarize Generate a PR summary
/ask <question> Ask about the current changes
/help See all available commands

*This introduction message will be shown to you only once, you will not see it in future PRs.

@hyperspace-pr-bot

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

Comment thread guides/security/authorization.md Resolved
Comment thread guides/security/authorization.md Resolved

@renejeglinsky renejeglinsky 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.

@vmikhailenko We can do the addition wrt sensitive data in another PR. I think it's important to have this note now for the release and ship it.

@renejeglinsky
renejeglinsky added this pull request to the merge queue Oct 22, 2026
Merged via the queue into main with commit 28e8f89 Oct 22, 2026
4 checks passed
@renejeglinsky
renejeglinsky deleted the java-deep-auth-cds-search branch October 22, 2026 12:18
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.

3 participants