odoo/o-spreadsheet#6486
Created by fw-bot
Merged
at 4f474490be929802ae3cbe7453e671272238a1da
Statuses:
- label
- odoo:saas-18.1-saas-17.4-pivot-month-granularity-sorting-adrm-437130-fw
- head
- f1edf9e5e749bd3aae8b6125a9d95128e16217d0
- merged
- 1 week ago by BI, Rémi Rahir (rar)
| odoo/o-spreadsheet | |
|---|---|
| saas-17.4 | #6335 |
| 18.0 | #6436 |
| saas-18.1 | #6486 |
| saas-18.2 | #6487 |
| saas-18.3 | #6488 |
| master | #6489 |
[FW][FIX] pivot: `month` granularity sorting
Description
The month granularity sorting wasn't working. The issue was that we were using Number(value) instead of toNumber(value) to convert the field value to a number.
Task: 4781810
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