Wednesday, December 14, 2022
4 changes · master
Code cleanup and technical improvements
This update reorganizes point of sale and related localization/payment modules into Odoo's newer JavaScript module format. It is an internal preparation step that should reduce future merge conflicts and make upcoming modernization work easier, without intended changes to day-to-day POS behavior.
Original PR description
*: l10n_co_pos, l10n_fr_pos_cert, l10n_gcc_pos, l10n_in_pos, l10n_sa_pos, pos_adyen, pos_discount, pos_epson_printer, pos_epson_printer_restaurant, pos_hr, pos_loyalty, pos_mercury, pos_restaurant, pos_restaurant_adyen, pos_restaurant_stripe, pos_sale, pos_sale_product_configurator, pos_six, pos_stripe We are about to refactor most of the Javasript code base of the point of sale and related modules. In doing so, we will move a lot of files and modernize the entire code-base. In order to avoid diffcult rebases, their conversion to odoo-modules is done as a first step to avoid getting lots of conflicts on files that were unindented, which marks the entire file as being in conflict. Conflicts will occur during forward ports but they will be easier to manage as the changes will be much smaller in scope, and the author will have the context of the change that causes the conflict in mind when dealing with it.
Spreadsheet dashboard data is now prepared earlier when dashboards are loaded, helping keep dashboard behavior consistent with related enterprise changes. A shared empty spreadsheet structure was also centralized for reuse, reducing duplication and supporting maintainability.
Original PR description
Description of the issue/feature this PR addresses: Current behavior before PR: Desired behavior after PR is merged: linked enterprise PR: https://github.com/odoo/enterprise/pull/34567 --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr
Spreadsheet documents and templates are now prepared before the spreadsheet screen opens, so the interface receives ready-to-use data instead of handling backend storage details. This reduces complexity in the Documents spreadsheet flow and should make future improvements easier and less error-prone without changing the core user experience.
This change converts several Point of Sale related modules to a newer internal JavaScript module format ahead of a broader modernization effort. It is not intended to change business functionality, but it should make future updates easier to deliver and reduce conflicts during ongoing development.
Original PR description
*: l10n_de_pos_cert, l10n_de_pos_res_cert, pos_hr_mobile, pos_iot, pos_l10n_se, pos_restaurant_iot, pos_settle_due We are about to refactor most of the Javasript code base of the point of sale and related modules. In doing so, we will move a lot of files and modernize the entire code-base. In order to avoid diffcult rebases, their conversion to odoo-modules is done as a first step to avoid getting lots of conflicts on files that were unindented, which marks the entire file as being in conflict. Conflicts will occur during forward ports but they will be easier to manage as the changes will be much smaller in scope, and the author will have the context of the change that causes the conflict in mind when dealing with it.