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.

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: connector client: switch PolicyService #90

Merged
merged 4 commits into from
Sep 14, 2023

Conversation

janpmeyer
Copy link
Contributor

What this PR changes/adds

Use EDC-connector-client for `ContractDefinition

Why it does that

Briefly state why the change was necessary.

Further notes

List other areas of code that have changed but are not necessarily linked to the main feature. This could be method
signature changes, package declarations, bugs that were encountered and were fixed inline, etc.

Linked Issue(s)

Closes #58 and Closese #65

@janpmeyer
Copy link
Contributor Author

At the current state on creating a new Policy the values are not used given in the Dialog.

Also i have the question about the assignee and assigner of a policy. Does the still exist? Because i don't see this fields in the management-Api Documentation. Or i just did not see it? @ndr-brt

@janpmeyer janpmeyer requested a review from ndr-brt September 13, 2023 11:52
@janpmeyer janpmeyer self-assigned this Sep 13, 2023
@janpmeyer janpmeyer added the enhancement New feature or request label Sep 13, 2023
@ndr-brt
Copy link
Member

ndr-brt commented Sep 13, 2023

@janpmeyer they are part of the ODRL model, but they are not used by the EDC, so they can be avoided

@janpmeyer
Copy link
Contributor Author

@ndr-brt fixed now the id of an policyDefiniton and also the permission field of the policy. But the fields prohibition and obligation are not working yet

@janpmeyer
Copy link
Contributor Author

With the new EDC-Connector-Client Beta-6 obligations and prohibitions are also working.

@janpmeyer janpmeyer marked this pull request as ready for review September 14, 2023 07:13
@ndr-brt ndr-brt merged commit 489b3a0 into eclipse-edc:main Sep 14, 2023
2 checks passed
farhin23 pushed a commit to FraunhoferISST/edc-dashboard that referenced this pull request Mar 5, 2024
* feat: switch PolicyDefinition to connector client (eclipse-edc#38)

* feat: fix id and permission on creating new Policy (eclipse-edc#58)

* feat: use get functions for obligations and prohibition in View (eclipse-edc#58)

* feat: remove unnecessary import (eclipse-edc#58)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

connector client: switch PolicyService
2 participants