orchestrator: replace businessKey #306
Conversation
Missing ChangesetsThe following package(s) are changed by this PR but do not have a changeset:
See CONTRIBUTING.md for more information about how to add changesets. Changed Packages
|
|
|
this is to be merged after we have an official OSL with the required changes (apache/incubator-kie-kogito-runtimes#3810.) |
| @@ -56,7 +57,6 @@ export const mapProcessInstanceToDetails = ( | |||
| category: instance.category, | |||
| state: instance.state, | |||
| description: instance.description, | |||
| businessKey: instance.businessKey, | |||
There was a problem hiding this comment.
Why isn't there assessmentInstanceId defined here? like for https://github.com/redhat-developer/rhdh-plugins/pull/306/files#diff-863d661b58ddfc134554cea4bf31465f4720f1c56accfc594368331bf13c52ccR145
Is it because the returned object was only used to send the request and as of know the buisnessKey is not given anymore but fetched directly from the DI so we do not need it?
| @@ -142,6 +142,7 @@ export function mapToProcessInstanceDTO( | |||
| duration: duration, | |||
| // @ts-ignore | |||
| workflowdata: variables?.workflowdata, | |||
| assessmentInstanceId: variables?.orchestratorAssessmentInstanceId as string, | |||
There was a problem hiding this comment.
where is assessmentInstanceId used in the front-end? I am asking as now the request will return this new variable instead of businessKey but I do not see any changes on the FE plugin
There was a problem hiding this comment.
this is why it is a draft PR. we need to wait for sonata to support it in official release and then someone will have to change the FE
christoph-jerolimov
changed the title
|
This PR has been automatically marked as stale because it has not had recent activity from the author. It will be closed if no further activity occurs. If the PR was closed and you want it re-opened, let us know and we'll re-open the PR so that you can continue the contribution! |
|
This PR has been automatically marked as stale because it has not had recent activity from the author. It will be closed if no further activity occurs. If the PR was closed and you want it re-opened, let us know and we'll re-open the PR so that you can continue the contribution! |
FLPATH-1926 https://issues.redhat.com/browse/FLPATH-1926 Signed-off-by: Yaron Dayagi <[email protected]>
|
ydayagi
requested review from
batzionb,
mareklibra and
gciavarrini
as code owners
|
This PR has been automatically marked as stale because it has not had recent activity from the author. It will be closed if no further activity occurs. If the PR was closed and you want it re-opened, let us know and we'll re-open the PR so that you can continue the contribution! |
FLPATH-1926
https://issues.redhat.com/browse/FLPATH-1926
Hey, I just made a Pull Request!
✔️ Checklist