Closed mileo closed 4 years ago
@sadamo @lfdivino @wagnerpereirasp
@mileo, nao acho interessante tentar integrar tudo isso antes de fazer as migracoes de 9 ou 10 ou digamos que e um pouco tarde para de repente querer incluir tudo isso antes de migrar. Tem alguns desses items que se vc refizer o PR na 9 ou na 10 depois vai dar na mesma (por examplo a integracao do modulo l10n_br_hr_payroll que nao existe hoje no repo). A unica diferenca e que nao ira atrasar a migracao. Portanto, acho interessante voces focar nos items dessa lista que tem impact nos modulos ja existente, de forma strutual e principalmente se a mudanca e importante e precisa de varios commits. Fora isso, outros PR em paralelo na 9 ou 10 e cherry-pick deveria resolver sem atrasar demais. Realmente os PR's de IBT, NT e separacao das visoes fazem muita diferenca na migracao do existente, so por isso pretendemos inclui-los antes de migrar.
Tem vários bugs na lista é muito importante inclui-los antes de fechar a versão v8. Mas concordo contigo que nem todos eles são essenciais.
Veremos oq que podemos fazer até sexta-feira.
Na minha opinião a alteração das visões impacta muito o fluxo dos usuários da versão 8. E deveria ser feita somente na v9.
Mas como vcs precisam do merge para um de seus clientes, vamos digerir a alteração e seria ótimo que vocês analisassem as alterações que fizemos, pois os PRs estão abertos antes desses.
Completo, visto que vai demorar uns dias para fecharmos a versão v10. E mais ainda a migração dos clientes vai ser muito mais facil se estas mudanças estejam integradas na v8.
@mileo pode mandar os PR's. Alias sao poucos PR's que voces mandaram na 8 onde nao demos retorno. Agora isso tb nao depende apenas da gente, quanto mais gente revisar melhor as chances de incluir os PR's, espero que vc pude deixar isso bem claro para os potentiais contribuidores na PyCon...
@danimaribeiro vc pode fazer a mesma coisa referente as suas mudanças? Vai conseguir vir para reunião na segunda?
Para mim, de accordo com os criterios que eu listei, de importante e urgente ou facil nessa lista tem:
[ ] 3. Melhorias nas pesquisas pesquisa por CNPJ/IE/Razão social no picking [ ] 8. Baixa bonificação: #337 (apesar que talvez iria mudar muito nas v9/10 de forma que nao seja pertinente atrasar a migracao por causa desse PR.) [ ] 11. Bug: Data das faturas: #203 [ ] 15. GNRE: Rebase #286 [ ] 18. Correção do campo origem na posição fiscal [ ] 19. Dados da FCI no Danfe #428 [ ] 20. Correção dos lançamentos contábeis da fatura
1 e 2 sao legais tb, mas tb se nao tiver ja pronto da para fazer um cherry pick na 8.0 e 9.0/10.0.
agora realmente a addicao de modulos como de RH ou POS e muito bem vinda mas tem que ser feita em paralelo como em todos projetos da OCA e nao ficar atrasando a migracao do que ja pode ser migrado.
Pessoal, verifiquei que nosso código referente ao frete esta bem diferente. Lembro que no passado refatorei colocando o código no button reset taxes, mas deu alguns bugs e ai eu voltei para o motor de impostos.
Acho q tenho uns testes em yml aqui referentes ao frete que não estão integrados, mas acredito que eles não sejam capazes de simular o erro. Poderiam realizar testes manuais
ping @danimaribeiro @Mackilem
@mileo Era esse o bug que estava dando no frete? https://github.com/OCA/l10n-brazil/pull/459