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

Merge from CatenaX fork #458

Merged
merged 30 commits into from
Mar 1, 2024
Merged

Merge from CatenaX fork #458

merged 30 commits into from
Mar 1, 2024

Conversation

ds-psosnowski
Copy link
Contributor

@ds-psosnowski ds-psosnowski commented Mar 1, 2024 •
edited by ds-jhartmann
Loading

Description

  • Added 'businessPartnerNumber' field to Tombstone model. This will be filled only when UsagePolicyValidation tombstone is being created.
  • Update to Spring Boot 3.1.9 to fix CVE's.
  • Fixed a case where Policy validation could result in a NullPointerException if either orConstraint or andConstraint of
    accepted policies were null

Pre-review checks

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

mkanal and others added 30 commits January 31, 2024 20:36
@mkanal
chore(concept):[#333] initial commit
8179ce1
@mkanal
chore(concept):[#333] add ds requirements
6c6827b
@mkanal
chore(concept):[#333] add links and references
ccf04ff
@mkanal
chore(concept):[#333] add assessment for data sov and add policy hub …
76ab311 
…connectivity concept
@ds-ext-kmassalski
feat(impl):[#404] add bpn to tombstone
4951f7d
@ds-ext-kmassalski
feat(impl):[#404] add bpn to tombstone
7a3e632
@ds-ext-kmassalski
feat(impl):[#404] simplify tombstone creation methods
4c0cb3b
@ds-ext-kmassalski
feat(impl):[#404] update changelog
c1fc5fa
@ds-ext-kmassalski
feat(impl):[#404] update changelog
ba1aa4d
@ds-ext-kmassalski
feat(impl):[#404] merge main
eaf5876
@mkanal
chore(concept):[#333] add description and policy hub flow
418b9c7
@mkanal
fix(concept):[#333] add header
5bbea7a
@mkanal
fix(concept):[#333] correct formating
634f247
@mkanal
fix(concept):[#333] fixed heading
ec22085
@mkanal
fix(concept):[#333] fixed heading of concept
fc8093d
@mkanal
chore(concept):[#333] rename from approach to strategy
f64da45
@ds-jhartmann
Merge pull request #792 from eclipse-tractusx/main
fa40007 
Sync with upstream
@mkanal
chore(concept):[#333] add constraint
1adaac7
@mkanal
fix(concept):[#333] fix nested list
8af3015
@mkanal
fix(concept):[#333] fix nested list and links
0bab172
@mkanal
fix(concept):[#333] fix typo
0401e60
@mkanal
fix(concept):[#333] fix link
5f4040d
@mkanal
fix(concept):[#333] fix toc
df1bdad
@mkanal
fix(concept):[#333] fixed nested list
da9ac67
@mkanal
fix(concept):[#333] refactore sentence
59f83f0
@mkanal
Merge pull request #791 from catenax-ng/chore/#333-data-sov-guardrail…
3ce7c39 
…s-presentation

Chore/#333 data sov guardrails presentation
@ds-ext-kmassalski
Merge pull request #790 from catenax-ng/feature/404-add-bpn-to-tombstone
6da77c9 
feat(impl):[#404] add bpn to tombstone
@ds-jhartmann
fix(irs-edc-client):[#436] fix nullpointer on missing and or or const…
72b0396 
…raint
@ds-jhartmann
Merge pull request #795 from catenax-ng/fix/#436-fix-policy-validatio…
c72452b 
…n-null-pointer

fix policy validation
@ds-jhartmann
Merge pull request #797 from eclipse-tractusx/main
54c01b7 
Sync with upstream
ds-jhartmann
ds-jhartmann approved these changes Mar 1, 2024
View reviewed changes
Copy link
Contributor

@ds-jhartmann ds-jhartmann 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-jhartmann ds-jhartmann merged commit 96077a3 into eclipse-tractusx:main Mar 1, 2024
16 of 18 checks passed
@ds-jhartmann ds-jhartmann mentioned this pull request Mar 1, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ds-jhartmann ds-jhartmann ds-jhartmann 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.
4 participants
@ds-psosnowski @ds-jhartmann @mkanal @ds-ext-kmassalski