Skip to content

Update several dependencies and plugins#525

Merged
mofterdinger merged 11 commits into
postgresfrom
pg_version_bump
Aug 7, 2025
Merged

Update several dependencies and plugins#525
mofterdinger merged 11 commits into
postgresfrom
pg_version_bump

Conversation

@mofterdinger

@mofterdinger mofterdinger commented Aug 6, 2025 •
edited
Loading

Copy link
Copy Markdown
Member

Update the following components:

  • SAP UI5 to 1.139.0
  • updated package-lock.json in root and app folder to latest available versions
  • several Maven build plugins to latest available versions

@mofterdinger
Bump SAP UI5 to 1.138.1
e388015
@mofterdinger
Bump cds-services to 4.2.0
588b294
@mofterdinger
update package-lock.json to latest versions
f0eaa8d
@mofterdinger
updated approuter package-lock.json
eaf75e5
@mofterdinger
update package-lock.json
6151364
@mofterdinger
update build plugins:
aa72d65 
maven-compiler-plugin ............................ 3.13.0 -> 3.14.0
maven-enforcer-plugin .............................. 3.5.0 -> 3.6.1
maven-surefire-plugin .............................. 3.5.2 -> 3.5.3
org.codehaus.mojo:flatten-maven-plugin ............. 1.6.0 -> 1.7.2
@mofterdinger mofterdinger changed the title Update several dependencies Update several dependencies and plugins Aug 6, 2025
agoerler
agoerler reviewed Aug 7, 2025
View reviewed changes

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

Do we also need to adjust this:

https://github.wdf.sap.corp/cds-java/cds-services/commit/d26d912c6b76a6bd12aeeffb774f4e40e2df00b9

?

@MattSchur

MattSchur commented Aug 7, 2025

Copy link
Copy Markdown
Contributor

We also have author.name in the header, this seems to cause the failing PATCH on Books(ID)/author

@mofterdinger

mofterdinger commented Aug 7, 2025

Copy link
Copy Markdown
Member Author

Hi, I added "editableHeaderContent": false to manifest.json, but without success. I'm still seeing getting the same error.

@mofterdinger

mofterdinger commented Aug 7, 2025 •
edited
Loading

Copy link
Copy Markdown
Member Author

After fixing another CDS syntax error in file app/common.cds it works as expected: dd5d451

@mofterdinger mofterdinger requested a review from agoerler August 7, 2025 12:59
@mofterdinger mofterdinger merged commit d625f80 into postgres Aug 7, 2025
2 checks passed
@mofterdinger mofterdinger deleted the pg_version_bump branch August 7, 2025 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@agoerler agoerler agoerler approved these changes
+1 more reviewer
@MattSchur MattSchur MattSchur approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@mofterdinger @MattSchur @agoerler