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][ADD] l10n_br_account_payment_instrument: novo módulo #2389

Closed
wants to merge 16 commits into from

Conversation

antoniospneto
Copy link
Contributor

(Em desenvolvimento)

Adiciona um novo modelo para armazenar as informações de um instrumento de pagamento iniciado por alguma instituição financeira brasileira, pode ser um "Boleto" ou um "Pix QR Code"

Essas informações fica associada a um ou mais contas a receber/apagar (account_move_line)

São utilizados para carregar as informações para dentro da ordem de pagamento (arquivo de remessa)

Prints:

image

image

image

image

@antoniospneto antoniospneto force-pushed the 14.0-POC-PAYMENT-INSTRUMENT branch 2 times, most recently from 37ada55 to 80dc901 Compare March 18, 2023 21:30
@rvalyi
Copy link
Member

rvalyi commented Mar 20, 2023

@antoniospneto @felipemotter @marcelsavegnago que tal tentar fazer o review e merge desse refactor antes de adicionar esse novo modulo #2386 ? (me parece que a gente pode ter o trabalho de limpeza e migração duas vezes talvez se a gente não fizer nessa ordem). Sera se o modulo seria muito independente desse refator?
cc @mbcosta @renatonlima

@antoniospneto
Copy link
Contributor Author

@rvalyi concordo com você , é melhor esperar o #2386 sim, por enquanto deixamos a pr em rascunho. Valeu!

@douglascstd
Copy link
Member

Pontos que melhoram a usabilidade do aplicativo - NÃO IMPACTAM O USO.
1- Botão de Confirmar
image
Os botões inferiores, quando preenche o QRcode, ele apresenta os botões Edit e Create, mas a ação esperada do usuário é Fechar a janela. Sugestão é incluir um botão com "Confirm" e esse botão ser destacado como próxima ação.

2- Ação de Salvar
As linhas de "Payables" são geradas somente após salvar a fatura. E exigindo a navegação do mouse para o botão salvar e depois acessar a guia "Payables". Sugestão é ativar essa geração ou até salvar o registro, na guia "Payables"
image

@marcelsavegnago
Copy link
Member

marcelsavegnago commented Apr 6, 2023

Pessoal.. como comentado pelo @rvalyi entendo que seria melhor revisar esta PR #2386 antes e já entrar com este módulo ajustado.

cc @antoniospneto @felipemotter @douglascstd

@felipemotter
Copy link
Contributor

@marcelsavegnago eu to com a agenda ferrada, esse mês eu to totalmente sem tempo, já tirei o tempo que eu não tinha proi account. Nas proximas semanas nosso foco virá para ca.

@rvalyi
Copy link
Member

rvalyi commented May 29, 2023

@antoniospneto vc poderia fazer um rebase por favor? Pois foi feito o merge do refator do bank lines do account_payment_order...

cc @felipemotter @marcelsavegnago @renatonlima @douglascstd @mbcosta

@antoniospneto antoniospneto force-pushed the 14.0-POC-PAYMENT-INSTRUMENT branch from 80dc901 to 29a9e72 Compare July 17, 2023 13:26
@antoniospneto antoniospneto marked this pull request as draft July 17, 2023 19:14
@antoniospneto antoniospneto force-pushed the 14.0-POC-PAYMENT-INSTRUMENT branch from 92b2a99 to d0aa968 Compare July 17, 2023 22:31
@antoniospneto antoniospneto force-pushed the 14.0-POC-PAYMENT-INSTRUMENT branch 2 times, most recently from 872d14b to 2840b2a Compare July 17, 2023 23:46
@antoniospneto antoniospneto force-pushed the 14.0-POC-PAYMENT-INSTRUMENT branch from 2840b2a to 7dbf318 Compare July 17, 2023 23:49
Copy link
Member

@douglascstd douglascstd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[FUNCTIONAL TEST] Need Improves

Realizado teste de entrada com Código de Barras:

  1. Ao digitar o código de barras, e pressionar "Enter" não é atualizado a tela e não mostra os detalhes

  2. Para que as linhas Payables apareçam é necessário salvar a Fatura, seria interessante a criação destas linhas antes de salvar, ou que o salvar, esteja no formulário para que o fluxo de digitação seja contínuo.

  3. O código lido pelo barcode, ele tem o dígito 0 no início:
    Leitor: 034198941501500001790001043510049102015000
    image


Teste de código de barras- OK - Usando a Composição e a Representação numérica:
image
image

Configuração do Código de Barras no boleto:
Pasted image 20221215203740


Teste realizado com PIX Pagamento QR Code - OK
image
Teste realizado somente até a leitura do Código QR. Não foi realizado o processo até a finalização do pagamento.
image

@antoniospneto
Copy link
Contributor Author

antoniospneto commented Aug 22, 2023

@douglascstd

  1. Ao digitar o código de barras, e pressionar "Enter" não é atualizado a tela e não mostra os detalhes

Esse é o comportamento padrão do Odoo, não é o "Enter" que faz atualizar, você tem que mudar o cursor para fora do campo, você pode fazer usando o "tab" ↹ ou clicando com mouse fora do campo.

  1. Para que as linhas Payables apareçam é necessário salvar a Fatura, seria interessante a criação destas linhas antes de salvar, ou que o salvar, esteja no formulário para que o fluxo de digitação seja contínuo.

Ok, vou tentar uma abordagem um pouco diferente, em breve atualizo aqui.

  1. O código lido pelo barcode, ele tem o dígito 0 no início:
    Leitor: 034198941501500001790001043510049102015000

Tem certeza que sua leitora está lendo corretamente ?
Pois o problema aqui não parece ser só zero a esquerda, está faltando digitos nesse barcode.
Sabe me informar qual é a linha digitável relativa a esse barcode ?

@marcelsavegnago
Copy link
Member

ping @douglascstd

@douglascstd
Copy link
Member

@douglascstd

  1. Ao digitar o código de barras, e pressionar "Enter" não é atualizado a tela e não mostra os detalhes

Esse é o comportamento padrão do Odoo, não é o "Enter" que faz atualizar, você tem que mudar o cursor para fora do campo, você pode fazer usando o "tab" ↹ ou clicando com mouse fora do campo.
OK entendido

  1. Para que as linhas Payables apareçam é necessário salvar a Fatura, seria interessante a criação destas linhas antes de salvar, ou que o salvar, esteja no formulário para que o fluxo de digitação seja contínuo.

Ok, vou tentar uma abordagem um pouco diferente, em breve atualizo aqui.

Perfeito

  1. O código lido pelo barcode, ele tem o dígito 0 no início:
    Leitor: 034198941501500001790001043510049102015000

Tem certeza que sua leitora está lendo corretamente ? Pois o problema aqui não parece ser só zero a esquerda, está faltando digitos nesse barcode. Sabe me informar qual é a linha digitável relativa a esse barcode ?
Não encontrei mais esse código original. Mas tenho outro que deve servir para o teste:
34194920800004244981482262670912040018950000
image

@antoniospneto
Copy link
Contributor Author

Oi @douglascstd mas veja bem, antes o código de barras que você passou tinha só 42 caracteres, mas o correto é ter 44 caracteres.

Esse que você passou por ultimo está correto, tem 44 caracteres. o sistema reconheceu sem erros:
image

ainda não consegui reproduzir a falha que você mencionou.

@douglascstd
Copy link
Member

ainda não consegui reproduzir a falha que você mencionou.

O erro que mencionei só acontece ao fazer a leitura pelo leitor de código de barras.. o leitor insere esse Zero do lado esquerdo. E sugiro utilizar somente os 44 digitos a direita que são inseridos no campo.

Pois o processo de leitura é:

  • posicionar o cursos (com o mouse) no campo barcode
  • executar a leitura do código de barras pelo leitor laser manual
  • confirmar

Desta forma, quando o leitor insere esse zero a mais a esquerda, as validações do Odoo não correm corretamente.

@antoniospneto
Copy link
Contributor Author

ainda não consegui reproduzir a falha que você mencionou.

O erro que mencionei só acontece ao fazer a leitura pelo leitor de código de barras.. o leitor insere esse Zero do lado esquerdo. E sugiro utilizar somente os 44 digitos a direita que são inseridos no campo.

Pois o processo de leitura é:

  • posicionar o cursos (com o mouse) no campo barcode
  • executar a leitura do código de barras pelo leitor laser manual
  • confirmar

Desta forma, quando o leitor insere esse zero a mais a esquerda, as validações do Odoo não correm corretamente.

@douglascstd será que esse problema não dá de resolver revisando as configurações do leitor de código de barras ?
Pois esse comportamento não é o normal, se você entrar no internet banking e por o código
034194920800004244981482262670912040018950000 (com o zero na frente, total de 45 digitos) vai dar erro também, o internet banking só vai processar se remover esse zero extra.

Acho que primeiro podemos tentar solucionar a configuração do leitor, se isso realmente não for possível podemos adicionar uma lógica para tratar esse caso aqui

Copy link

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.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Feb 25, 2024
@marcelsavegnago
Copy link
Member

@douglascstd @antoniospneto como ficamos em relação a este assunto ?

@github-actions github-actions bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Mar 3, 2024
Copy link

github-actions bot commented Jul 7, 2024

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.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Jul 7, 2024
@marcelsavegnago
Copy link
Member

@douglascstd @antoniospneto como ficamos em relação a este assunto ?

@antoniospneto
Copy link
Contributor Author

@marcelsavegnago temos em aberto a dificuldade técnica que está acontecendo com o leitor do código de barras utilizados nos testes pelo @douglascstd, a minha impressão é que o leitor não está com a configuração correta. É preciso analisar melhor, fazer mais testes com esse leitor para entender

@github-actions github-actions bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Jul 21, 2024
Copy link

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.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Nov 24, 2024
@github-actions github-actions bot closed this Dec 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
14.0 new feature stale PR/Issue without recent activity, it'll be soon closed automatically.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants