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

[14.0][IMP] l10n_br_fiscal: add event print feature #2259

Merged
merged 1 commit into from
Dec 20, 2022

Conversation

marcelsavegnago
Copy link
Member

@marcelsavegnago marcelsavegnago commented Dec 15, 2022

Adicionando recurso para impressão de eventos do documento fiscal. No caso adicionei o botão de impressão apenas nos eventos de Carta de Correção mas poderia ser utilizado para imprimir detalhes de qualquer evento.

Obs. Se for o caso pensei em implementar isso em um módulo chamado l10n_br_fiscal_report que poderia servir inclusive para agum outro relatório e desta forma não sobrecarregar ainda mais o módulo fiscal.

image

image

@marcelsavegnago
Copy link
Member Author

Se alguém tiver interesse em aprimorar o qweb eu agradeço :D

@OCA-git-bot
Copy link
Contributor

Hi @renatonlima,
some modules you are maintaining are being modified, check this out!

@antoniospneto
Copy link
Contributor

top @marcelsavegnago

@felipemotter
Copy link
Contributor

felipemotter commented Dec 15, 2022

@marcelsavegnago Seu lindo HAHHAHA estávamos precisando

@marcelsavegnago
Copy link
Member Author

@marcelsavegnago Seu lindo HAHHAHA estávamos precisando

hehehehehe fiz o mais simples possivel. Acho que atende.

@felipemotter
Copy link
Contributor

Isso não precisa de muita firula não... Tá de ótimo tamanho.

@renatonlima
Copy link
Member

Olá @marcelsavegnago,

Legal essa feature, eu só teria duas sugestões:

  1. Sobre o modelo de impressão poderia ser assim:

image

  1. Deixar o relatório modular, no l10n_br_fiscal ter o relatório vazio e sobrescrever no módulo l10n_br_nfe (Eu mas eu diria que isso também poderia ficar para depois)

@renatonlima
Copy link
Member

Um outro modelo:

image

@rvalyi
Copy link
Member

rvalyi commented Dec 16, 2022

eu tb so queria comentar que teve uma proposta de carta de template odt/py3o na lib erpbrasil.edoc.pdf erpbrasil/erpbrasil.edoc.pdf#22
mas considerando que o py3o vai ser descontinuado, que a popularidade do erpbrasil.edoc.pdf é bem relativa e que carta de correção tem um template muito simples, eu acho overkill de ir pro py3o para isso (pro DANFE que é complexo é razoável continuar com py3o) e eu apoio a ideia de fazer isso com qweb, levando em considerações do @renatonlima

@douglascstd
Copy link
Member

[TESTES FUNCIONAIS] Não foi possível testar a funcionalidade. Pois a Carta de Correção criada no RunBoat não é registrado como documento.

@felipemotter
Copy link
Contributor

Não seria interessante um merge dessa PR e deixarmos um upgrade para depois?

@marcelsavegnago marcelsavegnago force-pushed the 14.0-imp-l10n_br_fiscal_event_print branch from 3d4e3b1 to 8d09615 Compare December 20, 2022 13:54
@marcelsavegnago
Copy link
Member Author

marcelsavegnago commented Dec 20, 2022

Por ora adicionei ids em algumas divs e coloquei condicao para imprimir ou nao a justificativa e os avisos pertinentes a documentos do tipo CCe.

image

@douglascstd
Copy link
Member

Não existe um padrão de impressão para esse documento.
A obrigatoriedade é de registro das informações no SEFAZ, e isso já é feito antes mesmo desta PR.

Acredito que esse modelo atenda a necessidade de empresas em imprimir para que a essa impressão siga juntamente com a NF também impressa e a Mercadoria.

Copy link
Member

@douglascstd douglascstd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[FUNCTIONAL TEST]
Modelo apresentado cria função importante para necessidade de usuários para impressão.
Posição do botão para função de impressão OK.
Modelo OK

@marcelsavegnago marcelsavegnago changed the title [WIP][14.0][IMP] l10n_br_fiscal: add event print feature [14.0][IMP] l10n_br_fiscal: add event print feature Dec 20, 2022
Copy link
Member

@rvalyi rvalyi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

eu acho que já ta bem limpo e que melhorias poderiam vir depois

@rvalyi
Copy link
Member

rvalyi commented Dec 20, 2022

/ocabot merge minor

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 14.0-ocabot-merge-pr-2259-by-rvalyi-bump-minor, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 00e4506 into OCA:14.0 Dec 20, 2022
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at be04468. Thanks a lot for contributing to OCA. ❤️

@marcelsavegnago marcelsavegnago deleted the 14.0-imp-l10n_br_fiscal_event_print branch December 20, 2022 22:33
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.

7 participants