Repository navigation
Conversation
Previous status code were not in agreement with the respective specification. This adjust HTTP status codes returned to align.
|
This needs to be adapted when #627 is merged. |
zrgt
left a comment
There was a problem hiding this comment.
The spec defines PostAllAssetLinksById as "Creates or replaces all asset links associated to the Asset Administration Shell" (spec). The endpoint currently merges the posted links into the existing ones, and test_post_is_additive asserts that, see #669.
Since this PR already aligns this endpoint with the spec, could it also replace the links instead of merging them, and flip test_post_is_additive?
|
Related to the requested change for #669: both handlers in this PR also maintain the reverse index ( |
Previous status code were not in agreement with the respective specification.
This change adjusts the HTTP status codes.
Fixes #625