Wednesday, March 5, 2025
4 changes · saas-18.1
Resolved issues and error corrections
The translation button is now visible again in Studio when editing views and XML. This fixes a regression that prevented users from accessing translation actions in key Studio editing screens.
Original PR description
In studio, the translation button is always invisible in the ir.ui.view form and more importantly studio xml editor since e5a0100d09b3c150ae90e8fc4a5e3c7cb2fe83ec. opw-4519334
This fix makes an automated mail chat test more reliable by ensuring the chat composer is treated as unfocused before checking the compact chat counter. It helps keep the validation pipeline stable without changing the user-facing chat behavior.
Original PR description
Follow-up of odoo#199828 PR above added a new test that resulted in frequent runbot failures: ``` Failed assertions: 1. [toBe] Failed to find 1 of ".o-mail-ChatHub-compact" with text "1" (Timeout of…
Follow-up of odoo#199828 PR above added a new test that resulted in frequent runbot failures: ``` Failed assertions: 1. [toBe] Failed to find 1 of ".o-mail-ChatHub-compact" with text "1" (Timeout of 3 seconds). Found 0 instead. > Expected: true > Received: false 2. [errors] 1 unverified error(s) Error during test: Failed to find 1 of ".o-mail-ChatHub-compact" with text "1" (Timeout of 3 seconds). Found 0 instead. ``` This happens because the test had a chat window open, enabled the compact mode of chat hub, then posts a message and assert there's a counter on the chathub compact button. Everything works fine when a human do these interactions. However the test failed because there was no counter. This comes from chat window modeling that was thinking the composer was still focused, therefore it was marking the conversation as read, leading to consuming the "1" counter. Composer was kept in "focused" state because when programmatically simulating a `click`, the detection of loss of focus is made with a `focusout` event that is triggered normally on the previously focused element. In HOOT, the `@mail` click test helper was not triggering `focusout` on the previous element. Some attempts were made to programmatically trigger the "focusout", including some reliance of tests that do `mousedown.prevent` that are expected to not change focus on click, but tests keep failing. As this is important to make runbot green as soon as possible, this commit fixes the issue by helping the problematic test to blur/focusout the composer. Note that the `contains().click` from `web_test_helpers` seem to properly managed the focusous and the `mousedown.prevent`, but there are a lot of occurrences of mail `await click()` to change, and this without taking into account some issues in converting to `contains.click()` at the moment: - timeout of click is 200ms, which is too short for discuss tests - mail `click` feature uses `text` to make exact comparison of textnode, compared to `:contains()`. This is probably not a big issue as with `value` where asserting composer is empty asynchronously is very important for discuss The plan is to use web test helpers for mail tests, but this requires more time than the urgency to fix this test now. runbot-159692
This update addresses a problem around selecting customer names in the Point of Sale interface. It helps make cashier workflows more reliable when choosing or confirming a customer during checkout.
Original PR description
to be updated, or discarded ...
This fix ensures the administrator account has an email address during Sign and Knowledge test runs. It prevents false test failures in environments without demo data, improving release validation reliability without changing user-facing features.
Original PR description
## [FIX] sign: set email address to admin user Before this commit, the test `/sign:TestUi.test_ui` fails in no demo because no email address is set on admin user. This commit fixes the issue by setting an email address inside the test before launching the tour. runbot-error-98514 ## [FIX] knowledge: make sure admin user has email set in no demo Before this commit, `/knowledge:TestKnowledgeEditorCommands.test_knowledge_article_commands_tour` fails because the admin user does not have an email address set. This commit makes sure the email address is set on admin user before launching the tour to be sure the tour can be executed without any issue in no demo.