Uploaded image for project: 'camunda BPM'
  1. camunda BPM
  2. CAM-3598

Add Tasklist plugin point for task card

    Details

    • Type: Feature Request
    • Status: Closed
    • Priority: L3 - Default
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 7.7.0, 7.7.0-alpha1
    • Component/s: tasklist
    • Labels:

      Description

      Add plugin point to extend the task card in the list of tasks (e.g. using custom renderers for variables).

      Make it possible to override the variables list in the tasklist card item.

        Issue Links

          Activity

          Hide
          meyer Daniel Meyer added a comment - - edited

          Note:

          the topic may be broader but it is good to keep this concrete usecase (task card extension) in mind:

          • in the process engine and rest api we have an extensible value type system
          • in the webapps (tasklist / cockpit) I currently cannot provide the counterpart to custom value types
          • It would be nice if users could define a renderer, editor (and potentially form field for task forms) for a custom value type
          • this would be a more "cross-cutting" plugin in the sense that the renderer and editors woud not show up in a single place but in different places

          In addition:

          • the task card plugin point may have a value of it's own (independently from the variables topic)
          Show
          meyer Daniel Meyer added a comment - - edited Note: the topic may be broader but it is good to keep this concrete usecase (task card extension) in mind: in the process engine and rest api we have an extensible value type system in the webapps (tasklist / cockpit) I currently cannot provide the counterpart to custom value types It would be nice if users could define a renderer, editor (and potentially form field for task forms) for a custom value type this would be a more "cross-cutting" plugin in the sense that the renderer and editors woud not show up in a single place but in different places In addition: the task card plugin point may have a value of it's own (independently from the variables topic)
          Hide
          sebastian.stamm Sebastian Stamm added a comment -

          I am not sure we need the $timeout here. Could we achieve the same with a $scope.$apply?

          The tooltip for the expansion arrow always reads "Show less variables", even when the variables section is collapsed. That might be related to the refactoring of the expanded property (was simple boolean before, is object now).

          We should add the new plugin point to the Tasklist documentation

          Show
          sebastian.stamm Sebastian Stamm added a comment - I am not sure we need the $timeout here . Could we achieve the same with a $scope.$apply ? The tooltip for the expansion arrow always reads "Show less variables", even when the variables section is collapsed. That might be related to the refactoring of the expanded property (was simple boolean before, is object now). We should add the new plugin point to the Tasklist documentation
          Hide
          valentin.vago Valentin Vago added a comment -

          I tried the $scope.$apply, but it doesn't worked for me and on the other hand, $timeout is also used to solve other issues in that script (https://github.com/camunda/camunda-bpm-webapp/blame/60464cda073eed3952962bd2e52cdab207fbf073/ui/tasklist/client/scripts/tasklist/directives/cam-tasklist-tasks.js#L200, I just hooked on the solution which was present).

          Documentation... right! Forgot that, will do.

          Show
          valentin.vago Valentin Vago added a comment - I tried the $scope.$apply, but it doesn't worked for me and on the other hand, $timeout is also used to solve other issues in that script ( https://github.com/camunda/camunda-bpm-webapp/blame/60464cda073eed3952962bd2e52cdab207fbf073/ui/tasklist/client/scripts/tasklist/directives/cam-tasklist-tasks.js#L200 , I just hooked on the solution which was present). Documentation... right! Forgot that, will do.
          Hide
          valentin.vago Valentin Vago added a comment -

          Sorry.. mixed up with an other CAM issue number for the docs...
          that would be the one: https://github.com/camunda/camunda-docs-manual/commit/293d0544fd8743fafc980babb87a69c12785650b

          Show
          valentin.vago Valentin Vago added a comment - Sorry.. mixed up with an other CAM issue number for the docs... that would be the one: https://github.com/camunda/camunda-docs-manual/commit/293d0544fd8743fafc980babb87a69c12785650b
          Hide
          valentin.vago Valentin Vago added a comment -

          Added the docs (butit was a bit early, see previous comment )

          Show
          valentin.vago Valentin Vago added a comment - Added the docs (butit was a bit early, see previous comment )
          Hide
          sebastian.stamm Sebastian Stamm added a comment -

          The following is still unaddressed:

          The tooltip for the expansion arrow always reads "Show less variables", even when the variables section is collapsed. That might be related to the refactoring of the expanded property (was simple boolean before, is object now).

          Show
          sebastian.stamm Sebastian Stamm added a comment - The following is still unaddressed: The tooltip for the expansion arrow always reads "Show less variables", even when the variables section is collapsed. That might be related to the refactoring of the expanded property (was simple boolean before, is object now).
          Hide
          valentin.vago Valentin Vago added a comment -
          Show
          valentin.vago Valentin Vago added a comment - Mea culpa.. mea maxi culpa... it's now on master... https://github.com/camunda/camunda-bpm-webapp/commit/7d8fc75bc2c5025c322f0b98c18094d4b5f07fe7
          Hide
          sebastian.stamm Sebastian Stamm added a comment -

          Looks fine. Due to the changed labels for the translation keys, we need to make sure to update the tasklist translation files as well as mention the change in the migration guide. I created a ticket for that: CAM-7260

          Show
          sebastian.stamm Sebastian Stamm added a comment - Looks fine. Due to the changed labels for the translation keys, we need to make sure to update the tasklist translation files as well as mention the change in the migration guide. I created a ticket for that: CAM-7260
          Hide
          valentin.vago Valentin Vago added a comment -

          Nice catch!

          Show
          valentin.vago Valentin Vago added a comment - Nice catch!

            People

            • Assignee:
              Unassigned
              Reporter:
              sebastian.stamm Sebastian Stamm
            • Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: