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] account_internal_transfer: Compute partner_id for internal transfer #637

Closed
wants to merge 1 commit into from

Conversation

rov-adhoc
Copy link
Contributor

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

This commit adds a computed method _compute_partner_id that ensures
internal transfers are assigned the partner from journal_id.company_id.partner_id.
This avoids missing partner values in internal transfers.

@roboadhoc
Copy link

@roboadhoc
Copy link

Nothing to update (base, title, body). No update to pr head.

@rov-adhoc rov-adhoc force-pushed the 18.0-h-86054-rov branch 5 times, most recently from 950d5a7 to dd78935 Compare January 22, 2025 13:15
@rov-adhoc rov-adhoc force-pushed the 18.0-h-86054-rov branch 11 times, most recently from 7c0181b to 8d21820 Compare January 30, 2025 13:39
@rov-adhoc rov-adhoc changed the title [FIX] account_internal_transfer: Fix internal transfer third party checks [FIX] account_internal_transfer: Compute partner_id for internal transfer Jan 30, 2025
…sfers

This commit adds a computed method _compute_partner_id that ensures
internal transfers are assigned the partner from journal_id.company_id.partner_id.
This avoids missing partner values in internal transfers.
Copy link
Contributor

@zaoral zaoral left a comment

Choose a reason for hiding this comment

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

roboadhoc pushed a commit to ingadhoc/account-payment that referenced this pull request Feb 3, 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.

closes #607

Related: ingadhoc/account-financial-tools#637
Signed-off-by: Katherine Zaoral <[email protected]>
@roboadhoc roboadhoc closed this in 2df4be1 Feb 3, 2025
@roboadhoc roboadhoc added the 17.1 label Feb 3, 2025
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.

3 participants