Daily updates from Odoo
Wednesday, April 27, 2022
8 changes · master
Enhancements to existing features
Users can now give clear custom names to pivots, lists, and action links in Documents spreadsheets. This makes spreadsheets easier to understand when multiple reporting elements are inserted, reducing confusion and improving navigation.
Original PR description
Before this commit, the name of pivots/lists were the display name of the technical model. When users insert mutilple pivots/lists on the same spreadsheet, it quickly becomes difficulte to know what pivot/list is used for what. With this commit, users are now able to give a name, either from the spreadsheet selector dialog or from the side panel. Task-id 2696631
This update refreshes the spreadsheet component used in Documents with recent improvements. Users get smoother formula editing with arrow keys and easier chart actions via right-click menus, while internal test and code maintenance changes improve reliability.
Original PR description
### Contains the following commits: https://github.com/odoo/o-spreadsheet/commit/4c088fc7 [IMP] composer: arrow always moves the cursor when editing a formula https://github.com/odoo/o-spreadsheet/commit/759d1cff [IMP] charts: can open menu with right-click https://github.com/odoo/o-spreadsheet/commit/2ab69179 [FIX] tests: fix FakeTimers warning in grid tests https://github.com/odoo/o-spreadsheet/commit/4660f4dd [REF] highlights: rename `sheet` to `sheetId` https://github.com/odoo/o-spreadsheet/commit/d8d14870 [REF] highlights: use zones instead of string xcs https://github.com/odoo/o-spreadsheet/commit/b86dd634 [IMP] tests: introduce gridMouseEvent helper https://github.com/odoo/o-spreadsheet/commit/c12e1af9 [FIX] tests: adapt edge-scrolling tests to 86db8e6d
Payroll access has been refined with a new employee manager role. Managers can now view contracts for their direct reports without receiving broader payroll officer or administrator access to all company salary information.
Original PR description
Currently, there are two security toles in Payroll: officer and administrator. Both rols allow the user to have a full overview on all salaries of the company. This commit adds an employee manager role allowing an employee manager to view his reports' contracts. task-2731933
The live currency rate update process now handles provider errors more transparently and avoids waiting too long when external services do not respond. Company currency provider settings are also applied more consistently during setup, reducing configuration confusion.
Original PR description
Do not discard exceptions ========================= Simply returning False makes it impossible to know what the error was when it was raised since nothing is logged. It is also hiding the real nature…
Do not discard exceptions ========================= Simply returning False makes it impossible to know what the error was when it was raised since nothing is logged. It is also hiding the real nature of the error one might not even have thought of. A bare except is also bad practice because it is catching more than Exceptions Add timeout and raise for status ================================ We don't want the cron pool to be saturated for longer than needed because a distant server is not responding. It is also cleaner to fail with a good type of Exception and message thanks to `raise_for_status` instead of having keys missing in the json dict later or whatnot. Do not default provider only on company create ============================================== Having a default + `create` extension won't change the value if you change the country after having created the company, which might be the case when setuping a new database for instance. Require interval unit ===================== Since the code doesn't make a difference between `manually` and `False`, remove confusion by removing the ability to empty the field.
This update reorganizes parts of the Approvals and Sign apps so their screens are backed by clearer internal view models. It should not change day-to-day behavior for users, but it helps the product team maintain and evolve these apps more reliably over time.
Original PR description
This commit introduces models that define records being 1:1 map with components, as a step to move further to having essentially all business code in models. Having code in models is desirable to have very maintainable code, thanks to robust and declarative code with an ORM-like architecture. Task-2831082 community: https://github.com/odoo/odoo/pull/89735
This update adjusts internal approval app test helpers so they match a newer mock server structure. It is a behind-the-scenes preparation that reduces future migration effort and should not change the user experience.
Original PR description
This PR helps reducing the noise in the discuss new env PR. Indeed, the new mockServer has the same method except that they are not prefixed by any underscore. All calls have been changed, and the corresponding methods created. They will be deleted later. task-2582313 community: https://github.com/odoo/odoo/pull/89659
A new debug-only menu option lets administrators request a fresh spreadsheet snapshot when a file is slow to load. This improves loading performance by clearing the current undo and redo history for that spreadsheet session.
Original PR description
The purpose of this commit is to add a menu item in the file menu (only visible in debug mode) in order to request a snapshot. Requesting a snapshot will imply to clear the revisions of the current session, so the history (undo/redo) will be lost, but loading time will be reduce. So this button should be used when a spreadsheet takes long time to load. In an ideal world, we should be able to clearly identify when a session should be closed (when there is no more active users).
The mail enterprise code now uses the newer “isSmall” naming instead of “isMobile” when adapting Discuss views to smaller screens. This is an internal cleanup that makes future Discuss improvements easier to integrate without changing visible behavior for users.
Original PR description
In order to reduce the noise in the new discuss env PR, all the messaging.device.isMobile occurences have been renamed to isSmall in order to match the new env terminology. task-2582313 community: https://github.com/odoo/odoo/pull/89655