[16.0][MIG] l10n_es_aeat_mod349_special_prorate: Migration to 16.0 - #956
Closed
eantones wants to merge 2 commits into
Closed
[16.0][MIG] l10n_es_aeat_mod349_special_prorate: Migration to 16.0#956eantones wants to merge 2 commits into
eantones wants to merge 2 commits into
Conversation
Bridge module wiring the special prorate purchase taxes into the AEAT 349 model tax map (keys A and I). It is the missing 349 counterpart of l10n_es_aeat_mod303_special_prorate and l10n_es_aeat_sii_oca_special_prorate: l10n_es_special_prorate cannot declare the 349 membership itself without depending on l10n_es_aeat_mod349, so the wiring belongs in this auto_install glue module, installed whenever both sides are present. (cherry picked from commit 21bb17c)
Straight carbon of the 14.0 module: the aeat.349.map.line model and its tax_tmpl_ids one2many are identical on 16.0, so the data file ports verbatim. Adaptations: version, author normalized to the repository convention, README artifacts regenerated for the 16.0 branch.
eantones
added
the
no merge before Oxigen 18
Member
Author
|
Closing: the bridge cannot exist on 16.0 — its l10n_es_special_prorate dependency has never been migrated to 16.0 (no code on the branch, no PR ever opened against 16.0), so neither CI nor a 16.0 database can load this module. The bridge lives where both sides exist: 14.0 (#954) and 18.0 (#957). If the prorate family ever reaches 16.0, this can be rebuilt carrying the full history. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Carbon of the 14.0 module (#954): wires the special prorate purchase taxes into the AEAT 349 map lines A and I, the missing 349 sibling of the 303/SII prorate bridges. The aeat.349.map.line model and its tax_tmpl_ids one2many are identical on 16.0, so the data file ports verbatim; the migration commit only normalizes version and author and regenerates the README artifacts.
Data-only and auto_install: it wires itself whenever both l10n_es_special_prorate and l10n_es_aeat_mod349 are installed, in any order.
Do not merge until Oxigen is live in production on 18.0: the 14→18 migration chain consumes this branch via refs/pull.