Skip to content

Conversation

@AaronHForgeFlow
Copy link
Contributor

No description provided.

Copy link
Contributor

@JordiMForgeFlow JordiMForgeFlow left a comment

Choose a reason for hiding this comment

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

we could add a test to cover this new case

@AaronHForgeFlow AaronHForgeFlow force-pushed the 18.0-imp-account_move_line_purchase_info-pdiff branch from 84b9150 to 8e5a0a1 Compare December 23, 2025 14:10
@AaronHForgeFlow
Copy link
Contributor Author

Tests added 😇 BTW forward port is not needed after stock valuation apocalypse done in v19

pdiff_amls = self.aml_model.search(
[
("move_id", "=", invoice.id),
("account_id", "=", self.account_grni.id),
Copy link
Contributor

Choose a reason for hiding this comment

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

shouldn't it be the price diff account?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, an actually, it is not working for some reason I need to investigate. Thanks.

@AaronHForgeFlow AaronHForgeFlow force-pushed the 18.0-imp-account_move_line_purchase_info-pdiff branch from 8e5a0a1 to 8743f50 Compare December 23, 2025 15:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants