Tuesday, April 6, 2021
6 changes · master
Enhancements to existing features
The pull request updates the repository ownership rules so logistics-related code changes are automatically routed to the right team for review. This helps ensure relevant experts are involved earlier, improving review accountability without changing product behavior.
This change makes internal calculation ordering predictable so repeated CRM installations produce consistent database query counts. That consistency helps teams compare performance results more reliably and investigate remaining sources of variation.
Original PR description
[IMP] core: make compute order deterministic CRM install query count can vary from one execution to another, leading to difficulties when analysing performances evolution. The main reason for this is that some compute methods were called in different order. Even if compute order shouldn't have any effect on the final result, making it well defined will help finding other causes of non-determinism. The initial observation was that sorting Environment.fields_to_compute leads to a fixed number of query when installing crm. The main cause of non-determinisim is the usage of `set` impacting Field.compute_value and BaseModel._modified_triggers. Transforming all these `set` to `OrderedSet` solves the problem. The query count is now deterministic when installing a database from scratch, but not when updating a database with -i crm. OrderedSet is also slightly optimised by using a dict instead of an OrderedDict: dict order is deterministic since python3.6
This draft pull request adds test coverage for employee-related behavior in the HR module. It also includes a contributor license agreement file, helping ensure the contribution can be reviewed under project guidelines.
Original PR description
Description of the issue/feature this PR addresses: Together with my team we are investigating the software architecture of your system. To better understand the code base I'm trying to add a new test case to hr_employee Current behavior before PR: Some methods in hr_employee aren't covered by test. Desired behavior after PR is merged: Better test coverage. -- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr
Belgian accounting templates now record the non-deductible portion of VAT on the same account as the original base amount, instead of applying a separate default account. This aligns Belgian VAT handling with IPCF guidance and helps produce more compliant accounting entries by default.
Original PR description
According to the documentation from IPCF, the proportion of VAT that may not be claimed back must be recorded by default on the same account as the base amount in a different move line that this base amount. **Task ID:** #2373849 -- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr
This update improves day-to-day Helpdesk workflows by making website forms publish correctly, opening tickets in the right app, and preventing accidental SLA tag color changes. It also improves field service handoff by carrying sales information from tickets to tasks, adds kanban views for key setup areas, and supports archiving for helpdesk tags, ticket types, and stages.
Original PR description
…nts in helpdesk Currently, in the helpdesk module, - fsm integration: the sale_line_id set on the ticket is not propagated to the fsm task - after publishing the website, clicking on the edit button…
…nts in helpdesk Currently, in the helpdesk module, - fsm integration: the sale_line_id set on the ticket is not propagated to the fsm task - after publishing the website, clicking on the edit button opens the helpdesk form under website instead of the helpdesk app - sla tags on tickets: currently when we click on the tags, the color picker dropdown is shown - in settings: when enabling the 'website form' feature, the helpdesk team is not published by default on the website - in settings: team members: there is a 'keep empty...' label in the description of the feature and the field is not required - the kanban view is not implemented in the helpdesk team, stages, and sla policies menus - the helpdesk.tag, helpdesk.ticket.type and helpdesk.stage does not have any active field and corresponding archive filter So in this commit, - I have propagated the sale_line_id set on the ticket to the fsm task - clicking on the edit button opens the helpdesk form view under the helpdesk app instead of the website one - the sla tags are not clickable, so the color cannot be modified - when enabling the 'website form' feature, the helpdesk team is now published by default on the website - inside settings: team members: the 'keep empty...' label is removed from the description of the feature and the field is marked required now - the kanban view is implemented for the helpdesk team, stages, and sla policies menus - added active field and corresponding archived filter in the helpdesk.tag, helpdesk.ticket.type and helpdesk.stage TaskID: 2464407
The project screens now only show the analytic account option when analytic accounting or timesheets are actually configured. This reduces confusion for users by hiding an accounting-related field when it cannot be used or updated.
Original PR description
Currently, In project module -> configuration -> Project list view column "analytic account" is suggested as a hidden column when the accounting app or timesheet app are not configured. At the same time, the analytic account field is not shown in the project from view and cannot be updated. In this commit, I have made the "analytic account" field a hidden column only when the analytic accounting or timesheet app is configured. And the same applies to the project form view also. Task ID: 2451722