Search
Navigate
Branch
Wednesday, December 25, 2024
11 changes
3 changes
Miscellaneous changes
Issue introduced at https://github.com/odoo/odoo/commit/ce92dedea0fd3cdc73da6366c20b8052bb04f7e9 , While doing the fix for ewaybill, now it's causing issue for `E-Invoice`, because the partner buyer details were changed from Purchase Document to Move type being a Outbound In this fix we only change Buyer-Seller Details only if it's for e-waybill Description of the issue/feature this PR addresses: Current behavior before PR: Desired behavior after PR is merged: --- I conf
Original PR description
Issue introduced at https://github.com/odoo/odoo/commit/ce92dedea0fd3cdc73da6366c20b8052bb04f7e9 , While doing the fix for ewaybill, now it's causing issue for `E-Invoice`, because the partner buyer details were changed from Purchase Document to Move type being a Outbound In this fix we only change Buyer-Seller Details only if it's for e-waybill 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 Forward-Port-Of: odoo/odoo#191578
Since [1], a fix was introduced to display the actual value of a relational field instead of its ID. However, another issue occurs when a form is inside a tab (Tabs snippet) and a new tab is added. During this process, the currently active tab is cloned, including its field IDs. Although a method exists to update the IDs after cloning, the form rendering is triggered beforehand. This causes the labels property of input elements—which is expected to return a NodeList of associated <label> elem
Original PR description
Since [1], a fix was introduced to display the actual value of a relational field instead of its ID. However, another issue occurs when a form is inside a tab (Tabs snippet) and a new tab is added.…
Since [1], a fix was introduced to display the actual value of a relational field instead of its ID. However, another issue occurs when a form is inside a tab (Tabs snippet) and a new tab is added. During this process, the currently active tab is cloned, including its field IDs. Although a method exists to update the IDs after cloning, the form rendering is triggered beforehand. This causes the labels property of input elements—which is expected to return a NodeList of associated <label> elements—to fail because the cloned input IDs are not unique. This commit resolves the issue by using querySelector to target the associated label elements directly. Steps to reproduce: - Open the Website editor. - Drag & Drop a Tabs snippet. - In the first tab, drag & Drop a Form snippet. - Set the first field as a radio buttons. - Make the second field's visibility depend on the first field. - Add a new tab by clicking the "+" icon in the right panel. - Observe that a traceback occurs. A second issue related to [1] has been resolved for cases where there is only a single checkbox. In this scenario, the value displayed in the visibility condition dropdown menu should be "Yes". opw-4345340 [1]: https://github.com/odoo/odoo/commit/4b37f22cece5424d6992c6bafe619c9a457d184b Forward-Port-Of: odoo/odoo#190460
Israel is raising VAT rate from 17% to 18% on 1/1/2025. Changes: Added new taxes (17% -> 18%) Archived 17% taxes Added tax group (VAT 18%) Modified fiscal position to match those taxes Modified VAT report to include VAT Inputs 18% Updated default tax ids for IL res.company Backward-Port-Of: https://github.com/odoo/odoo/pull/191209 --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-Port-Of: odoo/odoo#191749
Original PR description
Israel is raising VAT rate from 17% to 18% on 1/1/2025. Changes: Added new taxes (17% -> 18%) Archived 17% taxes Added tax group (VAT 18%) Modified fiscal position to match those taxes Modified VAT report to include VAT Inputs 18% Updated default tax ids for IL res.company Backward-Port-Of: https://github.com/odoo/odoo/pull/191209 --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-Port-Of: odoo/odoo#191749
4 changes
Enhancements to existing features
The preparation display no longer shows a filter bar when the left sidebar is closed. This keeps filter choices less prominent, since they are usually set once per day, and frees up screen space so staff can see more orders at once.
Original PR description
Removal of the topbar displaying filters when the left sidebar is closed. This feature was not in the original specification and does not match expectations. Indeed, it shouldn't be easy for the user to change the filter selection, which is performed once per day and generally remains the same. Space must be preserved to display as many orders as possible. Before:  After: 
4 changes
Enhancements to existing features
This update refines the visual design of Odoo Discuss across sidebars, calls, member panels, mobile chat windows, and search. The changes make the collaboration interface less distracting, better aligned, and easier to scan while also correcting a missing search icon.
Original PR description
This commit makes minor style improvemens to make discuss a bit more enjoying to use: - less start spacing on non-compact discuss sidebar items - discuss sidebar item outline style when self in call…
Resolved issues and error corrections
This fix prevents an error from appearing when users select a document record in mobile view after reloading the page. It improves reliability for mobile users working with Documents by ensuring record selection behaves normally.
Original PR description
**Steps to reproduce:** Switch to mobile view Reload the page Select any record Traceback occurs **Technical:** After commit https://github.com/odoo/odoo/commit/ef3656e3468ba6dab207c8df65d12fbb75ae4c77, the 'isPageSelected' condition was removed for the ALL button in the "web.list.selection" template. As a result, the first two conditions will be true, and the system will then check for model.root.isRecordCountTrustable. However, at that point, the model is not available. Previously, the false value of isPageSelected would prevent the check for the model from occurring. **After this PR:** The user will not encounter any traceback when selecting a record in mobile view. Task-4402442
Miscellaneous changes
**Problem:** l10n_co_dian_certification_process (Activate the certification process) can still be checked if l10n_co_dian_test_environment (Test environment) is left unchecked. There are two ways to cause this: 1. | | l10n_co_dian_test_environment | l10n_co_dian_certification_process | |--------|--------|--------| | Start | False | False | | End | False | True | Here, the write call will trigger l10n_co_dian_certification_process is set from False to True without triggering the “
Original PR description
**Problem:** l10n_co_dian_certification_process (Activate the certification process) can still be checked if l10n_co_dian_test_environment (Test environment) is left unchecked. There are two ways to…
**Problem:** l10n_co_dian_certification_process (Activate the certification process) can still be checked if l10n_co_dian_test_environment (Test environment) is left unchecked. There are two ways to cause this: 1. | | l10n_co_dian_test_environment | l10n_co_dian_certification_process | |--------|--------|--------| | Start | False | False | | End | False | True | Here, the write call will trigger l10n_co_dian_certification_process is set from False to True without triggering the “if” condition since l10n_co_dian_test_environment is not set. 2. | | l10n_co_dian_test_environment | l10n_co_dian_certification_process | |--------|--------|--------| | Start | True | False | | End | False | True | Here, l10n_co_dian_test_environment is set from True to False from the first write call, which will trigger the “if” condition, causing it to also set l10n_co_dian_certification_process to False. However, then the l10n_co_dian_certification_process is set from False to True by the second original write call. **Purpose:** Make the l10n_co_dian_certification_process field a computed field such that l10n_co_dian_certification_process cannot be set to True if l10n_co_dian_test_environment is False. **Steps to Reproduce on Runbot:** 1. Install l10n_co_dian Case 1: 2. Start at the state, where both l10n_co_dian_test_environment and l10n_co_dian_certification_process are False 3. Check the box for l10n_co_dian_test_environment so that the box for l10n_co_dian_certification_process will also show up. Then check the box for l10n_co_dian_certification_process and uncheck the box for l10n_co_dian_test_environment. Finally save the new configuration. Case 2: 2. Start at the state, where l10n_co_dian_test_environment is True and l10n_co_dian_certification_process is False 3. Check the box for l10n_co_dian_certification_process then uncheck the box for l10n_co_dian_test_environment. Finally save the new configuration. opw-4225471 Forward-Port-Of: odoo/enterprise#76098 Forward-Port-Of: odoo/enterprise#73532
Purpose of this commit: Remove files that depends on legacy 'bus/../test/helpers'. Part of:3818666 Forward-Port-Of: odoo/enterprise#75949
Original PR description
Purpose of this commit: Remove files that depends on legacy 'bus/../test/helpers'. Part of:3818666 Forward-Port-Of: odoo/enterprise#75949
This commit makes minor style improvemens to make discuss a bit more enjoying to use: - less start spacing on non-compact discuss sidebar items - discuss sidebar item outline style when self in call is improved (better lines, more visible) - discuss sidebar item when there's a call is improved: outline color matches border, background colors are more pleasing, spacing of participants and icons feels more right - discuss app header buttons are less distracting (reduced opacity when not hovered, active color is more suble in dark theme) - member panel is narrower in width by default (default value was too large most of the time) - member panel categories (online / offline) are smaller and reduced opacity, so that they are less distracting - hover effect on offline members is less intense (opacity on hover is reduced) - member names have slightly reduced opacity (was more catchy than messages with white color) - call status in systray is `.fa-volume-up.text-danger` rather than `text-warning` blinking dot. Also blinking animation duration is 6 seconds rather than infinite. - call status in systray has style matching discuss sidebar, with outline around the button. The spacing is reduced so it looks like a cohesive button (previously looked like 2 different systray items). - call status of active participant in compact sidebar is in a more natural position (was top left, now it's bottom right). - "start a meeting" and "Invite a user" buttons have slightly less distracting icons (reduced opacity). "Invite a user" button has slightly less horizontal margin - search panel icon was missing (had typo `.oi-serch`) - call view user names is less distracting (muted name, background has less vertical padding) - call card background color matches the dark theme better. - chat window mobile buttons look better (rounded-circle instead of square, some margin around, and border opacity is reduced so click area is visible but absolutely not distracting) - threads panel (sub-threads) has search input start aligned with thread items (was slightly misaligned) Before <img width="2560" alt="before" src="https://github.com/user-attachments/assets/9f7025fa-26f5-4381-bc7d-195fda480626" /> After <img width="2560" alt="Screenshot 2024-12-24 at 22 55 20" src="https://github.com/user-attachments/assets/dde1698e-ba75-48a7-b453-82ea5e82f120" />
Miscellaneous changes
Since [1], a fix was introduced to display the actual value of a relational field instead of its ID. However, another issue occurs when a form is inside a tab (Tabs snippet) and a new tab is added. During this process, the currently active tab is cloned, including its field IDs. Although a method exists to update the IDs after cloning, the form rendering is triggered beforehand. This causes the labels property of input elements—which is expected to return a NodeList of associated <label> elem
Original PR description
Since [1], a fix was introduced to display the actual value of a relational field instead of its ID. However, another issue occurs when a form is inside a tab (Tabs snippet) and a new tab is added.…
Since [1], a fix was introduced to display the actual value of a relational field instead of its ID. However, another issue occurs when a form is inside a tab (Tabs snippet) and a new tab is added. During this process, the currently active tab is cloned, including its field IDs. Although a method exists to update the IDs after cloning, the form rendering is triggered beforehand. This causes the labels property of input elements—which is expected to return a NodeList of associated <label> elements—to fail because the cloned input IDs are not unique. This commit resolves the issue by using querySelector to target the associated label elements directly. Steps to reproduce: - Open the Website editor. - Drag & Drop a Tabs snippet. - In the first tab, drag & Drop a Form snippet. - Set the first field as a radio buttons. - Make the second field's visibility depend on the first field. - Add a new tab by clicking the "+" icon in the right panel. - Observe that a traceback occurs. A second issue related to [1] has been resolved for cases where there is only a single checkbox. In this scenario, the value displayed in the visibility condition dropdown menu should be "Yes". opw-4345340 [1]: https://github.com/odoo/odoo/commit/4b37f22cece5424d6992c6bafe619c9a457d184b Forward-Port-Of: odoo/odoo#191709 Forward-Port-Of: odoo/odoo#190460
Issue introduced at https://github.com/odoo/odoo/commit/ce92dedea0fd3cdc73da6366c20b8052bb04f7e9 , While doing the fix for ewaybill, now it's causing issue for `E-Invoice`, because the partner buyer details were changed from Purchase Document to Move type being a Outbound In this fix we only change Buyer-Seller Details only if it's for e-waybill Description of the issue/feature this PR addresses: Current behavior before PR: Desired behavior after PR is merged: --- I conf
Original PR description
Issue introduced at https://github.com/odoo/odoo/commit/ce92dedea0fd3cdc73da6366c20b8052bb04f7e9 , While doing the fix for ewaybill, now it's causing issue for `E-Invoice`, because the partner buyer details were changed from Purchase Document to Move type being a Outbound In this fix we only change Buyer-Seller Details only if it's for e-waybill 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 Forward-Port-Of: odoo/odoo#191578
Current behavior before PR: When a user mentioned a channel or another user in a message, the web notification displayed the mention URL on the recipient’s side, causing a UI issue. Before / After <div style="display: flex;"> <img src="https://github.com/user-attachments/assets/3e8d381a-5d08-4f43-89f4-382d7b9bcca1" width="48%" style="margin-right: 4%;" /> <img src="https://github.com/user-attachments/assets/9abb168c-fe7d-4828-b55f-140219deb003" width="48%" /> </div> Desired beha
Original PR description
Current behavior before PR: When a user mentioned a channel or another user in a message, the web notification displayed the mention URL on the recipient’s side, causing a UI issue. Before / After <div style="display: flex;"> <img src="https://github.com/user-attachments/assets/3e8d381a-5d08-4f43-89f4-382d7b9bcca1" width="48%" style="margin-right: 4%;" /> <img src="https://github.com/user-attachments/assets/9abb168c-fe7d-4828-b55f-140219deb003" width="48%" /> </div> Desired behavior after PR is merged: This commit resolves the issue by hiding the mention URL in web notification when a user mentions a channel or another user in a message. Task-4295310 --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-Port-Of: odoo/odoo#186744