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 (Gate) - Resolve Finished Cleaning Task #540

Merged

Conversation

alexsilva-CGI
Copy link
Contributor

Description

In this PR, the BPDM Gate can poll the BPDM Orchestrator whether there is already a final cleaning result for the respective cleaning task. When a new cleaning result for a finished cleaning task has been returned by the Orchestrator the Gate then saves the cleaning result as a generic business partner output, resolves the task identifier and updates the sharing state of the respective business partner. Also, some Unit tests have been added.

Related to #430

Pre-review checks

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

Copy link
Contributor

@nicoprow nicoprow left a comment

Choose a reason for hiding this comment

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

Also, I saw in the commits that there has been a problem with the first commit reintroducing dummy values in the orchestrator. Best just rebase and squash the commits onto main

@alexsilva-CGI alexsilva-CGI force-pushed the feat/resolve_finished_cleaning_task branch from 21d1b6c to 7a14e72 Compare October 19, 2023 12:38
@alexsilva-CGI alexsilva-CGI force-pushed the feat/resolve_finished_cleaning_task branch from 7a14e72 to c6edfe5 Compare October 19, 2023 12:39
@nicoprow nicoprow merged commit 1c7a84c into eclipse-tractusx:main Oct 19, 2023
7 checks passed
@nicoprow nicoprow deleted the feat/resolve_finished_cleaning_task branch October 19, 2023 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants