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

I get a notification when the currently selected task is deleted

    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

      AT:

      • In Tasklist, when viewing a tasks details
      • If the task is deleted (or re-assigned) while its details view is open, I get a notification that informs me that the task does not exist anymore
      • Notification contains a way to clear the current task detail view

      Hint:

      • We would need to perform additional requests to observe the task, similar to the auto-refresh feature for the list of tasks

        Issue Links

          Activity

          Hide
          valentin.vago Valentin Vago added a comment - - edited

          Wouldn't it be better to have the task details "frozen" or even better IMVHO simply hidden rather than allowing the user to close?
          It might be surprising for the user and appear to be a bit radical / brutal, sure, but in the end and whatever, the user has to pick a new task to complete...
          Selecting the next task automatically would be to much though.

          Show
          valentin.vago Valentin Vago added a comment - - edited Wouldn't it be better to have the task details "frozen" or even better IMVHO simply hidden rather than allowing the user to close? It might be surprising for the user and appear to be a bit radical / brutal, sure, but in the end and whatever, the user has to pick a new task to complete... Selecting the next task automatically would be to much though.
          Hide
          valentin.vago Valentin Vago added a comment -

          I added the re-assignement to the description.
          Shouldn't we use an other wording than "deleted" in the status message?
          Something like "not available" maybe...

          Show
          valentin.vago Valentin Vago added a comment - I added the re-assignement to the description. Shouldn't we use an other wording than "deleted" in the status message? Something like "not available" maybe...
          Hide
          sebastian.stamm Sebastian Stamm added a comment - - edited

          The refresh interval should be the same as for the list of tasks with auto-refresh enabled (10 seconds). Actually, the two requests for the list of tasks and for the single task should be in sync with each other, otherwise a task may be removed from the list but is still enabled in the detail view or vice-versa. A (potentially) naive idea would be to use the system time to synchronize the two intervals, e.g. doing a request whenever the system clock displays xx:xx:x0.

          The message to close the task should be more descriptive, e.g. "Close Task", instead of just "Dismiss" as dismiss can be misinterpret as "Dismiss this notification only".

          I am not sure our standard notification system is the best choice for displaying the notification. Right now we run into the following problem:

          • open task detail view-utils
          • open process start process modal
          • task is completed
          • notification is displayed in the process start modal
            (same with edit filter and probably also other modals)

          Maybe we need a dedicated place in the task detail view itself to display the notification.

          There is a lot of whitespace in the task file. Maybe we should have a comment instead to mark the begin and end of the task observation logic.

          We should also disable the Complete button of the task form when we notice that the task does not exist anymore

          Show
          sebastian.stamm Sebastian Stamm added a comment - - edited The refresh interval should be the same as for the list of tasks with auto-refresh enabled (10 seconds). Actually, the two requests for the list of tasks and for the single task should be in sync with each other, otherwise a task may be removed from the list but is still enabled in the detail view or vice-versa. A (potentially) naive idea would be to use the system time to synchronize the two intervals, e.g. doing a request whenever the system clock displays xx:xx:x0. The message to close the task should be more descriptive, e.g. "Close Task", instead of just "Dismiss" as dismiss can be misinterpret as "Dismiss this notification only". I am not sure our standard notification system is the best choice for displaying the notification. Right now we run into the following problem: open task detail view-utils open process start process modal task is completed notification is displayed in the process start modal (same with edit filter and probably also other modals) Maybe we need a dedicated place in the task detail view itself to display the notification. There is a lot of whitespace in the task file . Maybe we should have a comment instead to mark the begin and end of the task observation logic. We should also disable the Complete button of the task form when we notice that the task does not exist anymore
          Hide
          sebastian.stamm Sebastian Stamm added a comment -

          The refresh of the list of tasks always happens now, even if when auto-refresh option is not checked.
          If possible, we could give the user some visual feedback that the task detail tab cannot be changed anymore (e.g. with a cursor: not-allowed style).
          I am not super happy with a test, that always waits 10 seconds, but I am not sure how to test it otherwise. I would be fine with removing the test.

          Show
          sebastian.stamm Sebastian Stamm added a comment - The refresh of the list of tasks always happens now, even if when auto-refresh option is not checked. If possible, we could give the user some visual feedback that the task detail tab cannot be changed anymore (e.g. with a cursor: not-allowed style). I am not super happy with a test, that always waits 10 seconds, but I am not sure how to test it otherwise. I would be fine with removing the test.
          Hide
          valentin.vago Valentin Vago added a comment -

          Made the changes you suggested

          Show
          valentin.vago Valentin Vago added a comment - Made the changes you suggested

            People

            • Assignee:
              michael.schoettes Michael Schoettes
              Reporter:
              sebastian.stamm Sebastian Stamm
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: