[FIX] collaborative: don't snapshot in read-only mode

LucasLefevre commented 4 days ago

Description:

Steps to reproduce:

open two browser windows with 2 different users: Alice and Bob. Alice has write access and Bob have only read-only access
open the same spreadsheet with both users
Alice updates a few cells
Alice leaves the spreadsheet
Bob leaves the spreadsheet

=> Bob is faced with an access error because he is trying to snapshot the spreadsheet.

[ ] feature is organized in plugin, or UI components
[ ] support of duplicate sheet (deep copy)
[ ] in model/core: ranges are Range object, and can be adapted (adaptRanges)
[ ] in model/UI: ranges are strings (to show the user)
[ ] undo-able commands (uses this.history.update)
[ ] multiuser-able commands (has inverse commands and transformations where needed)
[ ] new/updated/removed commands are documented
[ ] exportable in excel
[ ] translations (_t("qmsdf %s", abc))
[ ] unit tested
[ ] clean commented code
[ ] track breaking changes
[ ] doc is rebuild (npm run doc)
[ ] status is correct in Odoo

robodoo commented 4 days ago

VincentSchippefilt commented 2 days ago

robodoo r+