[12.0][l10n_br_base][BACKPORT] add partner pix #2189
Conversation
|
Hi @renatonlima, |
rvalyi
changed the title
| @@ -0,0 +1,150 @@ | |||
| import phonenumbers | |||
| from email_validator import EmailSyntaxError, validate_email | |||
There was a problem hiding this comment.
Aproveitando que você esta alterando esse trecho de código se tiver tempo poderia utilizar o módulo:
https://github.com/odoo/odoo/blob/16.0/addons/mail/tools/mail_validation.py#L16-L21
É um trecho do Odoo que mantem a compatibilidade caso o módulo email_validator não esteja instalado, se vc procurar tem vários exemplos no código.
There was a problem hiding this comment.
Opa Mileo, eu não vi como é na v16 mas na v14 esse módulo limitava as funcionalidade do email_validator. Agora não lembro bem o motivo, mas eu tinha olhado essa opção.
rvalyi
force-pushed
the
12.0-add-partner-pix
branch
from
fcdffa0
to
50fc5e5
Compare
|
pessoal o Travis não esta passando por conta de um problema com o Travis já faz uma semana eu sugiro de mudar pro Github Actions com essa PR antes: #2130 |
|
@rvalyi Poderia fazer um rebase? |
rvalyi
force-pushed
the
12.0-add-partner-pix
branch
from
50fc5e5
to
20b9f2a
Compare
There was a problem hiding this comment.
Parece que tem dois problemas no travis
Na criação de um res.partner.bank esta exigindo que o res.bank tenha o codigo do banco central (code_bc)
https://github.com/OCA/l10n-brazil/actions/runs/3370226034/jobs/5590860128#step:8:147
Não achou a lib vcrpy
https://github.com/OCA/l10n-brazil/actions/runs/3370226034/jobs/5590860128#step:8:889
Acho que seria bom incluir esse commit via cherry-pick
8845fc4
|
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
github-actions
bot
added
the
stale
backport de #2123
pode ser interessante manter ainda a consistência com esse PR das chaves PIX para depois poder fazer algum backport de pagamentos PIX ou CNAB de pagamento na 12.0.