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

[17.0][FIX] l10n_es_aeat_mod303: Remove field 96 #3929

Merged

Conversation

pedrobaeza
Copy link
Member

@pedrobaeza pedrobaeza commented Jan 17, 2025

Forward-port + adaptation of #3924 and #3927

It was incorrectly mapped to "recargo de equivalencia" base amounts, which is not correct, as that base amounts are already included in regular operations, so it appears duplicated.

This field is for being filled in specific cases where the subject is on the special fiscal regime "Recargo de equivalencia".

The tests have been changed, as now field 88 doesn't contain this field.

@Tecnativa TT52996

@pedrobaeza pedrobaeza added this to the 17.0 milestone Jan 17, 2025
It was incorrectly mapped to "recargo de equivalencia" base amounts,
which is not correct, as that base amounts are already included in
regular operations, so it appears duplicated.

This field is for being filled in specific cases where the subject is
on the special fiscal regime "Recargo de equivalencia".

The tests have been changed, as now field 88 doesn't contain this field.

TT52996
@pedrobaeza pedrobaeza force-pushed the 17.0-fix-l10n_es_aeat_mod303-field_96 branch from 8a5d1ff to 7338bec Compare January 17, 2025 15:40
@pedrobaeza
Copy link
Member Author

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 17.0-ocabot-merge-pr-3929-by-pedrobaeza-bump-patch, awaiting test results.

@OCA-git-bot
Copy link
Contributor

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

@OCA-git-bot OCA-git-bot merged commit 5599cc8 into OCA:17.0 Jan 17, 2025
5 checks passed
@pedrobaeza pedrobaeza deleted the 17.0-fix-l10n_es_aeat_mod303-field_96 branch January 17, 2025 16:09
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