Skip to content

[17.0][MIG] l10n_br_fiscal: Migration to 17.0 #3740

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

Draft
wants to merge 2,002 commits into
base: 17.0
Choose a base branch
from

Conversation

marcelsavegnago
Copy link
Member

No description provided.

renatonlima and others added 30 commits April 24, 2025 12:40
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: l10n-brazil-16.0/l10n-brazil-16.0-l10n_br_fiscal
Translate-URL: https://translation.odoo-community.org/projects/l10n-brazil-16-0/l10n-brazil-16-0-l10n_br_fiscal/
Currently translated at 91.5% (1237 of 1351 strings)

Translation: l10n-brazil-16.0/l10n-brazil-16.0-l10n_br_fiscal
Translate-URL: https://translation.odoo-community.org/projects/l10n-brazil-16-0/l10n-brazil-16-0-l10n_br_fiscal/pt_BR/
changed a few names/methods xml -> file (could be XML or json)
moved a few methods/UI from l10n_br_nfe so they are shared
pluggable binding type detection
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: l10n-brazil-16.0/l10n-brazil-16.0-l10n_br_fiscal
Translate-URL: https://translation.odoo-community.org/projects/l10n-brazil-16-0/l10n-brazil-16-0-l10n_br_fiscal/
Some modules of the repo depend on stock and have
demo products of type 'product' (this type is added to
product.template in the stock module).
For some reason when running the tests, some inverse method fields
then fail when reading 'product' value for the product type.
It seems it is because l10n_br_fiscal doesn't depend on stock.
But we don't want such a dependency.
So to avoid the bug we add the 'product' value to the selection
which seems like an acceptable workaround.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: l10n-brazil-16.0/l10n-brazil-16.0-l10n_br_fiscal
Translate-URL: https://translation.odoo-community.org/projects/l10n-brazil-16-0/l10n-brazil-16-0-l10n_br_fiscal/
Revert "[FIX] restricted fields were required in domains"
This reverts commit 173a0f7.

As per OCA migration guide:
you should check if the fields protected by a group interact in some way with business logic or UI. For example, a field being part of a domain of another field, or being used in copy/defaults. An special case is the field company_id, which is always used internally for domains on many2one fields referencing multi-company aware records. On these cases, you have to add the same field twice, one with the group, and another invisible without it. Check core examples in odoo/odoo#95729.
@marcelsavegnago marcelsavegnago force-pushed the 17.0-mig-l10n_br_fiscal branch 2 times, most recently from 3ba31bb to 14ddcb5 Compare April 24, 2025 16:24
@marcelsavegnago marcelsavegnago force-pushed the 17.0-mig-l10n_br_fiscal branch from 308ea06 to 5651853 Compare April 24, 2025 17:32
@marcelsavegnago marcelsavegnago force-pushed the 17.0-mig-l10n_br_fiscal branch from 19d0949 to dc299d7 Compare April 25, 2025 12:07
@marcelsavegnago marcelsavegnago force-pushed the 17.0-mig-l10n_br_fiscal branch from bc4b6a8 to 9115fb4 Compare April 25, 2025 17:40
@marcelsavegnago marcelsavegnago force-pushed the 17.0-mig-l10n_br_fiscal branch from 9115fb4 to 7548a47 Compare April 25, 2025 17:59
@marcelsavegnago marcelsavegnago force-pushed the 17.0-mig-l10n_br_fiscal branch from 6e8dea3 to 51d4557 Compare April 25, 2025 20:40
@marcelsavegnago marcelsavegnago force-pushed the 17.0-mig-l10n_br_fiscal branch from 51d4557 to 9a5a484 Compare April 25, 2025 20:49
@marcelsavegnago marcelsavegnago force-pushed the 17.0-mig-l10n_br_fiscal branch 3 times, most recently from c7a3a5e to 25649aa Compare April 28, 2025 14:44
@marcelsavegnago marcelsavegnago force-pushed the 17.0-mig-l10n_br_fiscal branch from 25649aa to 12744df Compare April 28, 2025 14:53
@marcelsavegnago marcelsavegnago force-pushed the 17.0-mig-l10n_br_fiscal branch from 07ecb46 to 86b2368 Compare April 28, 2025 19:35
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.