CX-128 DCM update to incorporate changes from standard update

[tobzahn]

Description

This PR updates the DCM (CX-128) KIT to be reflect the changes from version 2.3 of the standard.
More information on how to handle specific n-tier scenarios is included in the KIT with this change.

Pre-review checks

Please ensure to do as many of the following checks as possible, before asking for committer review:

• DEPENDENCIES are up-to-date. Dash license tool. Committers can open IP issues for restricted libs.
• Copyright and license header are present on all affected files

[lanfrans]

OK for me

[Frank-Globisch]

@tobzahn thanks for your invitation, for whatever reason I still cannot add myself as reviewer... not sure if it is a local issue that will be resolved after the week-end reboot (though I doubt it) ...

[MWADCM]

Basically I agree. Nevertheless we have to take care for dependencies to Standard of BPDM CX-0010 & CX-076 since n-tier Standard will be affected by BPDM Process vv. Also cosidering "managed by concept"

[MWADCM]

Basically I agree. Nevertheless we have to take care for dependencies to Standard of BPDM CX-0010 & CX-076 since n-tier Standard will be affected by BPDM Process vv. Also cosidering "managed by concept"

[reginePrivat]

looks good to me

[stephanbcbauer]

@tobzahn just because i am curious, :) Did CoPilot help? So are these found issues "real" issues/fixes? Or does it just confuse everybody?

I think it was helpful. Good idea to use this. Copilot suggested several corrections for typos and it is very convenient that you can just accept them. I am still impressed that it also suggested to use other words or even terminology changes like the leftOperand / left operand comment. Copilot was absolutely correct that this is misleading. So yes, this is helpful.

By the way: you asked to increase the version number, but I am not sure how and where KIT versions are controlled. I would likely increase the minor version.

Cool thx for the feedback -> you should adapt the changelog.md in your Kit. increase the version and describe what has been changed. This is for documentation.

And technical (filter in the UI, nice view) you have to adapt data/kitData.jsyou will find the DCM somewhere and there you can adapt the Version. -> description about this file

And if you are really motivated, here is another issue -> some broken links in your KIT

[tobzahn]

you should adapt the changelog.md in your Kit. increase the version and describe what has been changed. This is for documentation.

Done with [cde198b].

And technical (filter in the UI, nice view) you have to adapt data/kitData.jsyou will find the DCM somewhere and there you can adapt the Version. -> description about this file

Done with [c2901ce].

[matbmoser]

just some questions still

[matbmoser]

has someone already checked if this looks good after compiled?

[LukasSchmetz]

@matbmoser @stephanbcbauer @Grand-Thibault Anything missing after the fixes @tobzahn did?

[matbmoser]

Has someone updated the changelog here? It looks for me that is not.
And also make sure you are synced with main, because the changes were done a long time ago. Maybe it syncs automatically when merging to main.

I dismiss my own review

[matbmoser]

So @stephanbcbauer you can approve it if you want, was not really involved on the review here. at least removed the blocker.

[stephanbcbauer]

LGTM