-
Notifications
You must be signed in to change notification settings - Fork 9.9k
[IMP] spreadsheet: basics of creating and managing spreadsheets #14108
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
965edf7
to
1eabf9e
Compare
1eabf9e
to
6fa5753
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for your work on this @mcsi-odoo, this doc is already looking great; I only have minor comments on the content you added, including the following:
- I wouldn't use a menuselection when referring to the New dropdown to avoid confusion since it's not in the menu bar and it looks like a button anyways. Instead, I'd use: Click :guilabel:
New
and select :guilabel:XYZ
(like we do in the Documents doc). - In 18.0, they replaced "workspaces" with "folders" (except in the Settings 🙈 )
I also think we could add a couple of things:
- It might be useful to mention that you can click the view name next to a spreadsheet’s name to go back to the previous view (e.g., Documents if the spreadsheet was opened/created from the Documents app, or the previous spreadsheet if the spreadsheet was created from another spreadsheet (see my comment on line 56 about adding this info) or by making a copy of another spreadsheet).
- What about share and access rights? The feature works the same as in other documents, but it could be worth mentioning 1) where users can click to share/define access rights to a spreadsheet (and possibly include the Freeze and share option), and 2) a ref to the relevant part of the Documents documentation where share/access rights are already documented.
I’d suggest adding a short section at the end of this page for this, and linking back to it on line 31. What do you think?
6fa5753
to
f0deb18
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks Audrey! Most comments resolved, with feedback integrated. Main change in this version is some additional info about Dashboards, in response to your good comment about mentioning it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for the changes @mcsi-odoo ! I only have minor comments so I'll go ahead and approve this already 🙂 You can r+ yourself when you've made the changes and the checks have passed. Thanks!
@robodoo delegate+
f0deb18
to
cd2f77a
Compare
@robodoo r+ |
task-4943625