Wednesday, April 19, 2023
9 changes · master
Resolved issues and error corrections
Link preview images in chat messages now stay within the chat window instead of spilling outside it. This makes shared links easier to view and keeps the chat layout clean and professional.
Original PR description
Before this PR, a link preview image would overflow out of the chat window. This because the max-width on the image is bigger than the space dedicated to the image inside the chat window. This PR set the max-width to 100% inside a chat window, so the image cover the space nicely. Before:  After: 
This fixes an issue in Point of Sale where removing an order line could cause the customer display to crash. The display update is now delayed slightly so it only refreshes once the order data is stable, improving reliability during checkout.
Original PR description
Steps to reproduce: - Activate customer display. - Open a pos session. - Open the customer display. - Add orderline. - Delete the orderline. Then there's traceback. This is caused by the fact that the `effect` used to update the customer display is synchronous, and when removing an order, `splice` is applied in multiple steps, and causes multiple notifications to the underlying reactive. At some of those steps, the reactive state is inconsistent (ie, one of the slots of the reactive array is empty) causing the call to map to result in an array with empty slots which cannot be used in `Object.fromEntries`. This commit fixes that my batching the updates to the customer display in the next microtask tick, meaning that the update will never be done with an inconsistent state.
List view totals for monetary fields now avoid showing misleading aggregates when records use different currencies. When totals are valid, the relevant currency is displayed so users can interpret amounts correctly.
Original PR description
This commit fixes how monetary fields are handled when doing aggregates in list view: aggregates should not be computed when all values are not in the same currency and the currency should be displayed with the aggregate when this is not the case. Steps to reproduce: create a list view with a monetary field in studio, add an aggregate of the values of this field and check the behavior with and without a currency field present. opw-2822174
Mobile chat windows now use the full screen height, making conversations easier to view and use on phones. Desktop behavior remains unchanged, and mobile users will no longer see desktop-only folding controls.
Original PR description
Before this commit, chat window take at most 95% of global height. This is ok in desktop, 5% room allow clicking on systray menu. In mobile, however, the intend is for chat window to take whole height. This commit makes chat window take whole height in mobile, while preserving the max-height 95% of global height in desktop. Also fix an issue where chat window were foldable in mobile, when this feature is desktop-only.
This fix adjusts the spacing at the end of incoming message bubbles in the Discuss chat window. It makes chat conversations look cleaner and more consistent, improving readability without changing functionality.
Original PR description
Before/after <img width="341" alt="before" src="https://user-images.githubusercontent.com/6569390/233110098-e4922f80-7799-4e21-b39c-888c66899f09.png"> <img width="338" alt="after" src="https://user-images.githubusercontent.com/6569390/233110151-3175ec49-aa70-48a0-9b99-20ffafa18c5c.png">
The invoice extraction process now includes a dedicated total tax amount value again under a clearer name. This helps the extraction server validate invoice data more reliably, reducing the risk of incorrect tax information being accepted.
Original PR description
`global_taxes_amount` was removed in 76f04a7. This commit introduces `total_tax_amount`, which is the same, but with a new name. This is reintroduced mainly to improve validation in the extract server.
The Helpdesk settings tooltip for email aliases was updated to reflect that alias domains are now configured separately from custom email servers. This prevents users from following outdated guidance when setting up incoming helpdesk emails.
Original PR description
Description of the issue/feature this PR addresses: the 'alias domain' has been splited from the 'custom email servers' feature in the general settings. As a consequence, our tip is now not correct. Changed it to not be misleading anymore. --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr
Users can now preview a document without losing the records they had already selected. This makes bulk document work smoother by avoiding the need to reselect items after checking a preview.
When an offer template is archived, it is now removed from the related job position. This keeps job position records cleaner and avoids users seeing or relying on inactive contract templates.
Original PR description
Currently, if we archived the offer template(Contract Template), the contract template is not removed from the Job Position.In this commit we remove it. task-3246004