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

When querying for open tasks, I want to have the option to fetch variables attached to the open tasks along with results #4487

Open
1 task
prakashpalanisamy opened this issue Jul 14, 2024 · 4 comments
Assignees
Labels
scope:core-api Changes to the core API: engine, dmn-engine, feel-engine, REST API, OpenAPI type:feature Issues that add a new user feature to the project.

Comments

@prakashpalanisamy
Copy link
Contributor

prakashpalanisamy commented Jul 14, 2024

User Story

As a developer, I need the ability to query for the list of tasks with the option to retrieve Task Local Variables and Task Variables as part of response by passing appropriate flags.

Functional Requirements

There can be appropriate flags added to the get-taskList post-taskList APIs' requests to enable the response to include the Task Variables as part of the response. This will save multiple API call to achieve the same outcome. The default behaviour can be set to NOT retrieve the variables as part of the response to not impact load/performance.

API Endpoints in Scope:

Links

Pull Requests

Preview Give feedback
  1. prakashpalanisamy
@prakashpalanisamy prakashpalanisamy added the type:feature Issues that add a new user feature to the project. label Jul 14, 2024
@prakashpalanisamy
Copy link
Contributor Author

prakashpalanisamy commented Jul 15, 2024

I'm willing to pick this up for implementation as well. @yanavasileva , Im tagging you so that you can take a look at the proposed Issue and provide your inputs. Thanks!!

@yanavasileva yanavasileva added the scope:core-api Changes to the core API: engine, dmn-engine, feel-engine, REST API, OpenAPI label Jul 16, 2024
@yanavasileva
Copy link
Member

Looks good. To be more explicit, you can add the exact REST endpoints in the ticket's description.

@prakashpalanisamy
Copy link
Contributor Author

prakashpalanisamy commented Jul 17, 2024

Looks good. To be more explicit, you can add the exact REST endpoints in the ticket's description.

Hi @yanavasileva.. Thank you for your quick response :) I have updated the Issue description to include the API endpoint in scope. Please review.

I need few inputs from you on the below items:

  1. I'm planning to introduce request parameters like withTaskVariablesInReturn and withTaskLocalVariablesInReturn
    - does the above both params make sense? or do you recommend to keep only LocalVariable in scope?
    - only on the parameters set to true explicitly on request, the users will be able to get response with variables
    - withTaskVariablesInReturn will take precedence over withTaskLocalVariablesInReturn, as the former covers the local variables as well.

Thank you!!

@yanavasileva
Copy link
Member

  • does the above both params make sense?

Yes, both make sense.

  • only on the parameters set to true explicitly on request, the users will be able to get response with variables
  • withTaskVariablesInReturn will take precedence over withTaskLocalVariablesInReturn, as the former covers the local variables as well.

Sounds good.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
scope:core-api Changes to the core API: engine, dmn-engine, feel-engine, REST API, OpenAPI type:feature Issues that add a new user feature to the project.
Projects
None yet
Development

No branches or pull requests

3 participants