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

[FIX] charts: fix zoomable scatter chart

Open hokolomopo opened this issue 2 months ago • 1 comments

Description:

[FIX] charts: fix zoomable scatter chart

The zoomable scatter chart would display a line chart in the "master", instead of a smaller scatter chart.

[IMP] tests: add zoomable chart plugin tests

The chart runtime generated for the "master" chart of zoomable charts was actually never properly tested, except indirectly through unrelated snapshot tests.

[MOV] tests: move zoomable chart tests to a dedicated folder

Task: 5143146

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 Oct 10 '25 13:10 hokolomopo

Pull request status dashboard

robodoo avatar Oct 10 '25 13:10 robodoo

Merge method set to rebase and fast-forward.

robodoo avatar Nov 07 '25 07:11 robodoo