[14.0][NEW] l10n_br_ie_search #2582
Conversation
ODBreno
force-pushed
the
14.0-l10n_br_ie_search
branch
from
c6d5abb to
e783f59
Compare
ODBreno
force-pushed
the
14.0-l10n_br_ie_search
branch
5 times, most recently
from
28b6c44 to
18cc3ed
Compare
ODBreno
force-pushed
the
14.0-l10n_br_ie_search
branch
2 times, most recently
from
5178c65 to
d2b0673
Compare
|
@rvalyi @marcelsavegnago @antoniospneto podem revisar, por favor? |
|
This PR has the |
|
@rvalyi ou @marcelsavegnago poderia rodar meu teste de novo? Deu erro no product_contract. |
|
Ola @ODBreno , da um um pull da branch 14.0 e faz um rebase da sua branch em cima da 14.0 pois vc vai precisar desse fix pros testes rodar #2602 Outra coisa, precisa ajeitar um pouco os commits da sua branch. Por examplo esse ultimo commit é interessante vc fazer um squash dele com algum commit anterior que mexe no mesmo arquivo, esse outro tb 5a4dc5e Basicamente vc precisa aprender os comandos git rebase -i, git stash/git stash pop, git cherry-pick, pois na OCA tem que evitar de mandar os commit de experimentos que vc teve que fazer no seu computador ate chegar na solução. Ja tem bem demais commit certos no projeto para poluir com isso que depois ferra as ferramentas de sincronização entre as branches ou a busca por explicações ou culpados (git blame). E olha que a gente não é extremista com isso, se fosse um repo que alguém como o Pedro Bazea mantem, ele ia te pedir de juntar tudo isso num commit apenas. Pode ter alguns no caso, mas uns 5 me parece ser o máximo, experimentos não tem que entrar pro repo da OCA. |
ODBreno
force-pushed
the
14.0-l10n_br_ie_search
branch
4 times, most recently
from
dac16ef to
efbf4fe
Compare
|
@rvalyi consegue dar uma olhada? Acho que o Breno fechou as alterações que você havia requisitado. |
|
This PR has the |
|
/ocabot merge nobump |
1 similar comment
|
/ocabot merge nobump |
|
What a great day to merge this nice PR. Let's do it! |
|
@marcelsavegnago your merge command was aborted due to failed check(s), which you can inspect on this commit of 14.0-ocabot-merge-pr-2582-by-marcelsavegnago-bump-nobump. 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. |
ODBreno
force-pushed
the
14.0-l10n_br_ie_search
branch
from
efbf4fe to
e886560
Compare
|
Mockei o teste do sintegraws, eu deveria ter feito isso desde o começo, nunca é bom deixar os testes dependendo de serviços externos. Se puderem tentar dar o merge de novo pra mim, por favor @rvalyi @marcelsavegnago. |
|
/ocabot merge nobump |
|
This PR looks fantastic, let's merge it! |
|
Congratulations, your PR was merged at a87e184. Thanks a lot for contributing to OCA. ❤️ |
Como foi discutido no PR #2555, foi feita a implementação da consulta da inscrição estadual pelo sintegraws e pelo sefaz, permitindo que o usuário escolha qual provedor deseja utilizar.