Friday, December 8, 2023
6 changes · master
Resolved issues and error corrections
ZPL product labels now print prices based on the pricelist selected in the label printing wizard. This keeps printed labels aligned with the intended customer or sales pricing and avoids showing the wrong product price.
Original PR description
before this commit, the pricelist concept is added in label printing in this commit: https://github.com/odoo/odoo/commit/168b56e0698e443508049cdb3507d1170a21725c and missed to adapt the zpl reports to respect the selected pricelist in the wizard after this commit, the price printed in the zpl product label will be based on the selected pricelist in the wizard --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr
Odoo now recognizes the port provided by a proxy server, so users accessing Odoo through a non-standard web port are no longer redirected to the wrong address. This improves reliability for companies hosting Odoo behind gateways such as Nginx.
Original PR description
Install nginx, setup it to listen on port 8080 instead of default 80
then setup a regular proxy_pass toward a local odoo listening on port
8069 with all the X-Forwarded- headers set:
listen 8080;
server_name mycompany.odoo.com;
proxy_set_header X-Forwarded-Host $host;
proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
proxy_set_header X-Forwarded-Proto $scheme;
proxy_set_header X-Forwarded-Port $server_port;
proxy_set_header X-Real-IP $remote_addr;
location / {
proxy_pass http://127.0.0.1:8069;
}
Inside your browser, access "http://mycompany.odoo.com:8080" you are
wrongly redirected to "http://mycompany.odoo.com:80".
The `X-Forwarded-Port` header wasn't taken into account by Odoo.
Closes: #64643Fixed an issue where employee avatar cards could fail to open when the Skills app was installed without Planning. Employee skills now display reliably because the shared tag display support is available from the core HR app.
Original PR description
Steps ===== - Install module hr_skills (but not planning) - Open the avatar card of an employee with skills linked to it Issue ===== A traceback appears indicating: "OwlError: Cannot find the definition of component "TagsList", missing static components key in parent". Cause ===== The AvatarCardResourcePopover component was patched in planning to add the component TagsList to it. This component was also used by the patch applied on AvatarCardResourcePopover in hr_skills. However, when hr_skills is installed but not planning, the component TagsList is used (to display the skills of the employee) but no append to the AvatarCardResourcePopover (or the AvatarCardEmployeePopover that inherits from it). Fix === The component TagsList is added in hr instead of planning as this module is required by both planning and hr_skills.
This update makes small popover windows more reliable by allowing them to open even when optional details are not provided. It prevents a crash in this situation, improving stability for users and developers building interface elements.
Original PR description
With this commit, it is no longer necessary to pass props when adding a popover using the popover service. Before this commit, there was a crash if no props were passed. Task ID: 3611253 Description of the issue/feature this PR addresses: Current behavior before PR: Desired behavior after PR is merged: --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr
This fixes crashes when opening employee or resource avatar cards in cases where related modules or access rights differ. Users can now view relevant avatar details without unexpected errors, including planning users who do not have HR permissions.
Original PR description
[FIX] planning: add TagsList component to avatar card in hr instead of planning =============================================================================== Steps ----- - Install module hr_skills…
[FIX] planning: add TagsList component to avatar card in hr instead of planning =============================================================================== Steps ----- - Install module hr_skills (but not planning) - Open the avatar card of an employee with skills linked to it Issue ----- A traceback appears indicating: "OwlError: Cannot find the definition of component "TagsList", missing static components key in parent". Cause ----- The AvatarCardResourcePopover component was patched in planning to add the component TagsList to it. This component was also used by the patch applied on AvatarCardResourcePopover in hr_skills. However, when hr_skills is installed but not planning, the component TagsList is used (to display the skills of the employee) but no append to the AvatarCardResourcePopover (or the AvatarCardEmployeePopover that inherits from it). Fix --- The component TagsList is added in hr instead of planning as this module is required by both planning and hr_skills. [FIX] planning: avoid traceback for non hr-user when opening avatar card ======================================================================== Steps ----- - Install planning - Log in as a planning user with no hr access - Open the planning app - Click on the avatar of a resource with roles linked to it Issue ----- An access right error message appears indicating that the current user can't access the default_role_id field of the resource. Cause ----- This field of the resource.resource model is protected to user with hr access rights. In the avatar card component it was read, whatever the rights of the current user were. Fix --- A check on the access rights of the current user is added in the component to avoid to read that field if it is not allowed.
Users can once again create a new shift directly from the selection window opened from a planning Gantt cell. This fixes a regression that hid the create option, reducing friction for scheduling teams.
Original PR description
Before this commit, due to a bad rebase the override of `getSelectCreateDialogProps` in `sale_planning` module has been reintroduced and so that override disables the create button in select create dialog displayed when we click on gantt cell. This commit removes that override to allow the user to be able to create a new shift when he is in the select create dialog.