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

[FIX] chart: remove empty datasets for chart

Open anhe-odoo opened this issue 1 year ago • 1 comments

Task Description

This task aims to undisplay empty datasets from a chart (when all the values are undefined, not when they are equals to 0). This has been done by ignoring the empty datasets during the runtime creation.

Related Task

  • Task: 4085864

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

anhe-odoo avatar Jul 31 '24 13:07 anhe-odoo

Pull request status dashboard

robodoo avatar Jul 31 '24 13:07 robodoo

@anhe-odoo could you close it if we agree that it's outdated?

rrahir avatar Sep 24 '25 11:09 rrahir

@rrahir I don't remember why we let this PR apart and restart from a fresh master-targeted PR (https://github.com/odoo/o-spreadsheet/pull/4842), but I suppose it was related to the "stable" policy as it's not really a bug but a change in behavior so, yes, we should close it.

anhe-odoo avatar Sep 25 '25 10:09 anhe-odoo