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.

Sign up for GitHub

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

Jump to bottom

[14.0][IMP] l10n_br_mdfe: add damdfe report #3506

Merged
merged 4 commits into from
Nov 27, 2024
Merged

[14.0][IMP] l10n_br_mdfe: add damdfe report #3506

merged 4 commits into from
Nov 27, 2024

Conversation

marcelsavegnago
Copy link
Member

No description provided.

@OCA-git-bot
Copy link
Contributor

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

@marcelsavegnago marcelsavegnago force-pushed the l10n_br_mdfe-add-report branch 7 times, most recently from e62479e to f213726 Compare November 26, 2024 14:18
@marcelsavegnago marcelsavegnago marked this pull request as ready for review November 26, 2024 14:40
@antoniospneto
Copy link
Contributor

@marcelsavegnago eu não entendi o porque da criação do relatório qweb no arquivo l10n_br_mdfe/report/damdfe_mdfe.xml
sendo que o relatório da DAMDFE tá sendo gerado com o brazilfiscalreport

@marcelsavegnago
Copy link
Member Author

@marcelsavegnago eu não entendi o porque da criação do relatório qweb no arquivo l10n_br_mdfe/report/damdfe_mdfe.xml sendo que o relatório da DAMDFE tá sendo gerado com o brazilfiscalreport

Ops.. de fato.. eu fui removendo as coisas relacionadas com a lib e estava pensando mesmo nisso.. vou dar uma olhada.. vlw

antoniospneto
antoniospneto requested changes Nov 26, 2024
View reviewed changes
Copy link
Contributor

@antoniospneto antoniospneto left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Verificar comentários, a logica foi copiada da nfe, mas aqui não precisamos da parte que trata o erpbrasil, tbm foi copiado o código que era pra impressão da nfce que tbm não tem necessaidade.

@marcelsavegnago
Copy link
Member Author

Verificar comentários, a logica foi copiada da nfe, mas aqui não precisamos da parte que trata o erpbrasil, tbm foi copiado o código que era pra impressão da nfce que tbm não tem necessaidade.

@antoniospneto @CristianoMafraJunior acredito ter feito a remoção de tudo o que precisava. Se puderem revisar novamente eu agradeço.

rvalyi
rvalyi approved these changes Nov 27, 2024
View reviewed changes
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.

LGTM. Parabéns pelo trabalho!

2024-11-27_11-53

@marcelsavegnago
Copy link
Member Author

Vou adicionar o Neto como contribuinte deste modulo e do cte.. em minutos to mandando os commits

@marcelsavegnago
Copy link
Member Author

@antoniospneto parabens pela Lib .. ajudou bastante.. Ainda vamos precisar complementar informacoes tanto no DACTE como no DAMDFE mas agora ta fácil.. muito obrigado pela ajuda.

@marcelsavegnago
Copy link
Member Author

Vou adicionar o Neto como contribuinte deste modulo e do cte.. em minutos to mandando os commits

Feito

@rvalyi
Copy link
Member

rvalyi commented Nov 27, 2024

Vou adicionar o Neto como contribuinte deste modulo e do cte.. em minutos to mandando os commits

Feito

legit

@rvalyi
Copy link
Member

rvalyi commented Nov 27, 2024

/ocabot merge minor

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 14.0-ocabot-merge-pr-3506-by-rvalyi-bump-minor, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit e94de96 into OCA:14.0 Nov 27, 2024
6 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

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

@marcelsavegnago marcelsavegnago deleted the l10n_br_mdfe-add-report branch November 27, 2024 20:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@CristianoMafraJunior CristianoMafraJunior CristianoMafraJunior approved these changes

@antoniospneto antoniospneto antoniospneto approved these changes

@rvalyi rvalyi rvalyi approved these changes

@mileo mileo Awaiting requested review from mileo

Assignees
No one assigned
Labels
approved merged 🎉
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@marcelsavegnago @OCA-git-bot @antoniospneto @rvalyi @CristianoMafraJunior