Skip to content

[16.0][ADD] account_edi_ubl_cii_check_total#1264

Open
sbejaoui wants to merge 1 commit intoOCA:16.0from
acsone:16.0-account_edi_ubl_cii_check_total-sbj
Open

[16.0][ADD] account_edi_ubl_cii_check_total#1264
sbejaoui wants to merge 1 commit intoOCA:16.0from
acsone:16.0-account_edi_ubl_cii_check_total-sbj

Conversation

@sbejaoui
Copy link

This addon extends the UBL invoice import process to automatically populate the suppliers check total field based on the value found in the XML file.

@sbejaoui
Copy link
Author

sbejaoui commented Dec 22, 2025

cc/ @rousseldenis , @jbaudoux, @phschmidt

This addon extends the UBL invoice import process to automatically populate the suppliers
check total field based on the value found in the XML file.
@sbejaoui sbejaoui force-pushed the 16.0-account_edi_ubl_cii_check_total-sbj branch from ea23811 to 8effa81 Compare December 22, 2025 16:31
@sbejaoui sbejaoui changed the title [ADD] account_edi_ubl_cii_check_total [16.0][ADD] account_edi_ubl_cii_check_total Dec 22, 2025
Copy link

@phschmidt phschmidt left a comment

Choose a reason for hiding this comment

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

Functional review

Copy link

@rousseldenis rousseldenis left a comment

Choose a reason for hiding this comment

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

LGTM

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

Copy link
Contributor

@jbaudoux jbaudoux left a comment

Choose a reason for hiding this comment

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

shouldn't it be auto_install ?

Copy link
Contributor

@hparfr hparfr left a comment

Choose a reason for hiding this comment

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

shouldn't it be auto_install ?

+1

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.

6 participants