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][REM] l10n_br_account_payment_order, l10n_br_account_payment_brcobranca: Removendo Campos, Objetos e Visões obsoletos #3365

Merged

Conversation

mbcosta
Copy link
Contributor

@mbcosta mbcosta commented Sep 13, 2024

Remove CNAB objects fields views obsolete.

Removendo Campos, Objetos e Visões obsoletos, depende dos PRs #3243 #3360 os commits desses PRs foram incluídos aqui para poder validar o PR, depois do merge será feito o rebase e esses commits deverão sair, mesmo assim já é possível revisar o PR olhando os dois últimos commits.

Com o merge #3337 os objetos l10n_br_cnab.mov.instruction.code, l10n_br_cnab.return.move.code e l10n_br_cnab.boleto.wallet.code, suas Visões e os Campos referentes em outros objetos se tornaram obsoletos, foram mantidos apenas para permitir a migração, esse PR está removendo.

O script de migração deve atualizar os XMLs IDs para o novo objeto l10n_br_cnab.code e remover os objetos l10n_br_cnab.mov.instruction.code, l10n_br_cnab.return.move.code e l10n_br_cnab.boleto.wallet.code.

IMPORTANTE: Não atualize um Banco de Dados em Produção diretamente crie um novo Banco para Homologação em caso de algum erro por favor veja de colocar aqui o LOG com a mensagem.

Na atualização dos Bancos de Dados recomendo rodar uma versão por vez para acompanhar se tudo ocorre sem erros

Aproveitando essa "limpeza" estou considerando remover algumas Constantes que pelo o que vi com o grep não estão sendo utilizados:

Aqui seria importante um retorno do @mileo e @antoniospneto para saber se isso está sendo usado de alguma forma em algum outro modulo.

Outro ponto não foi possível remover o campo cnab_processor do l10n_br_account_payment_order/models/account_payment_mode.py por estar sendo usado pelo modulo l10n_br_cnab_structure é possível mante-lo para aguardar essa refatoração porém com o PR do CNAB Config o campo já vai deixar de aparecer na Visão do Modo de Pagamento, então é preciso avaliar essas alteração no modulo.

cc @rvalyi @renatonlima @marcelsavegnago

@OCA-git-bot
Copy link
Contributor

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

@mbcosta mbcosta force-pushed the 14.0-REM-cnab_objects_fields_views_obsolete branch from 40fb0c5 to 4dd4b95 Compare September 18, 2024 17:55
@mbcosta mbcosta force-pushed the 14.0-REM-cnab_objects_fields_views_obsolete branch from 4dd4b95 to 2bc5db9 Compare September 19, 2024 17:46
@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@rvalyi
Copy link
Member

rvalyi commented Sep 23, 2024

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

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

@OCA-git-bot OCA-git-bot merged commit 01e8327 into OCA:14.0 Sep 23, 2024
7 checks passed
@OCA-git-bot
Copy link
Contributor

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

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.

6 participants