[14.0][FIX] l10n_br_account: validate move on document #3093
Conversation
|
@felipemotter @antoniospneto essa ainda ta de rascunho? |
|
Sim está, peguei um erro usando essa PR que quando confirmo pela fatura da erro como se tivessa tentando validar duas vezes... estou vendo a melhor abordagem. |
felipemotter
force-pushed
the
14.0-val-move-doc
branch
from
8bd1a00
to
a54c6db
Compare
felipemotter
force-pushed
the
14.0-val-move-doc
branch
2 times, most recently
from
442d8f7
to
d6de147
Compare
felipemotter
force-pushed
the
14.0-val-move-doc
branch
from
d6de147
to
e303fda
Compare
|
Essa PR já está funcionando tanto para Validar quanto para colocar em rascunho, só estou ajeitando os testes. |
|
PR pronta para review... engloba a validação e voltar para draft a fatura a partir do documento fiscal. |
marcelsavegnago
requested review from
douglascstd,
kaynnan,
WesleyOliveira98,
Matthwhy,
rvalyi and
renatonlima
|
nao precisa atualizar o banco, mas tou fazendo o merge com minor para sinalizar que eh um pouco mais importante do que patch pelas mudanças dos metodods. /ocabot merge patch |
|
What a great day to merge this nice PR. Let's do it! |
|
@rvalyi your merge command was aborted due to failed check(s), which you can inspect on this commit of 14.0-ocabot-merge-pr-3093-by-rvalyi-bump-patch. 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. |
|
/ocabot merge minor |
|
This PR looks fantastic, let's merge it! |
|
Congratulations, your PR was merged at 6a03ca6. Thanks a lot for contributing to OCA. ❤️ |
Um problema semelhante ao corrigido na PR #3090, estou corrigindo a validação do account.move quando se valida pela visualização do l10n_br_fiscal.document. Com a possibilidade de visualizar diretamente o documento depois da PR #2761 , isso acabou ficando quebrado. Logo dou exemplos.