[14.0][REF] Porte da geração do certificado fake para o erpbrasil.assinatura #1843
Conversation
marcelsavegnago
requested review from
mbcosta,
lfdivino,
gabrielcardoso21,
rvalyi,
mileo and
marcelsavegnago
|
@netosjb tira o seu ultimo commit por favor: na v14 o requirement.txt é gerido direitamente a partir dos manifests dos modulos. Isso garante uma testagem melhor que vc vai conseguir instalar os modulos da OCA pelo pip. So que o filho da mae não aceita branches de desenvolvimento no Nisso ate o @mileo resolver publicar uma nova release do erpbrasil.assinatura ou não, eu resolvi fazer um fork temporario no repo da Akretion com um novo empacotamento erpbrasil.assinatura-nopyopenssl, vc os 2 ultimos commits: eu acho que podemos usar esse fork por enquanto pelo menos pas destravar as coisas na v14.0 Ai eu fiz um PR para vc usar no seu PR, vé se isso não resolve Engenere#8 |
|
@netosjb na vdd vc nao pode usar meu PR porque ele ta incluindo seu commit no requirements.txt. mas enfim vc entendeu basta fazer a mesma alteração no manifest e tirar esse seu ultimo commit do requirements.txt. |
antoniospneto
force-pushed
the
14.0-ref-certicates2
branch
from
a9e4831 to
ec51ca4
Compare
|
boaaa @rvalyi agoraa sim :D vou só adicionar mais um commit por causa do pre-commit |
|
tá falhando pq ele não tá conseguindo carregar esses pacotes 'Crypto' tem haver com isso: erpbrasil/erpbrasil.assinatura#31 |
|
Eu abri um plano de suporte premium para vc @netosjb Isso que é release ai https://pypi.org/project/erpbrasil.assinatura-nopyopenssl/ |
aee tudo verdinho agora, obrigado @rvalyi ! |
obrigado vc @netosjb vc ta ajudando muito com essas coisas! |
|
/ocabot merge minor |
|
On my way to merge this fine PR! |
|
Congratulations, your PR was merged at 3e8ae92. Thanks a lot for contributing to OCA. ❤️ |
Substitui a PR #1778, depende da PR no erpbrasil.assinatura: erpbrasil/erpbrasil.assinatura#32
Conforme foi sugerido pelo @renatonlima fiz o porte da criação do certificado para dentro do erpbrasil.assinatura.