-
-
Notifications
You must be signed in to change notification settings - Fork 251
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
[14.0][ADD][l10n_br_sped_base] SPED BASE MODULE #3100
Conversation
9708040
to
a79d9e9
Compare
[ADD] extracted l10n_br_sped_ecd module [ADD] extracted l10n_br_sped_efd_icms_ipi module [ADD] extracted l10n_br_sped_efd_pis_cofins module [ADD] extracted l10n_br_sped_ecf module [FIX] back to simple SPED layout version support
saves a bunch of N+1 queries by using browse to prefetch the register children.
a79d9e9
to
d93d79c
Compare
d93d79c
to
2729ffe
Compare
hum vou tentar dar um coverage no metodo _pull_records_from_odoo que ainda não tem teste aqui... |
Fiz um rebase no que estavamos fazendo e abri um PR akretion#218 com uma correção que sobrou. |
[Fix] model search
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
Finalmente eu pude dar uma boa limpa no modulo l10n_br_sped_base que eu tinha mandado em num PR de rascunho em 2023 (se alguem jà começou a usar em produção deve ser facil adaptar; a estrutura de dados mudou muito pouco).
A seguir, vou subir os PRs dos modulos para cada arquivo de SPED que eu adaptei tb... (esses continuam com mapping bem partial porem; status alpha; para completar...)
OBS: mesmo se o nosso foco na Akretion já é a v16, é fato que algumas empresas vão querer fazer o SPED ou partes do SPED na v14. E na nossa avaliação vai ser tranquilo fazer os cherry-picks dos mappings entre as versões 14.0 e 16.0 então é vantagem ter os modulos de SPED ja na v14 e contar com as contribuições destas empresas pros mappings.