Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Feature/536 fix handling of conflict response #480

Merged
merged 5 commits into from
Mar 11, 2024
Merged

Feature/536 fix handling of conflict response #480

merged 5 commits into from
Mar 11, 2024

Conversation

ds-ext-sceronik
Copy link
Contributor

Description

Changed

  • EdcPolicyDefinitionService, EdcContractDefinitionService and EdcAssetService return throw AlreadyExist exceptions when Conflict is returned from EDC

ds-mwesener
ds-mwesener reviewed Mar 11, 2024
View reviewed changes
...eclipse/tractusx/irs/edc/client/policy/model/exception/EdcPolicyDefinitionAlreadyExists.java Outdated
@@ -0,0 +1,31 @@
/********************************************************************************
* Copyright (c) 2022,2024 Bayerische Motoren Werke Aktiengesellschaft (BMW AG)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

License header

ds-mwesener
ds-mwesener reviewed Mar 11, 2024
View reviewed changes
...pse/tractusx/irs/edc/client/contract/model/exception/EdcContractDefinitionAlreadyExists.java Outdated
@@ -0,0 +1,31 @@
/********************************************************************************
* Copyright (c) 2022,2024 Bayerische Motoren Werke Aktiengesellschaft (BMW AG)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

License header

ds-mwesener
ds-mwesener reviewed Mar 11, 2024
View reviewed changes
...rg/eclipse/tractusx/irs/edc/client/asset/model/exception/EdcAssetAlreadyExistsException.java
@@ -0,0 +1,31 @@
/********************************************************************************
* Copyright (c) 2022,2024 Bayerische Motoren Werke Aktiengesellschaft (BMW AG)
* Copyright (c) 2021,2024 Contributors to the Eclipse Foundation
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

License header

ds-mwesener
ds-mwesener approved these changes Mar 11, 2024
View reviewed changes
Copy link
Contributor

@ds-mwesener ds-mwesener left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ds-mwesener ds-mwesener merged commit 6aa3de4 into eclipse-tractusx:main Mar 11, 2024
15 of 17 checks passed
@ds-mwesener ds-mwesener deleted the feature/536-fix-handling-of-conflict-response branch March 11, 2024 07:35
ds-jhartmann added a commit to ds-jhartmann/item-relationship-service that referenced this pull request Jun 13, 2024
@ds-jhartmann
Merge pull request eclipse-tractusx#480 from catenax-ng/chore/update-…
660352d 
…envs-to-3.3.3

chore(charts): Update DEV and INT to 3.3.3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ds-mwesener ds-mwesener ds-mwesener approved these changes

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
@ds-ext-sceronik @ds-mwesener @dsmf