Skip to content

[18.0][FIX] l10n_es_aeat_sii_oca: No error if no description - #4870

Merged
OCA-git-bot merged 1 commit into
OCA:18.0from
Tecnativa:18.0-fix-l10n_es_aeat_sii_oca-no_description
Feb 26, 2026
Merged

[18.0][FIX] l10n_es_aeat_sii_oca: No error if no description#4870
OCA-git-bot merged 1 commit into
OCA:18.0from
Tecnativa:18.0-fix-l10n_es_aeat_sii_oca-no_description

Conversation

@pedrobaeza

Copy link
Copy Markdown
Member

When initializing an invoice, there are no descriptions, and unidecode gives an error, so let's avoid putting the conditional.

Follow-up of #4865

@Tecnativa

When initializing an invoice, there are no descriptions, and unidecode
gives an error, so let's avoid putting the conditional.

Follow-up of OCA#4865
@pedrobaeza pedrobaeza added this to the 18.0 milestone Feb 26, 2026
@pedrobaeza

Copy link
Copy Markdown
Member Author

Fusión directa para evitar el error:

/ocabot merge patch

@OCA-git-bot

Copy link
Copy Markdown
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 18.0-ocabot-merge-pr-4870-by-pedrobaeza-bump-patch, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Feb 26, 2026
Signed-off-by pedrobaeza
@OCA-git-bot

Copy link
Copy Markdown
Contributor

It looks like something changed on 18.0 in the meantime.
Let me try again (no action is required from you).
Prepared branch 18.0-ocabot-merge-pr-4870-by-pedrobaeza-bump-patch, awaiting test results.

@OCA-git-bot
OCA-git-bot merged commit 89b081e into OCA:18.0 Feb 26, 2026
6 of 7 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

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

@pedrobaeza
pedrobaeza deleted the 18.0-fix-l10n_es_aeat_sii_oca-no_description branch February 26, 2026 16:16
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