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

[16.0][REF] l10n_br_account_payment_order: Na v16 é possível usar campos tipo Many2Many no Agrupar Por/Group By #3594

Merged

Conversation

mbcosta
Copy link
Contributor

@mbcosta mbcosta commented Jan 22, 2025

Group By M2M.

Na v16 é possível usar campos tipo Many2Many no Agrupar Por/Group By, PR simples que resolve um ROADMAP e reduz código, a Revisão pode ser feita abrindo a tela dos Códigos CNAB

image

Apesar de ter apenas removido uma linha do ROADMAP https://github.com/OCA/l10n-brazil/blob/16.0/l10n_br_account_payment_order/readme/ROADMAP.md

image

o pre-commit acabou alterando outros pontos.

Mantive comentado o bank_id apenas para uma futura confirmação se existe algum caso onde mais de um Banco usam o mesmo conjunto de Códigos, talvez seja algo remoto mas se existir isso evitaria ter que duplicar Códigos no Data, talvez depois de incluir pelo menos os Códigos dos Bancos existentes dento do BRCobranca teremos uma quantidade maior de casos para avaliar.

Um outro ponto estou considerando remover o objeto l10n_br_cnab_data_abstract.py isso foi feito para buscar diminuir código duplicado antes de juntar os Códigos CNAB mas agora está sem uso, mantive para ser possível usar em outras implementações do CNAB mas se não houver uso isso poderá ser removido, por isso caso algum outro desenvolvedor esteja vendo de usar isso em algum módulo que já existe ou que se pretende incluir será importante informar sobre isso para que o objeto seja mantido.

cc @OCA/local-brazil-maintainers

@mbcosta mbcosta force-pushed the 16.0-REF-pay_order_possibily_group_by_many2many branch from f733c3f to 3a26f3b Compare January 22, 2025 19:48
antoniospneto
antoniospneto approved these changes Jan 23, 2025
@rvalyi
Copy link
Member

rvalyi commented Jan 24, 2025

/ocabot merge minor

@OCA-git-bot
Copy link
Contributor

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

@OCA-git-bot
Copy link
Contributor

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

@OCA-git-bot OCA-git-bot merged commit b260d20 into OCA:16.0 Jan 24, 2025
7 checks passed
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.

5 participants