odoo/o-spreadsheet#4587
Created by fw-bot
Merged
at 0d6a524bbc615ac7de6b27b99a86b0f726e7b275
Statuses:
- label
- odoo:saas-17.3-17.0-locale-without-thousands-adrm-lsc9-fw
- head
- 6a99b3b7c697083df813d9e3c2e742dc9d530e80
- merged
- 3 months ago by BI, Adrien Minne (adrm)
| odoo/o-spreadsheet | |
|---|---|
| 17.0 | #4577 |
| saas-17.1 | #4578 |
| saas-17.2 | #4586 |
| saas-17.3 | #4587 |
| master | #4588 |
[FW][FIX] locale: handle undefined thousands separator
Description
This commit adds the handling of undefined thousands separator in the locales.
Also fix a bug where we couldn't canonicalize a number with multiple thousands separators.
Also add a warning in the console when an invalid locale is filtered from the side panel.
Task: : 0
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