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

[FIX] l10n_latam_check_ux: Fix internal transfer third party checks #607

Open
wants to merge 1 commit into
base: 18.0
Choose a base branch
from

Conversation

rov-adhoc
Copy link
Contributor

@rov-adhoc rov-adhoc commented Jan 28, 2025

This commit implements the necessary corrections to ensure
that the deposit of third-party checks, through the wizard,
behaves the same way as an internal transfer.
This ensures consistency in the process and prevents validation discrepancies.

@roboadhoc
Copy link
Contributor

@rov-adhoc rov-adhoc force-pushed the 18.0-h-86054-rov branch 3 times, most recently from ba9467c to 60c6f56 Compare January 28, 2025 19:44


def _create_payments(self):
outbound_payment = super(L10nLatamPaymentMassTransfer,self.with_context(
Copy link
Contributor

Choose a reason for hiding this comment

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

add docstring to the method

l10n_latam_check_ux/__manifest__.py Outdated Show resolved Hide resolved
super(AccountPayment, rec.with_context(
default_l10n_latam_move_check_ids=rec.l10n_latam_move_check_ids,
))._create_paired_internal_transfer_payment()
super(AccountPayment, self - third_party_checks)._create_paired_internal_transfer_payment()
Copy link
Contributor

Choose a reason for hiding this comment

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

this super should be inside or after the if of 31 line?

@rov-adhoc rov-adhoc force-pushed the 18.0-h-86054-rov branch 2 times, most recently from 254d16d to 9754d16 Compare January 30, 2025 12:59
@rov-adhoc rov-adhoc changed the title [IMP] l10n_latam_check_ux: remove unnecessary dependencies [FIX] l10n_latam_check_ux: Fix internal transfer third party checks Jan 30, 2025
@rov-adhoc rov-adhoc force-pushed the 18.0-h-86054-rov branch 6 times, most recently from efb542e to 52229cd Compare January 30, 2025 16:25
This commit implements the necessary corrections to ensure
that the deposit of third-party checks, through the wizard,
behaves the same way as an internal transfer.
This ensures consistency in the process and prevents validation discrepancies.
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