Feature/tag icms uf destino

[danimaribeiro]

WIP.

[mileo]

=DDDDDDDDDDDDDDDDD

[mileo]

Amanhã vou até acordar mais cedo pra testar

[mileo]

Obrigado

[danimaribeiro]

Acho que foi commits demais nesse PR, fiz na branch errada, vou corrigir. Este PR também depende do #15 tag CEST.

[rvalyi]

@danimaribeiro @mileo @renatonlima acabou de ter esse commit no repo original https://github.com/aricaldeira/PySPED/commit/319ba19fca4962cad5c5cff2a6e22bc692ca8170 Porem nao parece contemplar o ICMS da UF de destino.

Agora eu penso que a gente aqui deveria fazer que nem a branch OCA/OCB: ok a gente precisa de um fork do repo original, mas pelo menos podemos ter a politica que todos os patches aplicados sejam propostos no upstream (nao parece ser o caso nesse PR), assim a gente minimiza a divergencia e fica ate um justo retorno para ter sinergias com o projeto original, mesmo que infelizmente nao ta bem integrando os nossos PR's (uma coisa que eu acho preocupante com essa lib de transmissao, mesmo que hoje nao temos nada melhor).

[danimaribeiro]

Eu particularmente sempre olho o histórico de PR de cada projeto, se vejo que o mantenedor do projeto não responde aos PRs abertos eu nem perco meu tempo propondo um novo PR que vai ficar eternamento aberto.

No caso do repositório pysped já cheguei a propor mudanças: https://github.com/aricaldeira/PySPED/pull/16 Que no caso nem foi respondido com sim, e nem um não.

É justo fazer o PR ao projeto principal, sim é justo, mas também é necessário que o autor do projeto esteja disposto a aceitar contribuições e ao menos responder que não é o caso do pysped.

Este branch já teve mais PR aceitos que o repositório original, então acredito que seja apenas uma questão do Ary começar a delegar para outras pessoas, ou pelo menos começar a responder os PRs abertos, quando isto acontecer com certeza podemos pegar e mover completamente o foco para o projeto principal.

[rvalyi]

@danimaribeiro a gente tem exactamente o mesmo problema com odoo/odoo e OCA/OCB, os caras nao olham 10% dos PR integrados na OCB. Porem assim tb e uma forma de nao poder ser acusado de fork ofensivo e mesmo que so 10% vai ja e isso. O @renatonlima e o @mileo tambem ja tiveram PR's aceitos mas a maioria nao foi considerada.

Eu entendo totalmente o autor de um projeto open source nao conseguir dar atencao a todas contribuicoes isso e ate a coisa mais comum do mundo infelizmente. Mas enfim e meio por isso que nao sou a favor de "casar" a localizacao brasileira do Odoo totalmente ao pysped. Como falei em outro thread, para qualquer solucao de transmissao e preciso primeiro mapear os dados do Odoo para entrar na lib de transmissao e isso tem que ser feito em Python na mesma versao do que o Odoo; nao tem como ser de outro jeito. Nisso poderiamos ate aceitar que o pysped seja realmente o padrao (porem na condiçao que seja isolado e razoavelemente simples de trocar). Agora uma vez a porra do XML montado, todo restante pode ser feito pelo pysped (perfeito para hoje) mas tb seria facil usar outra coisa e ai temos que manter essa parte plugavel pelo menos porque realmente e debativel se o pysped vai ser a melhor solucao a longo prazo.

[rvalyi]

@danimaribeiro uma outra vantagem "social" de mandar o PR tambem no repo original e que isso manda um recado aos desenvolvedores que vao um dia olhar o pysped ou que ja estao seguindo. Entao assim maximiza a probabilidade de alguem testar e melhorar o PR ou ate de se juntar para cerscer a massa critica do fork e deixar ele mais viavel.

[mileo]

@rvalyi como eu e o @danimaribeiro fizemos na migração da localização p/ 8.0? Entendo

[rvalyi]

@mileo It's not the same we had a total clash for 6 months but also your PR were not in the proper form it was a big blob of questionable work back in June and we thought we had done a much better job because we designed the whole code (or 95% of it if your prefer). Now, soon after we analysed all your features and are ready to merge them all, provided they come in an acceptable review-able form as all other OCA projects do. It's really not the same, the original pysped repo has been ignoring PR's for much more than 6 months and its due to its nature, it's not a life accident.

[rvalyi]

@mileo com a OCA, imagine se algo assim acontecer de novo, agora que voces estao accostumados com os processos da OCA voces poderiam assumir a gestao da migracao. Isso tambem porque com uns 70% de coverage que vamos ter deixa muito mais seguro que nada importante acaba sendo quebrado por alguem que nao teria idieia de porque algum determinado codigo e assim. Infelizmenente isso nao eram as condicoes na hora da migracao para 8.0. E alem disso tinhamos ja feito a maioria do trabalho a gente so nao podia publicar naquele momento que isso ia fazer nossos clientes nao pagar e matar a gente que tinha feito a grande maioria do investimento no projeto. Isso sim era a coisa injusta que nao podia ser. Salvamos a nossa pele e agora o projeto esta no trilho de novo. Ou seja com certeza a estrutura da OCA e a diversidade dos contribuidores que estamos escalando aos poucos vai evitar estruturalmente o tipo de risco de morte que algo como o pysped corre hoje.

[rvalyi]

@mileo agora quando voce considera que o projeto da localizacao brasileira do odoo vai atingir facilmente uma decada de vida. Me desculpe mas 6 meses de inatividade da liderança e algo bem comum nos projetos open source e nao e o fim do mundo. Olha quanto tempo vai levar para migrar o connector magento para v9 e quanto tempo se levou nas migracoes de antes. 6 meses nao sao 2 anos e se vc planeja um negocio com open source vc tem que ser resiliente com a possibilidade de 6 meses de inatividade de alguma dependencia do seu projeto sem ter que crucificar os autores da dependencia por isso. Adoro o CMS Locomotive por examplo: a ultima versao estavel usa Rails 3 enquanto o Rails 4 esta no ar ha mais de 3 anos, nem por isso eu vou criticar o autor (alias a v3 de agora esta finalmente usando o Rails 4). Vc vai ver a zona que vai ficar do lado do odoo daqui pouco, so espero que os caras vao conseguir estabilizar pelo menos um pouco a v9 antes. Mas ai vou criticar sim porque isso sera a consequencia de anos de se arriscar com dinheiro facil dos outros em vez de encarrar a sustentabilidade do produto.