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

[POC] keybinds: Add keyboard shortcut registries

Open rrahir opened this issue 3 years ago • 1 comments

Description:

description of this task, what is implemented and why it is implemented that way.

Odoo task ID : 2593227

review checklist

  • [ ] undo-able commands (uses this.history.update)
  • [ ] multiuser-able commands (has inverse commands and transformations where needed)
  • [ ] translations (_lt("qmsdf %s", abc))
  • [ ] unit tested
  • [ ] clean commented code
  • [ ] feature is organized in plugin, or UI components
  • [ ] exportable in excel
  • [ ] importable from excel
  • [ ] in model/core: ranges are Range object, and can be adapted (adaptRanges)
  • [ ] in model/UI: ranges are strings (to show the user)
  • [ ] new/updated/removed commands are documented
  • [ ] track breaking changes
  • [ ] public API change (index.ts) must rebuild doc (npm run doc)
  • [ ] code is prettified with prettier (in each commit, no separate commit)
  • [ ] status is correct in Odoo

rrahir avatar Jul 13 '21 14:07 rrahir

hey. you may be interested to know that we improved the registry system in web: registries can have sub registries, which are created lazily. It really simplified our registry management (see web/static/src/core/registry.js and maybe look for uses of it in master). Not sure though if it works well with a fully typed system.

ged-odoo avatar Jul 13 '21 15:07 ged-odoo