odoo/o-spreadsheet#6259
Created by fw-bot
Merged
at c3d7040e6e92301a9dab3f1ca149126792bc8e00
Statuses:
- label
- odoo:master-saas-18.3-fix-compile-types-adrm-431859-fw
- head
- 67e1e828d8f080c6a6a7c484c6c9477052e6bc13
- merged
- 8 months ago by BI, Adrien Minne (adrm)
| odoo/o-spreadsheet | |
|---|---|
| saas-18.3 | #6210 |
| saas-18.4 | |
| 19.0 | |
| master | #6259 |
[FW][FIX] ts: fix compiling of `.d.ts` files
It wasn't possible to compile the lib into .d.ts since 8a02a6b, error ... exported class expression may not be private or protected.[1]
It seems that we cannot have an anonymous class with private/protected methods. We had a similar issue with the PivotPresentationLayer.
This commit adds explicit typing to the functions generating the funnel chart classes, so typescript doesn't try to infer types for them.
[1]: see microsoft/TypeScript#30355 or microsoft/TypeScript#36060
Description:
description of this task, what is implemented and why it is implemented that way.
Task: TASK_ID
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
Forward-Port-Of: #6210