Skip to content

Upgrade to support CDS 9#155

Merged
rishikunnath2747 merged 13 commits intodevelopfrom
cds9
Aug 13, 2025
Merged

Upgrade to support CDS 9#155
rishikunnath2747 merged 13 commits intodevelopfrom
cds9

Conversation

@rishikunnath2747
Copy link
Copy Markdown
Contributor

@rishikunnath2747 rishikunnath2747 commented Jul 31, 2025

Describe your changes

This PR contains changes in the plugin which are required to support @sap/cds ^9 dependency. It contains breaking changes for the plugin. It is also usable with cds 8 and xssec 3

Changes in application

  1. Change @sap/cds to ^9
  2. Changes @sap/cds-hana to @cap-js/hana 1.8.1 - because cds 9 requires cap-js/hana and
  3. Change @sap/xssec to ^4 - otherwise it throws error with cds 9
  4. Change @cap-js/sqlite to 2.0.2 - because older version internally it uses cds >=7.6 <9
  5. Change @sap/cds-dk to ^9

Changes in plugin

  1. Change @sap/xssec to ^4 as if we leave it as ^3 it still throws error

Integration tests for CDS 9

image

Integration tests for CDS 8

image

Type of change

  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Checklist before requesting a review

  • I have tested the functionality on my cloud environment.
  • I have provided sufficient automated/ unit tests for the code.
  • I have increased or maintained the test coverage.
  • I have ran integration tests on my cloud environment.
  • I have validated blackduck portal for any vulnerability after my commit.

Upload Screenshots/lists of the scenarios tested

  • I have Uploaded Screenshots or added lists of the scenarios tested in description

rashmiangadi05 and others added 5 commits July 4, 2025 13:26
one of the customer set  odata-containment to 'true' in package.json
"cds": {
"odata": {
"containment": true
}
It seems like req.data.ID is always empty using odata-containment.
We need to use req.params[0].ID
rashmiangadi11
rashmiangadi11 previously approved these changes Aug 11, 2025
@yashmeet29 yashmeet29 closed this Aug 11, 2025
@yashmeet29 yashmeet29 reopened this Aug 11, 2025
@rishikunnath2747 rishikunnath2747 merged commit 57b56c3 into develop Aug 13, 2025
9 checks passed
@rishikunnath2747 rishikunnath2747 deleted the cds9 branch August 13, 2025 07:36
Juergen-Goebl-msggroup pushed a commit to Andreas-Meindl-msggroup/sdm that referenced this pull request Apr 14, 2026
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