-
Notifications
You must be signed in to change notification settings - Fork 10.2k
[IMP] spreadsheet: remove default templates #11832
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
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.
Hello @mcsi-odoo great job on this PR! I added a few comments and suggestions; feel free to contact me if something isn't clear and we'll discuss 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.
I'd like to rework this page a little further, for clarity, also introducing how to use templates (including that + New Spreadsheet button, as you mentioned).
e15c15b
to
8f2453e
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.
A few minor suggestions @mcsi-odoo (feel free to reach out if there's anything you'd like to discuss!) but otherwise this looks very good to me 🙂 Thank you!
task-4278043
8f2453e
to
afdf893
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 @mcsi-odoo ! It looks great!
@robodoo r+
task-4278043 closes #11832 Signed-off-by: Audrey Vandromme (auva) <auva@odoo.com>
task-4278043 closes #11832 Signed-off-by: Audrey Vandromme (auva) <auva@odoo.com>
task-4278043