-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Introduce taskState
to runtime and history task API
#4046
Comments
I am working on this feature but unable to assign the issue to myself. Could someone look into it. Thanks |
Hi @jyotisahu9, Thank you for raising this feature request.
That's great, please don't forget to link the ticket to your pull request.
The tickets are being process by the team, no need for an action here. Best, |
Hi @yanavasileva , I have updated the feature details. Could you please review the same. Thanks, |
lifecycleState
in the response of GET task
Hi @jyotisahu9, Please note that the In order to query for user operation log entities you can use the following REST API: I still am not sure what do you want to achieve? Once you have the Best, |
lifecycleState
in the response of GET task
@yanavasileva I and Jyoti work for the same team, the value of this is, we are building a custom tasklist screen where state of task needs to be displayed to our business operations associates and currently we did not had easy way to retrieve state other than user operation logs entries at the same time we dont wanna make an additional API call from our custom tasklist UI which would be a problem for performance. Hence we wanna modify existing GET /task/{id} API to return state which we already invoke from our tasklist. I believe there are few discussion from others who require this state in the forum. Please see a screenshot of our custom tasklist UI where we need this |
I am proceeding with changes to persist lifeCycle status in new column TASK_STATE_ in ACT_RU_TASK table. Will do required changes in Task entity classes & Task xml files. All the task related API's like create, complete, delete that updates the lifecycle status will require code changes to persist taskStatus in new column. Also Get task api will have required changes to return task State as response parameter. Please do let me know if you have any review comments. Thanks, |
Hi @jyotisahu9 and @HarishMalavade, I am sorry for the late reply. Thank you for the provided information. Let's iterate once more to be sure what specifics do you require in this feature request. It's better to ensure the expectations and design are approved before doing the implementation. Otherwise a contribution can be rejected. You query the runtime task table (
Do you query completed tasks, or with other words historic data, to populate your task list? The listed possible output (Assign, Claim, ...) in the ticket description refers to actions that can be done on a task and not so much of a state. Best regards, |
Thanks for your response. So for the scope of this work we are thinking we will add new column both in ACT_RU_TASK and ACT_HI_TASK table to persist lifecycle state(Init, created, completed, deleted, updated) This will help us avoid computing these status values based on other parameters and simplify our tasklist UI so we dont have to query Operational log. We feel this also should be reusable by other community members that are building tasklist in C7 and we are excited and looking forward to make the code changes and contribute back to community. |
Hi @HarishMalavade, Thank you for your input. I guess you imply that a new field will be created for the lifecycle state. After briefly looking at the code, reuse of the Also if you need to track completed and/or deleted state, the information can be populated only in the historic table.
I see the potential there as well. Feel free to proceed with the contribution, I will try to support you on the way. Best, |
I am closing the ticket due to inactivity. |
Hi @yanavasileva , I am working on this issue. I have already done code changes, swagger docs changes and just left with test case changes. I will be creating PR to review soon. Thanks, |
Hi @yanavasileva , I have created a PR #4218 for this change . Requesting your review. Thanks, |
taskState
to runtime and history task API
Changes are merged and will be available to test with the next alpha release: |
Dev2QA handover.Feature report test
After each operation check via REST API what's the
Possible output: Documentation |
User Story (Required on creation)
As a Operations engineer, I can keep track of the task state via Java or REST API.
Functional Requirements (Required before implementation)
taskState
field is exposed to the runtime and history task APITechnical Requirements (Required before implementation)
taskState
column to the runtime and historic task tableslifecycleState
and reuse it as much as possibleLimitations of Scope
Hints
Goal for the request
Update Get Task API to add lifecycleState as output
Current API that retreives a task by id : /task/{id}
Current Response:
Proposed solution
Add taskState as response parameter
Current api returns details from ACT_HI_TASKINST table for a specific task id.
Task state is stored in ACT_HI_OP_LOG table. We will have to join ACT_HI_TASKINST with ACT_HI_OP_LOG table for ENTITY_TYPE_ = 'Task' to get task state.
Possible output:
Init
Created
Completed
Deleted
Updated
Alternatives considered
Create new api that returns taskState as response parameter.
In that case we will have to make 2 api calls, one is get api call to get task details
& other one is new api call to retrieve the task lifecycleState.
Links
Breakdown
Pull Requests
Dev2QA handover
The text was updated successfully, but these errors were encountered: