Tuesday, May 28, 2024
7 changes
3 changes
Resolved issues and error corrections
This fix ensures unread message counters in Discuss are initialized with the latest channel data, avoiding timing issues that could make counters appear incorrect during message notifications. It improves reliability for users without changing the messaging workflow.
Original PR description
The `channels.fetch()` was put inside `incrementUnreadCounter()` because it was assumed to be the only flow where the counter could change when all channels are not yet fetched, but it is also possible that the counter changes by just calling `getOrFetch()` in `_handleNotificationNewMessage()` and receiving a new counter directly from the RPC, therefore not doing the increment. In that case `initChannelsUnreadCounter` also becomes unreliable, so the force fetch is moved there instead. runbot-61556 runbot-62976
This fixes a subtle template handling issue where empty text could be saved differently after template inheritance changes. It helps avoid unnecessary differences in generated views and keeps existing templates stable when Studio customizations are applied.
Original PR description
When calling apply_inheritance_specs and moving a node (before after or inside), we merge the text content of the adjacents nodes. If the parent and target node both have no text, we should not set the text to an empty string. When a node has no text, it is serialized as follows: `<node/>` But if it has an empty string, it has the following representation: `<node></node>` In the linked PR, we now apply the studio inheritance manually, and since we use the resulting tree directly instead of parsing the result, the `remove_blank_text` option of the parser has no effect. This causes existing tests to show some difference. opw-3888345 opw-3924997
The Discuss notification settings menu now displays longer text correctly, including in languages such as Vietnamese. This prevents menu options from being cut off or misaligned when users mute channels, improving usability for multilingual teams.
Original PR description
Before this commit, the notification settings in discuss app that is used to mute channels was not showing items properly in some languages like vietnamese. This happens because the width is hard-coded to 150px as to keep the dropdown menu small. This commit fixes the issue by putting a max-width of 250px for dropdown menu and sub-menu, so that it keeps right positioning of submenu. If text is too long, it know wraps. 
1 change
Resolved issues and error corrections
This fixes an error that could occur when importing bank transaction data through online synchronization. The update ensures company information is handled correctly, allowing bank statement lines to be created without interruption.
Original PR description
"company_id" has been added to data retrun by "_format_transactions" method in https://github.com/odoo/enterprise/commit/393ca019515757c275df17b9319198f18471af60. This would cause an error when creating a new "account.bank.statement.line.transient". This commit adds company_id as field on account.bank.statement.line.transient.
3 changes
Resolved issues and error corrections
A fix in the core Kanban view system introduced an asynchronous timing change that affected how data loads. Web Studio tests have been updated to account for this new timing behavior, ensuring tests continue to work correctly with the improved Kanban functionality.
Original PR description
PR odoo/odoo#149626 fixes an issue in kanban views with progressbar by introducing a new hook in the RelationalModel, called before loading the root. This hook being async, it delays by at least a micro-tick the loading of the data. As a consequence, a studio tests had to be adapted, as it stepped all network calls, and two of them was done in // (so before the fix, the loading of the data was fired first, not it is not). Forward-Port-Of: odoo/enterprise#62746
A spelling error in the approvals module error message has been corrected. The message previously said "lease" instead of "least" when prompting users to attach documents. This fix ensures users see the correct, professional messaging when using the approvals feature.
Original PR description
There was a typo in the error message "You have to attach at **lease** one document.", where _lease_ should be _least_. This commit fixes that. We also export the latest version of the .pot file. Forward-Port-Of: odoo/enterprise#63069 Forward-Port-Of: odoo/enterprise#63051
This update removes unused code from the stock module that was no longer being used. The change cleans up the codebase by eliminating a `next_serial` argument that was added in a previous update but is no longer needed, helping maintain code quality and reduce technical debt.
Original PR description
 This `next_serial` argument was added in https://github.com/odoo/odoo/pull/117513.