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

[REF] l10n_it_fatturapa_pec: refactoring clean_messages_dict and avoid error if key not exist #4523

Merged
merged 1 commit into from
Jan 10, 2025

Conversation

micheledic
Copy link
Contributor

This PR is a simple refactoring of function and we avoid the error if some key does not exist

Copy link
Contributor

@robyf70 robyf70 left a comment

Choose a reason for hiding this comment

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

LGTM

@francesco-ooops
Copy link
Contributor

@micheledic puoi aprire una issue per tracciamento? grazie!

@micheledic
Copy link
Contributor Author

@micheledic puoi aprire una issue per tracciamento? grazie!

potrebbe andar bene questa issue in cui viene segnalato proprio il problema della chiave non esistente su versioni vecchie #4521

@micheledic micheledic force-pushed the 16.0_ref_clean_message_dict branch from 3bb917a to 4d4c128 Compare January 8, 2025 10:24
@tafaRU
Copy link
Member

tafaRU commented Jan 10, 2025

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

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

@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 15dedcf. 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.

KeyError: 'x_odoo_message_id' in l10n_it_fatturapa_pec
6 participants