search

OCA / l10n-brazil

Localização brasileira oficial do Odoo.
https://odoo-community.org/psc-teams/brazil-66
GNU Affero General Public License v3.0
235 stars 244 forks source link

[14.0][UPD] l10n_br_zip: cep_ws_provide default #3170

Closed Tiago370 closed 1 month ago

OCA-git-bot commented 1 month ago

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

antoniospneto commented 1 month ago

@Tiago370 por que a alteração?

Tiago370 commented 1 month ago

@antoniospneto o provider "correios" tá obsoleto, a lib recusa requisição para esse provider.

antoniospneto commented 1 month ago

Obrigado pela explicação @Tiago370

Será que já não vale a pena remover ele dessa lista também?

https://github.com/OCA/l10n-brazil/blob/f1198c51e4bc7050c55c0efbff56db97f4855649/l10n_br_zip/models/res_config_settings.py#L16-L20

antoniospneto commented 1 month ago

@Tiago370,

Obrigado pela contribuição.

Poderia, por favor, adicionar um script de migração para substituir o valor removido pelo novo valor padrão nos registros afetados? Isso ajudará a garantir a integridade dos dados.

Agradeço pela atenção.

Tiago370 commented 1 month ago

@antoniospneto a próxima versão seria 14.0.3.0.0?

antoniospneto commented 1 month ago

@antoniospneto a próxima versão seria 14.0.3.0.0?

nesse caso acho que pode ser o minor do módulo, ficando assim: 14.0.2.2.0 ai vc precisa já alterar no manifest pra essa versao ok

mileo commented 1 month ago

/ocabot merge minor

OCA-git-bot commented 1 month ago

On my way to merge this fine PR! Prepared branch 14.0-ocabot-merge-pr-3170-by-mileo-bump-minor, awaiting test results.

OCA-git-bot commented 1 month ago

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