Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[16.0][IMP] account_reconcile_oca: Improve _compute_reconcile_data_info. When changing account_id on journal item, data displayed in reconcilitation menu now is updated #773

Draft
wants to merge 1 commit into
base: 16.0
Choose a base branch
from

Conversation

sergiobstoj
Copy link
Member

@sergiobstoj sergiobstoj commented Dec 27, 2024

Current behaviour:

When changing account_id on move line, the account displayed in reconciliation menu is not getting updated.

before-pr.mp4

Behaviour with this PR:

account_id displayed in reconciliation menu is updated.

after-pr.mp4

@OCA-git-bot
Copy link
Contributor

Hi @etobella,
some modules you are maintaining are being modified, check this out!

@pedrobaeza pedrobaeza added this to the 16.0 milestone Dec 30, 2024
@pedrobaeza
Copy link
Member

In fact, I see that account change as something incorrect, as you are changing the liquidity account.

@sergiobstoj sergiobstoj force-pushed the 16.0-fix-_compute_reconcile_data_info branch from 1b3c5c5 to cf211f2 Compare January 3, 2025 12:55
@sergiobstoj
Copy link
Member Author

I improved new method _check_recompute_reconcile_data to make it more generic and handle some other fields that could be changed.

About the account, I understand @pedrobaeza that you say it should not allow the account change? We are having some cases where it was necessary to change it due to mistake on counterpart account.

@pedrobaeza
Copy link
Member

If you change the bank account, it's because you have added the transaction in the wrong journal, and just changing the bank account is not enough. You have to remove the transaction in that bank, and create it in the correct bank.

I still state that this base of this PR is not correct.

@sergiobstoj sergiobstoj force-pushed the 16.0-fix-_compute_reconcile_data_info branch from cf211f2 to 804d344 Compare January 3, 2025 13:08
…en changing certain fields on journal item, data displayed in reconcilitation menu is now updated
@sergiobstoj sergiobstoj force-pushed the 16.0-fix-_compute_reconcile_data_info branch from 804d344 to ee803ef Compare January 3, 2025 13:54
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