Tuesday, March 14, 2023
5 changes · master
Resolved issues and error corrections
The German SKR03 accounting template now classifies account 1380 correctly in the balance sheet. This helps German companies rely on more accurate financial reporting and chart of accounts setup.
Original PR description
By checking the script checking if all the account supposed to be in the BS are. The account 1380 seems to be miss configured task-id: 3041738 --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr
The Expense app was missing a required link to another component, which prevented businesses from installing it by itself. This fix adds the missing requirement so the app can be installed independently as expected.
Original PR description
Before this commit, it was not possible to install hr_expense alone. This commit enable the installation of standalone hr_expense by adding missing dependency. --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr
This fixes an internal web test helper that could get stuck when a component was removed after an error. The change makes automated tests more reliable, helping developers catch issues without affecting regular users.
Original PR description
The `afternextrender` helper is meant to be used in some tests that want to do something after owl has completed all its work following a specific action. It is useful when the action may cause some…
The `afternextrender` helper is meant to be used in some tests that want to do something after owl has completed all its work following a specific action. It is useful when the action may cause some additional renders, so waiting for one nextTick is not enough. Unfortunately, it has a bug: it waits for Owl scheduler to be empty. But in some cases, when a component is destroyed because of an error, the scheduler task list will not be cleared until the next render completes. It is implemented that way in Owl to reduce the amount of work it does. But this means that in some testing scenarios, the afterNextRender promise will simply not be resolved, since there is no upcoming render to clear the tasklist. With this commit, we just avoid the issue by ignoring the fibers coming from destroyed component. Note that I think that it is not a good semantic for tests, as it is a little bit too "black-boxy". Ideally, we should rather wait for something in the DOM. There is a waitUntil helper somewhere that may be useful for this situation 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
The Mongolian balance sheet report has been updated to include accounts that were previously missing. This helps ensure the balance sheet balances correctly and provides more reliable financial reporting for Mongolian localization users.
Original PR description
When checking if the BS is balanced with a script, it seems that some accounts were missing from the BS. This PR correct that task-id: 3041738
The German reporting setup now includes accounts that were previously present in the chart of accounts but missing from the balance sheet report. This helps ensure balance sheet reporting is more complete and accurate for German accounting users.
Original PR description
By checking the script checking if all the account supposed to be in the BS are. Seems some account were present in the COA but not in the Balance sheet, this PR corrects that. task-id: 3041738