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][REF] l10n_br_purchase: Usando o campo related para o País da Empresa ao invés de um campo função #3134

Closed mbcosta closed 2 months ago

mbcosta commented 2 months ago

It's unnecessary create a field for Country Company because can be done by related.

Usando o campo related para o País da Empresa ao invés de um campo função, PR simples que corrigi algo que acabou passando por não causar nenhum erro, quando estive testando o caso de compatibilidade com casos internacionais eu precisei validar se o Active Company era o mesmo do Company, o mesmo ocorreu no l10n_br_sale mas corrigi antes do merge, com isso deve desaparecer o warning

WARNING odoo odoo.addons.base.models.ir_model: Two fields (active_company_country_id, company_id) of purchase.order() have the same label: Company.

cc @rvalyi @renatonlima @marcelsavegnago @mileo

OCA-git-bot commented 2 months ago

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

rvalyi commented 2 months ago

/ocabot merge minor

OCA-git-bot commented 2 months ago

@rvalyi The merge process could not start, because command git fetch --quiet --force --prune https://github.com/OCA/l10n-brazil 'refs/heads/*:refs/heads/*' failed with output:

fatal: unable to access 'https://github.com/OCA/l10n-brazil/': gnutls_handshake() failed: The TLS connection was non-properly terminated.
rvalyi commented 2 months ago

/ocabot merge minor

OCA-git-bot commented 2 months ago

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

OCA-git-bot commented 2 months ago

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