Skip to content

Conversation

@AungKoKoLin1997
Copy link
Contributor

@AungKoKoLin1997 AungKoKoLin1997 commented Dec 22, 2025

This module adds a bank account field to partners and sets it as the Recipient Bank on account moves based on the configuration of the invoice’s company.

@qrtl QT6258

@AungKoKoLin1997 AungKoKoLin1997 force-pushed the 18.0-add-account_move_parnter_bank_by_customer branch 2 times, most recently from 13db893 to 0de7f7f Compare December 24, 2025 09:57
@AungKoKoLin1997 AungKoKoLin1997 changed the title [18.0][ADD] account_move_partner_bank_by_customer [18.0][ADD] account_move_partner_bank_source Dec 24, 2025
@AungKoKoLin1997 AungKoKoLin1997 force-pushed the 18.0-add-account_move_parnter_bank_by_customer branch from 0de7f7f to 9cebbed Compare December 24, 2025 10:18
Copy link
Member

@yostashiro yostashiro left a comment

Choose a reason for hiding this comment

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

Partial review.

@AungKoKoLin1997 AungKoKoLin1997 force-pushed the 18.0-add-account_move_parnter_bank_by_customer branch 2 times, most recently from e563958 to 8431acd Compare December 25, 2025 07:34
@AungKoKoLin1997 AungKoKoLin1997 force-pushed the 18.0-add-account_move_parnter_bank_by_customer branch from 8431acd to cc08872 Compare December 26, 2025 07:01
@AungKoKoLin1997 AungKoKoLin1997 changed the title [18.0][ADD] account_move_partner_bank_source [18.0][ADD] account_move_partner_bank Dec 26, 2025
@yostashiro yostashiro force-pushed the 18.0-add-account_move_parnter_bank_by_customer branch from 6f4400e to cda8983 Compare December 27, 2025 13:05
Copy link
Member

@yostashiro yostashiro left a comment

Choose a reason for hiding this comment

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

@AungKoKoLin1997 Please review my last update and squash commits. Otherwise, the design should be good now.

@yostashiro yostashiro force-pushed the 18.0-add-account_move_parnter_bank_by_customer branch from cda8983 to 15da6a8 Compare December 27, 2025 15:02
Copy link

@hoangtrann hoangtrann left a comment

Choose a reason for hiding this comment

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

Code LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants