[14.0][FIX] QR Code não está sendo impresso em dispositivos android nas versões mais recentes

[felipezago]

Nas versões mais novas do android o QR code da nota não está sendo impresso, mesmo aparecendo na tela.

Isso acontece devido a um problema de regex na própria lib "qrcode.js". Existe até um PR em aberto resolvendo esse problema davidshimjs/qrcodejs#256.

Para solucionar baixei a nova versão da lib, fiz a alteração necessária do regex e importei novamente no projeto. Também precisei criar um evento para imprimir automaticamente apenas quando o QR code estiver 100% renderizado.

[OCA-git-bot]

Hi @ygcarvalh, @mileo, @lfdivino, @luismalta,
some modules you are maintaining are being modified, check this out!

[rvalyi]

@felipezago eu acho que vc ganharia a instalar o pre-commit no seu computador :-)

python -m pip install pre-commit
# cd projeto l10n-brazil
pre-commit

https://pre-commit.com/

Que aí vc roda localmente antes de mandar no GitHub, que fica matando as focas com aquecimento global de rodar a CI do projeto todo ;-)

[OCA-git-bot]

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]

Então, eu acho que para facilitar o debug seria bom que a lib l10n_br_pos/static/lib/qrcode.min.js nao seja "minificada" mas deixar o Odoo fazer o minify dela, pois ele faz sozinho. Assim, se vc botar a lib original não minificada, quando alguém botar o ?debug=assets no URL pelo menos a lib viria sem minificação e seria mais fácil debugar esse tipo de problema que vc resolveu agora.

Eu acabei de olhar, no OCA/web, tem apenas 2 módulos que tem libs minificadas (uma sendo base64.js) e nas v15 e v16 não tem mais nenhum, ou seja eu acho que não é uma pratica muito legal.

Vale para a outra lib sBarcode.code128.min.js tb.

[felipezago]

Então, eu acho que para facilitar o debug seria bom que a lib l10n_br_pos/static/lib/qrcode.min.js nao seja "minificada" mas deixar o Odoo fazer o minify dela, pois ele faz sozinho. Assim, se vc botar a lib original não minificada, quando alguém botar o ?debug=assets no URL pelo menos a lib viria sem minificação e seria mais fácil debugar esse tipo de problema que vc resolveu agora.

Eu acabei de olhar, no OCA/web, tem apenas 2 módulos que tem libs minificadas (uma sendo base64.js) e nas v15 e v16 não tem mais nenhum, ou seja eu acho que não é uma pratica muito legal.

Vale para a outra lib sBarcode.code128.min.js tb.

feito!

[rvalyi]

LGTM (code, não testei)

[lfdivino]

LGTM!

[rvalyi]

Só para deixar claro @lfdivino @ygcarvalh e todo pessoal da Kmee que estão registrados como mantenedor do módulo vcs tem permissão de dar o ocabot merge quando o PR afeta apenas o módulo que vcs estão como mantenedores (no caso seria menor eu acho). Nesse tipo de situação faz até mais sentido vcs que dominam o módulo fazer do que eu por examplo. Agora atenção, mantenedor não deve fazer besteira tá. Mas aí que tá ou é pose ou é para valer :-)

[rvalyi]

/ocabot merge minor

[OCA-git-bot]

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

[OCA-git-bot]

@rvalyi your merge command was aborted due to failed check(s), which you can inspect on this commit of 14.0-ocabot-merge-pr-2522-by-rvalyi-bump-minor.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

[rvalyi]

/ocabot merge minor

[OCA-git-bot]

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

[OCA-git-bot]

@rvalyi your merge command was aborted due to failed check(s), which you can inspect on this commit of 14.0-ocabot-merge-pr-2522-by-rvalyi-bump-minor.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

[rvalyi]

Aff o merge não ta indo porque ele ta tentando puxar a nfelib 2 claro. Então uma solução é aprovar esse PR trivial que resolve o problema: #2541

[felipezago]

Aff o merge não ta indo porque ele ta tentando puxar a nfelib 2 claro. Então uma solução é aprovar esse PR trivial que resolve o problema: #2541

Pronto, agora com a aprovação do PR acredito que não vai mais dar problema

[rvalyi]

/ocabot merge minor

[OCA-git-bot]

This PR looks fantastic, let's merge it!
Prepared branch 14.0-ocabot-merge-pr-2522-by-rvalyi-bump-minor, awaiting test results.

[OCA-git-bot]

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