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

Add findForCorrelationId method on TransferProcessStore #3062

Closed
ndr-brt opened this issue May 22, 2023 · 0 comments · Fixed by #3128
Closed

Add findForCorrelationId method on TransferProcessStore #3062

ndr-brt opened this issue May 22, 2023 · 0 comments · Fixed by #3128
Assignees
@ndr-brt
Labels
enhancement New feature or request refactoring Cleaning up code and dependencies
Milestone
Milestone 10

Comments

@ndr-brt
Copy link
Member

ndr-brt commented May 22, 2023

Feature Request

currently, to get a TransferProcess by its correlationId there's the need for a double call to processIdForDataRequestId and findById, and the first is never used without the second.

Which Areas Would Be Affected?

TransferProcessStore

Why Is the Feature Desired?

To align with ContractNegotiationStore

Solution Proposal

If possible, provide a (brief!) solution proposal.
@ndr-brt ndr-brt added enhancement New feature or request refactoring Cleaning up code and dependencies labels May 22, 2023
@ndr-brt ndr-brt added this to the Milestone 10 milestone May 22, 2023
@ndr-brt ndr-brt mentioned this issue May 31, 2023
Merged
@ndr-brt ndr-brt self-assigned this May 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ndr-brt ndr-brt

Labels
enhancement New feature or request refactoring Cleaning up code and dependencies
Projects
None yet
Milestone
Milestone 10
Development

Successfully merging a pull request may close this issue.

refactor: add findForCorrelationId method on TransferProcessStore Think-iT-Labs/edc-connector
1 participant
@ndr-brt