Uploaded image for project: 'Nuxeo Drive '
  1. Nuxeo Drive
  2. NXDRIVE-2132

Do not allow Direct Edit on proxies

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: NoFixVersionApplicable
    • Fix Version/s: 4.4.4
    • Component/s: Direct Edit
    • Release Notes Summary:
      Direct Edit on proxies is no more allowed
    • Release Notes Description:
      Hide

      Direct Edit on proxies is now forbidden. Allowing it was too bad for both server and client sides as it was:

      • filling up server logs with errors (see NXDRIVE-2131);
      • filling up client logs (see NXDRIVE-2131);
      • generating a huge amount of Sentry events, exploding our quota.

      Finally, such move was the next logical step after NXP-28497 where the Direct Edit button has been hidden for those ducuments.

      This is a 2-part fix alongside NXDRIVE-2131.

      Show
      Direct Edit on proxies is now forbidden. Allowing it was too bad for both server and client sides as it was: filling up server logs with errors (see NXDRIVE-2131 ); filling up client logs (see NXDRIVE-2131 ); generating a huge amount of Sentry events, exploding our quota. Finally, such move was the next logical step after NXP-28497 where the Direct Edit button has been hidden for those ducuments. This is a 2-part fix alongside NXDRIVE-2131 .
    • Tags:
    • Backlog priority:
      800
    • Sprint:
      nxDrive 11.1.33
    • Story Points:
      3

      Description

      There is a similar work done on the Web-UI side: NXP-28497.

      Direct Edit on proxy documents should be forbidden as it would:

      • fill up server logs with errors (see NXDRIVE-2131);
      • fill up client logs (see NXDRIVE-2131);
      • generate a huge amount of Sentry events, filing up our quota;

      Expected result:

      • Display a graphical error, as it is already done when trying to Direct Edit a version of a document (see NXDRIVE-1639 and associated commits), saying that it is not allowed to edit proxies.
      • A functional test must be added (the Document.CreateLiveProxy operation should be used).

        Attachments

          Issue Links

            Activity

              People

              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0 minutes
                  0m
                  Logged:
                  Time Spent - 1 day
                  1d