[14.0][MIG] l10n_br_pos + NFC-e

[mileo]

• [x] Add POS default accounts: https://github.com/OCA/l10n-brazil/pull/2099
• [x] Make module installable;
• [x] Port Javascript to OWL;
• [x] Extract CF-E / SAT in a new module;
• [ ] Add Tests;

[oca-clabot]

Hey @mileo, thank you for your Pull Request.

It looks like some users haven't signed our Contributor License Agreement, yet. You can read and sign our full Contributor License Agreement here: https://odoo-community.org/page/cla Here is a list of the users:

• Atilla Silva [email protected] (no github login found)

Appreciation of efforts, OCA CLAbot

[marcelsavegnago]

Pessoal. Entendo que esta PR não se trata de uma migração e sim de inclusão de um novo módulo na localização (pelo menos eu nao localizei este módulo em nenhuma versão) e portanto recomendo a alteração do título da PR. Fora isso, entendo que se faz necessário alguns squashs dos commits.

[marcelsavegnago]

Hey @mileo, thank you for your Pull Request.

It looks like some users haven't signed our Contributor License Agreement, yet. You can read and sign our full Contributor License Agreement here: https://odoo-community.org/page/cla Here is a list of the users:

• Atilla Silva [email protected] (no github login found)

Appreciation of efforts, OCA CLAbot

@mileo pode avaliar isso e providenciar a assinatura da CLA ?

[rvalyi]

outra coisa, pode ser legal testar os vários módulos nessa PR que junta os módulos, mas depois para realmente rolar um merge seria interessante separar um PR apenas do l10n_br_pos para começar porque senão fica complicado mesmo para revisar (pode ser com git filter-branch por examplo). Lembrando que um PR pode usar um outro PR com essa tecnica https://github.com/OCA/maintainer-tools/wiki/Use-temporary-reference(s)-to-another-pull-request(s) Batendo o olho de longe o código do l10n_br_pos parece bem razoavel. Seria interessante fazer mais um pequeno esforço com os testes para não abaixar o test coverage do projeto (86% hoje).

[mileo]

Pessoal. Entendo que esta PR não se trata de uma migração e sim de inclusão de um novo módulo na localização (pelo menos eu nao localizei este módulo em nenhuma versão) e portanto recomendo a alteração do título da PR. Fora isso, entendo que se faz necessário alguns squashs dos commits.

Como já usávamos o módulo desde a v8 pra gente acaba sendo um migração.

@ygcarvalh realizou o squash dos commits possíveis.

[mileo]

outra coisa, pode ser legal testar os vários módulos nessa PR que junta os módulos, mas depois para realmente rolar um merge seria interessante separar um PR apenas do l10n_br_pos para começar porque senão fica complicado mesmo para revisar (pode ser com git filter-branch por examplo). Lembrando que um PR pode usar um outro PR com essa tecnica https://github.com/OCA/maintainer-tools/wiki/Use-temporary-reference(s)-to-another-pull-request(s) Batendo o olho de longe o código do l10n_br_pos parece bem razoavel. Seria interessante fazer mais um pequeno esforço com os testes para não abaixar o test coverage do projeto (86% hoje).

Semana q vem vamos separar em vários PR, na v12 quando esse PR foi feito era apenas um módulo.

Agora dividimos ele em 3:

• BASE
• CF-E (SP e Ceará(Ainda em DEV)
• NFC-E (Ainda em DEV)

Nesse momento já estamos conseguindo emitir NFC-E via backend, já que vamos abrir os PRs relacionados e continuar o módulo de NFC-E.

[rvalyi]

eu acho que o código dessa PR já foi extraído em outros PRs onde o merge foi feito, então eu fechei esse PR...