odoo/o-spreadsheet#6914
Created by BI, Adrien Minne (adrm)
- label
- odoo:master-carousel-figure-various-fixes-adrm
- head
- f88eb2e8453b4bbd91fddb9fa135620202c82ec4
| odoo/o-spreadsheet | |
|---|---|
| master | #6914 missing r+ |
[IMP] model: improve figure export data
Description:
Currently, charts/images/carousels are exported inside figure.data
in the JSON. That's not ideal, figure.data then needs to be typed as
any, and the charts plugin need to be aware of carousels for its
import.
This commit changes the export, now each plugin has its own key in the
data, so it can be typed correctly and they are not interlinked.
Task: 5002886
review checklist
- [ ] 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