Uploaded image for project: '[Read Only] - Addon Channel Manager'
  1. [Read Only] - Addon Channel Manager
  2. CHANNELMGR-1750

Add-to-project/edit document must be disabled when in review state

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Normal
    • Resolution: Won't Fix
    • None
    • None
    • None

    Description

      When a project is in review, the visual editor must not show to option to add a document to the active project. Instead the message the docunment can't be added must be shown

      Reproduction locally

      • create a new project P
      • add a channel C
      • "Request review"
      • go to channel C, and make sure project P has been selected
      • edit a document

      Current sitation

      • a dialog is shown with the "Add to project" option.

      Desired situation

      The edit option should not be visible ( see CHANNELMGR-1731 ).

      Reproduction cluster wide

      • log in as user U1 on browser B1
      • create a new project P on browser B1
      • add a channel C on browser B1
      • log in as user U2 on browser B2
      • go to the projects perspective and select project P
      • "request review" as user U2 on browser B2
      • on browser B1 go to channel C, and make sure project P has been selected
      • on browser B1 edit a document

      current situation

      • a dialog is shown with the "add to project" option and when the user clicks that option on error is shown

      Desired situation

      The whole channel manager has been designed according to the principle of "optimistic locking". So it is consistent.
      Secondly, if we would update the project, in the current code base the whole channel would have to be reloaded on each edit action by the user. This will result in poor performance and bad UX.

      Conclusion.

      This issue can be closed as "won't fix" once CHANNELMGR-1731 has been fixed.

      Attachments

        Issue Links

          Activity

            People

              Unassigned Unassigned
              abanck Arent-Jan Banck (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: