Vauxoo / addons-vauxoo

All our modules related to developments that solves generic issues on Odoo, or that solve internal problems on Odoo Core, if something is here, maybe it is solving an issue in your company, try it and report what you see.
http://www.vauxoo.com
193 stars 288 forks source link

[MIG] account_bank_reconcile_transfer: Migrate to v15.0 T#59105 #1576

Closed rolandojduartem closed 1 year ago

rolandojduartem commented 1 year ago

This module lets create the internal transfers from the statement line directly with no extra steps, reconciling their journal entries.

Changelog:

Send:

Screenshot from 2022-10-25 12-36-48

Screenshot from 2022-10-25 12-37-09

Screenshot from 2022-10-25 12-45-39

Screenshot from 2022-10-25 12-45-52

Receive:

Screenshot from 2022-10-25 12-36-31

Screenshot from 2022-10-25 13-34-27

Screenshot from 2022-10-25 12-45-06

Screenshot from 2022-10-25 12-45-17

Reference:

[1] https://github.com/odoo/odoo/commit/caeb782841fc5a7ad71a196e2c9ee67644ef9074

rolandojduartem commented 1 year ago

Hi, @tomeyro, please, may you leave me your feedback testing this PR? This PR is in draft state.

cc @luisg123v @desdelinux @nhomar

rolandojduartem commented 1 year ago

Sorry, a typo was fixed.

rolandojduartem commented 1 year ago

@tomeyro friendly reminder.

tomeyro commented 1 year ago

@rolandojduartem It correctly creates the transfer (payment in and out), but it seems the reconcile process is not fully completed.

I noticed that the payment didn't have a link to the bank statement, which appears after reconciling a payment to a bank statement line.

20221024181658

Then I reverted the reconciliation and reconciled again by selecting the payment manually.

(Note that I used that 'last year insterests' line from the demo data for the example)

20221024181803

And then the button to the statement appears on the payment.

20221024181749

Please check that. The rest of the process seems to be working as expected.

rolandojduartem commented 1 year ago

@tomeyro, I updated this PR, if you can, your feedback is very welcome. As you can see in the picture in the description, the process is working very well.

cc @luisg123v @desdelinux

tomeyro commented 1 year ago

@rolandojduartem @luisg123v LGTM

rolandojduartem commented 1 year ago

Closed because it was migrated on other repository