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.
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
feat: query policydefinition, contractdefinition via privateProperties #3691
feat: query policydefinition, contractdefinition via privateProperties #3691
Conversation
…ition_via_properties
…ition_via_properties
…ition_via_properties' into feat/search_policy_contractdefinition_via_properties
Codecov ReportAttention:
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #3691 +/- ##
==========================================
+ Coverage 71.74% 71.75% +0.01%
==========================================
Files 919 921 +2
Lines 18457 18477 +20
Branches 1037 1039 +2
==========================================
+ Hits 13242 13259 +17
- Misses 4756 4757 +1
- Partials 459 461 +2 ☔ View full report in Codecov by Sentry. |
…ition_via_properties
…ition_via_properties' into feat/search_policy_contractdefinition_via_properties
can label be assigned to this PR?. Apart from that PR is ready. Unit test, endtoend tests are there for query. |
…ition_via_properties' into feat/search_policy_contractdefinition_via_properties
...aggregate-services/src/main/java/org/eclipse/edc/connector/service/query/QueryValidator.java
Outdated
Show resolved
Hide resolved
core/control-plane/control-plane-aggregate-services/build.gradle.kts
Outdated
Show resolved
Hide resolved
...java/org/eclipse/edc/connector/service/policydefinition/PolicyDefinitionServiceImplTest.java
Show resolved
Hide resolved
...java/org/eclipse/edc/connector/service/policydefinition/PolicyDefinitionServiceImplTest.java
Show resolved
Hide resolved
...aggregate-services/src/main/java/org/eclipse/edc/connector/service/query/QueryValidator.java
Show resolved
Hide resolved
…ition_via_properties
core/control-plane/control-plane-aggregate-services/build.gradle.kts
Outdated
Show resolved
Hide resolved
...aggregate-services/src/main/java/org/eclipse/edc/connector/service/query/QueryValidator.java
Show resolved
Hide resolved
...er/src/test/java/org/eclipse/edc/test/e2e/managementapi/PolicyDefinitionApiEndToEndTest.java
Show resolved
Hide resolved
@jimmarino Please check it |
…ition_via_properties
Check what? If you want me to re-review the PR, please resolve the open issues and re-request the review. |
I already resolved last 4 review comments you have given. @ndr-brt approved PR. If all ok for you, please approve. |
Please resolve the outstanding conversations and I will take another look. |
resolved. |
What this PR changes/adds
Search policydefinition, contractdefinition via privateProperties
Why it does that
PrivateProperties are stored in policydefinition, contractdefinition as json values. It is necessary to support search/query function via these stored json values.
Linked Issue(s)
Closes #3623