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

Nota fiscal de servico: merge do modulo base_nfse #392

Closed
rvalyi opened this issue Aug 21, 2016 · 5 comments
Closed

Nota fiscal de servico: merge do modulo base_nfse #392

rvalyi opened this issue Aug 21, 2016 · 5 comments

Comments

@rvalyi
Copy link
Member

rvalyi commented Aug 21, 2016

Ao nivel da OCA, podemos melhorar as abstracoes para a emissao das notas fiscais de servico. Especialmente podemos integrar o trabalho do @danimaribeiro no modulo base_nfse que esta hoje proposto no merge no repo odoo-brazil/odoo-brazil-eletronic-documents:
odoo-brazil/odoo-brazil-eletronic-documents#105

Esse nodulo base_nfse nao e muito complexo e nao tem nehnuma dependencia problematica (ao contrario dos modulos que dependem do pysped la) entao realmente e possivel integrar esse modulo nesse repo da OCA.

A ideia porem e de deixar os templates de mapping das notas dos municipios em repos satelites (pode ser um so) onde a burocracia e menor e para deixar o projeto OCA bem struturado por camadas.

Precisamos entender exactemente porque o base_nfse depende do modulo l10_br_account_product e se poderiamos refatorar umas coisas no l10_br_account para nao ter esse dependencia. Caso matar essa dependencia, poderiamos considerar de fazer um merge do modulo base_nfse dentro do modulo l10n_br_account_service ja que nao teria justificativa do l10n_br_account_service existir sem essas funcionalidades.

Antes de proceder com esse merge, temos porem que finalizar esse outro do @renatonlima que estrutura as visoes e menus de documentos fiscais de forma muito mais conveniente para extender alem: #363 O Renato deve estar terminando isso ja ja...

@mileo
Copy link
Member

mileo commented Aug 22, 2016

Tem que tomar um pouco de cuidado com a NFS-E do DF, pois ela usa o webservice da NF-E.

@rvalyi
Copy link
Member Author

rvalyi commented Aug 23, 2016

Tambem so avisando que isso nao vai ser uma prioridade grande aqui na Akretion entao @mileo ou @danimar se voces querem assumir essa tarefa, podem ficar a vontade de propor o PR (acho que teria que extrair o base_nfse do outro repo preservando o historico, fazendo um rebase nesse repo e talvez depois entao comecar um merge com os arquivos do l10n_br_account_service caso seja pertinente).

@mileo
Copy link
Member

mileo commented Aug 23, 2016

Te dizer que estou de saco cheio de emitir nfs-e na mão. Devo ver isso em breve.

@danimaribeiro
Copy link
Contributor

Estou no momento implementando a NFSe de São Paulo e estou aproveitando o momento para fazer um refactor desse nfse_base e dos outros módulos.
Estou jogando a parte de gerar para uma bilbioteca externa.

A paulistana por exemplo ficou com um código bem enxuto e fácil de dar manutenção.
https://github.com/danimaribeiro/PyTrustNFe/blob/master/pytrustnfe/nfse/paulistana/__init__.py
Porém falta alguns testes ainda.

Vou precisar mais umas três semanas para finalizar ainda.

@rvalyi
Copy link
Member Author

rvalyi commented Aug 23, 2016

@danimaribeiro sem problema, valeu pelo trabalho!

@mileo mileo closed this as completed Apr 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants