o-spreadsheet icon indicating copy to clipboard operation
o-spreadsheet copied to clipboard

[IMP] model: improve figure export data

Open hokolomopo opened this issue 4 months ago • 2 comments

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

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

hokolomopo avatar Aug 11 '25 08:08 hokolomopo

Pull request status dashboard

robodoo avatar Aug 11 '25 08:08 robodoo

@hokolomopo @LucasLefevre unable to stage: merge conflict

robodoo avatar Sep 09 '25 11:09 robodoo