← Back to team overview

openerp-brazil-team team mailing list archive

Re: lp:~vdittgen/openerp.pt-br-localiz/tax_application_behaviour into lp:openerp.pt-br-localiz

 

Oi Raphael,

A gente olhou (alias ja tinha olhado nas branches de voces) um pouco com o
> Renato e tem coisas que a gente viu que sao bem e outras que a gente tem
> algumas duvidas (sera se todos os flags de taxas sao requiridos mesmo?).
>

As flags tornam a parametrização mais fácil e natural.


>
> Tecnicamente, teria que fazer uma coisa antes do merge: que seria, levar o
> modulo l10n_br de volta (caso os flags realmente forem necessarios) e
> depois fazer o commit da alteraçao de voces.
>

Exatamente, na nossa Branch eu fiz isto, ontem a noite acabei pulando esta
etapa, mas se chegarmos em um acordo sobre as modificações eu proponho o
merge novamente fazendo o revert da exclusão do l10n_br.


>
> Inclusive, isso é uma coisa que da par debater, porque levando o modulo
> l10n_br de vola aqui, a gente vai ter o problema que o modulo esta em dobro
> e que os usuarios nao conseguem instalar o nao instalam a versao certa,
> como era no fim de 2011. So para lembrar, levou 5 meses para a OpenERP SA
> fazer o merge da atualizaçao do l10n_br dentro dos addons, entao a gente
> tem que tomar cuidado para nao soffrer de novo esse tipo de situaçao.
>

Sabemos disso, por isso temos que retornar o l10n_br para a localização, é
complicado sempre termos que aguardar a OpenERP SA para qualquer
modificação, acho que é muito ruim depender deles se queremos que a coisa
realmente funcione.

É bom que eles tenham aquele l10n_br básico para testes simples, mas o
módulo oficial ter que estar sob o nosso controle já que eles não tem
nenhuma agilidade.


>
> Finalemente, quando voce fizer um pach desse, é totalmente legitimo voce
> incluir os seus copyrights no arquivo python que mudaram. Agora, a respeito
> de adicionar a Proge como autor do modulo assim como a Akretion, a gente
> acha que ainda é precipitado. No sentido do que hoje a Proge fez apenas 1%
> dos commits enquanto a Akretion fez 90% (
> https://www.ohloh.net/p/openerpbrasil/analyses/latest ).
> A gente nao vai exigir voces fazer a metade para poder ter esse tipo de
> visibilidade, mas a gente acha que deveria ser que nem fizemos com o
> Magentoerpconnect (onde a CampToCamp chegou 2 anos depois mas virou
> core-editor tambem) e botar o limite perto de 25% o que e bastante razoavel
> e abre o jogo. A gente vai mandar um email esse final de semana para
> formalizar essas regras. Na verdade o % vai baixando de accordo com o
> numero de "core-editors" (com 3 core-editors o limite seria 33.33%/2 =
> 16.66% dos commits) , a gente depois passara a regra que a gente quer
> applicar nesse projeto.
>

Concordo não pensei muito sobre o fato já que tinha a "OpenERP Brasil" como
co-autor, realmente no momento somos contribuidores do módulo e acho justo
atingir o percentual que tu sugere para se tornar co-autor.


>
> Hoje temos bastante trabalho, mas amanha a gente comença a dar perspetiva
> para poder integrar essa melhoria.
>

Ótimo.


>
> Novamente, nossos parabens por começar a contribuir ao projeto da
> localizaçao. Precisa de mais gente sujando a mao assim para isso avançar
> mais rapidamente.
>


Grato, precisamos aprovar este merge para seguir contribuindo nesta Branch,
estas modificações são importantes para nós, sem isto teremos que optar por
tocar a nossa própria Branch.

Estamos com os testes funcionais prontos, vou criar mais alguns casos de
uso e adiciona-los no repositório na semana que vem.

-- 

--

Att,

Vinicius Dittgen

PROGE - Soluções em Gestão Empresarial

Fone: +55 (51) 3085.0096  Cel: +55 (51) 9193.2020

Skype: vdittgen

*www.proge.com.br*

Canal IRC: #openerp-br




Seja um voluntário
www.centrorepensar.com.br
Centro especializado em Deficiência Intelectual, Autismo, Síndrome de
Down e Dificuldade de Aprendizagem

https://code.launchpad.net/~vdittgen/openerp.pt-br-localiz/tax_application_behaviour/+merge/104653
Your team OpenERP Brazil Team is subscribed to branch lp:openerp.pt-br-localiz.


References