Skip to content

[IMP][16.0] base_ubl: country#1229

Merged
OCA-git-bot merged 1 commit intoOCA:16.0from
jbaudoux:16-imp-base_ubl-country
Jan 12, 2026
Merged

[IMP][16.0] base_ubl: country#1229
OCA-git-bot merged 1 commit intoOCA:16.0from
jbaudoux:16-imp-base_ubl-country

Conversation

@jbaudoux
Copy link
Copy Markdown
Contributor

Fallback on the company country when the partner do not have one.

The country is a peppol required field. So always try to fill it whenever possible. If the partner do not have a country, it is very likely that it's because he is located in the same country than the company

cc @lmignon @alexis-via

@jbaudoux jbaudoux changed the title [IMP] base_ubl: country [IMP][16.0] base_ubl: country Oct 10, 2025
@jbaudoux
Copy link
Copy Markdown
Contributor Author

/ocabot merge minor

@OCA-git-bot
Copy link
Copy Markdown
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 16.0-ocabot-merge-pr-1229-by-jbaudoux-bump-minor, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Nov 13, 2025
Signed-off-by jbaudoux
@OCA-git-bot
Copy link
Copy Markdown
Contributor

@jbaudoux your merge command was aborted due to failed check(s), which you can inspect on this commit of 16.0-ocabot-merge-pr-1229-by-jbaudoux-bump-minor.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

Fallback on the company country when the partner do not have one
@jbaudoux jbaudoux force-pushed the 16-imp-base_ubl-country branch from b4b331f to 37fd706 Compare January 12, 2026 12:17
@jbaudoux
Copy link
Copy Markdown
Contributor Author

/ocabot merge minor

@OCA-git-bot
Copy link
Copy Markdown
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 16.0-ocabot-merge-pr-1229-by-jbaudoux-bump-minor, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 6ba22c3 into OCA:16.0 Jan 12, 2026
6 of 7 checks passed
@OCA-git-bot
Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at da7a21a. Thanks a lot for contributing to OCA. ❤️

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.

4 participants