Merge lp://qastaging/~vdittgen/openerp.pt-br-localiz/tax_application_behaviour into lp://qastaging/openerp.pt-br-localiz/6.1

Proposed by Vinicius Dittgen - Proge.com.br
Status: Needs review
Proposed branch: lp://qastaging/~vdittgen/openerp.pt-br-localiz/tax_application_behaviour
Merge into: lp://qastaging/openerp.pt-br-localiz/6.1
Diff against target: 3479 lines (+3258/-59)
18 files modified
l10n_br/__init__.py (+24/-0)
l10n_br/__openerp__.py (+69/-0)
l10n_br/account.py (+150/-0)
l10n_br/account_view.xml (+114/-0)
l10n_br/data/account.account.template.csv (+639/-0)
l10n_br/data/account.account.type.csv (+7/-0)
l10n_br/data/account.tax.code.template.csv (+11/-0)
l10n_br/data/account_tax_template.xml (+755/-0)
l10n_br/data/l10n_br_account_chart_template.xml (+27/-0)
l10n_br/data/l10n_br_data.xml (+673/-0)
l10n_br/i18n/l10n_br.pot (+170/-0)
l10n_br/i18n/pt_BR.po (+188/-0)
l10n_br/l10n_br.py (+84/-0)
l10n_br/l10n_br_view.xml (+80/-0)
l10n_br/security/ir.model.access.csv (+3/-0)
l10n_br_account/account.py (+197/-56)
l10n_br_account/account_invoice.py (+66/-2)
l10n_br_sale/sale.py (+1/-1)
To merge this branch: bzr merge lp://qastaging/~vdittgen/openerp.pt-br-localiz/tax_application_behaviour
Reviewer Review Type Date Requested Status
Renato Lima - http://www.akretion.com Needs Resubmitting
Review via email: mp+104653@code.qastaging.launchpad.net

Description of the change

Add tax_add,tax_retain and tax_include behavior. Can support service invoices with retained taxes.
fix price_subtotal calc in invoice, and tax calc in sale order for tax_add type(like ipi).

To post a comment you must log in.
Revision history for this message
Alexandre Leitão (alexandre-leitao) wrote :
Download full text (12.3 KiB)

Vinicius,

Este merge resolve o caso da retenção do ir, confins e etc pra notas de serviço?

Atenciosamente,
Alexandre Leitão

Em 03/05/2012, às 20:40, "Vinicius Dittgen - Proge.com.br" <email address hidden> escreveu:

> Vinicius Dittgen - Proge.com.br has proposed merging lp:~vdittgen/openerp.pt-br-localiz/tax_application_behaviour into lp:openerp.pt-br-localiz.
>
> Requested reviews:
> OpenERP Brazil Core Team (openerp-brazil-core-team)
>
> For more details, see:
> https://code.launchpad.net/~vdittgen/openerp.pt-br-localiz/tax_application_behaviour/+merge/104653
>
> Add tax_add,tax_retain and tax_include behavior. Can support service invoices with retained taxes.
> fix price_subtotal calc in invoice, and tax calc in sale order for tax_add type(like ipi).
> --
> 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.
> === added directory 'l10n_br' === added file 'l10n_br/__init__.py' --- l10n_br/__init__.py 1970-01-01 00:00:00 +0000 +++ l10n_br/__init__.py 2012-05-03 23:39:20 +0000 @@ -0,0 +1,24 @@ +# -*- encoding: utf-8 -*- +################################################################################# +# # +# Copyright (C) 2009 Renato Lima - Akretion # +# # +#This program is free software: you can redistribute it and/or modify # +#it under the terms of the GNU Affero General Public License as published by # +#the Free Software Foundation, either version 3 of the License, or # +#(at your option) any later version. # +# # +#This program is distributed in the hope that it will be useful, # +#but WITHOUT ANY WARRANTY; without even the implied warranty of # +#MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the # +#GNU General Public License for more details. # +# # +#You should have received a copy of the GNU General Public License # +#along with this program. If not, see . # +################################################################################# + +import account +import l10n_br + +# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4: + === added file 'l10n_br/__openerp__.py' --- l10n_br/__openerp__.py 1970-01-01 00:00:00 +0000 +++ l10n_br/__openerp__.py 2012-05-03 23:39:20 +0000 @@ -0,0 +1,69 @@ +# -*- encoding: utf-8 -*- +################################################################################# +# +# Copyright (C) 2009 Renato Lima - Akretion +# +# This program is free software: you can redistribute it and/or modify +# it under the terms of the GNU Affero General Public License as published by +# the Free Software Foundation, either version 3 of the License, or +# (at your option) any later version. +# +# This program is distributed in the hope that it will be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU General Public License for more details. +# +# You should have received a copy of the GNU General Public License +# along with this program. If not, see . +# +################################################################################# + +{ + 'name': 'Bra...

Revision history for this message
Vinicius Dittgen - Proge.com.br (vdittgen) wrote :
Download full text (13.5 KiB)

Sim Alexandre, ele faz os lançamentos de débito e crédito dos impostos
retidos, já que estes são pagos pelo cliente.

2012/5/3, Alexandre Leitão <email address hidden>:
> Vinicius,
>
> Este merge resolve o caso da retenção do ir, confins e etc pra notas de
> serviço?
>
> Atenciosamente,
> Alexandre Leitão
>
> Em 03/05/2012, às 20:40, "Vinicius Dittgen - Proge.com.br"
> <email address hidden> escreveu:
>
>> Vinicius Dittgen - Proge.com.br has proposed merging
>> lp:~vdittgen/openerp.pt-br-localiz/tax_application_behaviour into
>> lp:openerp.pt-br-localiz.
>>
>> Requested reviews:
>> OpenERP Brazil Core Team (openerp-brazil-core-team)
>>
>> For more details, see:
>> https://code.launchpad.net/~vdittgen/openerp.pt-br-localiz/tax_application_behaviour/+merge/104653
>>
>> Add tax_add,tax_retain and tax_include behavior. Can support service
>> invoices with retained taxes.
>> fix price_subtotal calc in invoice, and tax calc in sale order for tax_add
>> type(like ipi).
>> --
>> 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.
>> === added directory 'l10n_br' === added file 'l10n_br/__init__.py' ---
>> l10n_br/__init__.py 1970-01-01 00:00:00 +0000 +++
>> l10n_br/__init__.py 2012-05-03 23:39:20 +0000 @@ -0,0 +1,24 @@ +# -*-
>> encoding: utf-8 -*-
>> +#################################################################################
>> +# # +# Copyright (C) 2009 Renato Lima - Akretion # +# # +#This program is
>> free software: you can redistribute it and/or modify # +#it under the
>> terms of the GNU Affero General Public License as published by # +#the
>> Free Software Foundation, either version 3 of the License, or # +#(at your
>> option) any later version. # +# # +#This program is distributed in the
>> hope that it will be useful, # +#but WITHOUT ANY WARRANTY; without even
>> the implied warranty of # +#MERCHANTABILITY or FITNESS FOR A PARTICULAR
>> PURPOSE. See the # +#GNU General Public License for more details. # +# #
>> +#You should have received a copy of the GNU General Public License #
>> +#along with this program. If not, see . #
>> +#################################################################################
>> + +import account +import l10n_br + +#
>> vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4: + ===
>> added file 'l10n_br/__openerp__.py' --- l10n_br/__openerp__.py 1970-01-01
>> 00:00:00 +0000 +++ l10n_br/__openerp__.py 2012-05-03 23:39:20 +0000 @@
>> -0,0 +1,69 @@ +# -*- encoding: utf-8 -*-
>> +#################################################################################
>> +# +# Copyright (C) 2009 Renato Lima - Akretion +# +# This program is free
>> software: you can redistribute it and/or modify +# it under the terms of
>> the GNU Affero General Public License as published by +# the Free Software
>> Foundation, either version 3 of the License, or +# (at your option) any
>> later version. +# +# This program is distributed in the hope that it will
>> be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty
>> of +# MERCHANTABILITY or F...

Revision history for this message
Vinicius Dittgen - Proge.com.br (vdittgen) wrote :
Download full text (13.5 KiB)

Aliás, ele faz retenção de qualquer imposto, basta marcar como imposto
retido na configuração da taxa.

2012/5/3, Alexandre Leitão <email address hidden>:
> Vinicius,
>
> Este merge resolve o caso da retenção do ir, confins e etc pra notas de
> serviço?
>
> Atenciosamente,
> Alexandre Leitão
>
> Em 03/05/2012, às 20:40, "Vinicius Dittgen - Proge.com.br"
> <email address hidden> escreveu:
>
>> Vinicius Dittgen - Proge.com.br has proposed merging
>> lp:~vdittgen/openerp.pt-br-localiz/tax_application_behaviour into
>> lp:openerp.pt-br-localiz.
>>
>> Requested reviews:
>> OpenERP Brazil Core Team (openerp-brazil-core-team)
>>
>> For more details, see:
>> https://code.launchpad.net/~vdittgen/openerp.pt-br-localiz/tax_application_behaviour/+merge/104653
>>
>> Add tax_add,tax_retain and tax_include behavior. Can support service
>> invoices with retained taxes.
>> fix price_subtotal calc in invoice, and tax calc in sale order for tax_add
>> type(like ipi).
>> --
>> 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.
>> === added directory 'l10n_br' === added file 'l10n_br/__init__.py' ---
>> l10n_br/__init__.py 1970-01-01 00:00:00 +0000 +++
>> l10n_br/__init__.py 2012-05-03 23:39:20 +0000 @@ -0,0 +1,24 @@ +# -*-
>> encoding: utf-8 -*-
>> +#################################################################################
>> +# # +# Copyright (C) 2009 Renato Lima - Akretion # +# # +#This program is
>> free software: you can redistribute it and/or modify # +#it under the
>> terms of the GNU Affero General Public License as published by # +#the
>> Free Software Foundation, either version 3 of the License, or # +#(at your
>> option) any later version. # +# # +#This program is distributed in the
>> hope that it will be useful, # +#but WITHOUT ANY WARRANTY; without even
>> the implied warranty of # +#MERCHANTABILITY or FITNESS FOR A PARTICULAR
>> PURPOSE. See the # +#GNU General Public License for more details. # +# #
>> +#You should have received a copy of the GNU General Public License #
>> +#along with this program. If not, see . #
>> +#################################################################################
>> + +import account +import l10n_br + +#
>> vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4: + ===
>> added file 'l10n_br/__openerp__.py' --- l10n_br/__openerp__.py 1970-01-01
>> 00:00:00 +0000 +++ l10n_br/__openerp__.py 2012-05-03 23:39:20 +0000 @@
>> -0,0 +1,69 @@ +# -*- encoding: utf-8 -*-
>> +#################################################################################
>> +# +# Copyright (C) 2009 Renato Lima - Akretion +# +# This program is free
>> software: you can redistribute it and/or modify +# it under the terms of
>> the GNU Affero General Public License as published by +# the Free Software
>> Foundation, either version 3 of the License, or +# (at your option) any
>> later version. +# +# This program is distributed in the hope that it will
>> be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty
>> of +# MERCHANTABILITY or FITNESS FOR A P...

Revision history for this message
Fabio Negrini (fnegrini) wrote :
Download full text (14.3 KiB)

E faz o controle automático conforme a regra dos 5000,00 por período?
Considera o tipo do fornecedor nesta regra? ex.: Simples nacional não retém.

> Aliás, ele faz retenção de qualquer imposto, basta marcar como imposto
> retido na configuração da taxa.
>
> 2012/5/3, Alexandre Leitão <email address hidden>:
> > Vinicius,
> >
> > Este merge resolve o caso da retenção do ir, confins e etc pra notas de
> > serviço?
> >
> > Atenciosamente,
> > Alexandre Leitão
> >
> > Em 03/05/2012, às 20:40, "Vinicius Dittgen - Proge.com.br"
> > <email address hidden> escreveu:
> >
> >> Vinicius Dittgen - Proge.com.br has proposed merging
> >> lp:~vdittgen/openerp.pt-br-localiz/tax_application_behaviour into
> >> lp:openerp.pt-br-localiz.
> >>
> >> Requested reviews:
> >> OpenERP Brazil Core Team (openerp-brazil-core-team)
> >>
> >> For more details, see:
> >> https://code.launchpad.net/~vdittgen/openerp.pt-br-
> localiz/tax_application_behaviour/+merge/104653
> >>
> >> Add tax_add,tax_retain and tax_include behavior. Can support service
> >> invoices with retained taxes.
> >> fix price_subtotal calc in invoice, and tax calc in sale order for tax_add
> >> type(like ipi).
> >> --
> >> 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.
> >> === added directory 'l10n_br' === added file 'l10n_br/__init__.py' ---
> >> l10n_br/__init__.py 1970-01-01 00:00:00 +0000 +++
> >> l10n_br/__init__.py 2012-05-03 23:39:20 +0000 @@ -0,0 +1,24 @@ +# -*-
> >> encoding: utf-8 -*-
> >> +##########################################################################
> #######
> >> +# # +# Copyright (C) 2009 Renato Lima - Akretion # +# # +#This program is
> >> free software: you can redistribute it and/or modify # +#it under the
> >> terms of the GNU Affero General Public License as published by # +#the
> >> Free Software Foundation, either version 3 of the License, or # +#(at your
> >> option) any later version. # +# # +#This program is distributed in the
> >> hope that it will be useful, # +#but WITHOUT ANY WARRANTY; without even
> >> the implied warranty of # +#MERCHANTABILITY or FITNESS FOR A PARTICULAR
> >> PURPOSE. See the # +#GNU General Public License for more details. # +# #
> >> +#You should have received a copy of the GNU General Public License #
> >> +#along with this program. If not, see . #
> >> +##########################################################################
> #######
> >> + +import account +import l10n_br + +#
> >> vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4: + ===
> >> added file 'l10n_br/__openerp__.py' --- l10n_br/__openerp__.py
> 1970-01-01
> >> 00:00:00 +0000 +++ l10n_br/__openerp__.py 2012-05-03 23:39:20 +0000 @@
> >> -0,0 +1,69 @@ +# -*- encoding: utf-8 -*-
> >> +##########################################################################
> #######
> >> +# +# Copyright (C) 2009 Renato Lima - Akretion +# +# This program is free
> >> software: you can redistribute it and/or modify +# it under the terms of
> >> the GNU Affero General Public License as published by +# the ...

Revision history for this message
Alexandre Leitão (alexandre-leitao) wrote :

Valeu meu caroŠ vai me ajudar muito!!!!

Alexandre Leitão, Owner
OpenERP Ready Partner
Tel: (011) 3744-3170 | Mobile: (011) 8435-7021
www.infonetware.com.br <http://www.infonetware.com.br/>

On 04/05/12 11:02, "Vinicius Dittgen - Proge.com.br"
<email address hidden> wrote:

>onfiguração da tax

Revision history for this message
Vinicius Dittgen - Proge.com.br (vdittgen) wrote :
Download full text (16.1 KiB)

Fabio,

  Este merge altera o cadastro do imposto para que seja possível ter
impostos retidos e que seus lançamentos sejam feitos da forma correta.

  Sobre a alteração das alíquotas no momento da venda levando em conta os
parâmetros de *pagamentos do cliente recebido no mês* e *tipo fiscal do
cliente* deve ser feita manualmente via posição fiscal. Provavelmente
iremos estender o módulo de regra de posição fiscal para que use a posição
correta baseado nesses parâmetros.

  Neste momento já é possível lançar impostos retidos, a escolhas das
alíquotas é manual.

abs

2012/5/4 Fabio Negrini <email address hidden>

> E faz o controle automático conforme a regra dos 5000,00 por período?
> Considera o tipo do fornecedor nesta regra? ex.: Simples nacional não
> retém.
>
> > Aliás, ele faz retenção de qualquer imposto, basta marcar como imposto
> > retido na configuração da taxa.
> >
> > 2012/5/3, Alexandre Leitão <email address hidden>:
> > > Vinicius,
> > >
> > > Este merge resolve o caso da retenção do ir, confins e etc pra notas de
> > > serviço?
> > >
> > > Atenciosamente,
> > > Alexandre Leitão
> > >
> > > Em 03/05/2012, às 20:40, "Vinicius Dittgen - Proge.com.br"
> > > <email address hidden> escreveu:
> > >
> > >> Vinicius Dittgen - Proge.com.br has proposed merging
> > >> lp:~vdittgen/openerp.pt-br-localiz/tax_application_behaviour into
> > >> lp:openerp.pt-br-localiz.
> > >>
> > >> Requested reviews:
> > >> OpenERP Brazil Core Team (openerp-brazil-core-team)
> > >>
> > >> For more details, see:
> > >> https://code.launchpad.net/~vdittgen/openerp.pt-br-
> > localiz/tax_application_behaviour/+merge/104653
> > >>
> > >> Add tax_add,tax_retain and tax_include behavior. Can support service
> > >> invoices with retained taxes.
> > >> fix price_subtotal calc in invoice, and tax calc in sale order for
> tax_add
> > >> type(like ipi).
> > >> --
> > >> 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.
> > >> === added directory 'l10n_br' === added file 'l10n_br/__init__.py' ---
> > >> l10n_br/__init__.py 1970-01-01 00:00:00 +0000 +++
> > >> l10n_br/__init__.py 2012-05-03 23:39:20 +0000 @@ -0,0 +1,24 @@ +# -*-
> > >> encoding: utf-8 -*-
> > >>
> +##########################################################################
> > #######
> > >> +# # +# Copyright (C) 2009 Renato Lima - Akretion # +# # +#This
> program is
> > >> free software: you can redistribute it and/or modify # +#it under the
> > >> terms of the GNU Affero General Public License as published by # +#the
> > >> Free Software Foundation, either version 3 of the License, or # +#(at
> your
> > >> option) any later version. # +# # +#This program is distributed in the
> > >> hope that it will be useful, # +#but WITHOUT ANY WARRANTY; without
> even
> > >> the implied warranty of # +#MERCHANTABILITY or FITNESS FOR A
> PARTICULAR
> > >> PURPOSE. See the # +#GNU General Public License for more details. #
> +# #
> > >> +#You should have received a copy of the GNU General Public License #
> > >> +#along with this program. If not, see . ...

Revision history for this message
Fabio Negrini (fnegrini) wrote :

Vinícius,

Obrigado pelo esclarecimento e parabéns pela iniciativa.

Abs

Revision history for this message
Raphaël Valyi - http://www.akretion.com (rvalyi) wrote :

Ola Vinicius,

primeiro obrigado por levar esse assunto de vola na localizaçao.

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?).

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.

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.

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.

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

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.

Revision history for this message
Vinicius Dittgen - Proge.com.br (vdittgen) wrote :
Download full text (3.6 KiB)

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 par...

Read more...

Revision history for this message
Renato Lima - http://www.akretion.com (renatonlima) wrote :

Vinicius,

Como conversado nas mensagens anteriores, o l10n_br devemos manté-lo no addons para não dificultar a instalação e posteriormente o desenvolvimento já que teria que também um l10n_br no addons e outro na localização, então sugiro algumas mudanças:

1. Não incluir o módulo l10n_br na branch da localização;
2. Deixa o comportamento e o flag já implementando de tax_discount;
3. Deixar somente o desenvolvimento no calculo de impostos retidos.

Futuramente podemos tratar do item 2, até porque existe a possibilidade de colocarmos no core do OpenERP um flag com a função do tax_discount e remover isso da localização

review: Needs Resubmitting

Unmerged revisions

316. By Vinicius Dittgen - Proge.com.br

Add tax_add,tax_retain and tax_include behavior, support to service invoices.
fix price_subtotal calc in invoice, and tax calc in sale order.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.