Skip to content

Conversation

shgabr
Copy link
Contributor

@shgabr shgabr commented Oct 6, 2025

This commit adds 1 new accounts 201205 and modifies existing 201200 account to be used as the tax receivable and tax payable accounts in Estonia localization.

task-5144484

Current behavior before PR:
The same account is set for the closing VAT Payable and Receivable.

Desired behavior after PR is merged:
New accounts modified and added to the VAT payable and receivable accounts.


I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

@robodoo
Copy link
Contributor

robodoo commented Oct 6, 2025

Pull request status dashboard

@shgabr shgabr requested review from a team and FlorianGilbert and removed request for a team October 13, 2025 09:22
Copy link
Contributor

@FlorianGilbert FlorianGilbert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey, we'll need an upgrade script for these changes :)

@shgabr shgabr force-pushed the master-ee_tax_accounts-sgab branch 2 times, most recently from 19a2e2c to 54f28cf Compare October 15, 2025 10:27
@shgabr shgabr requested a review from FlorianGilbert October 15, 2025 12:33
Copy link
Contributor

@FlorianGilbert FlorianGilbert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@robodoo r+ delegate+

@robodoo
Copy link
Contributor

robodoo commented Oct 15, 2025

@shgabr @FlorianGilbert linked pull request(s) odoo/upgrade#8634 not ready. Linked PRs are not staged until all of them are ready.

This commit adds 1 new accounts 201205 and modifies existing
201200 account to be used as the tax receivable and tax payable
accounts in Estonia localization.

task-5144484
@shgabr shgabr force-pushed the master-ee_tax_accounts-sgab branch from 54f28cf to e587406 Compare October 21, 2025 09:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants