On 2015/03/19 15:30:40, ced wrote: > Please don't make nested patch when there is few ...
8 years, 6 months ago
(2015-03-19 15:32:53 UTC)
#4
On 2015/03/19 15:30:40, ced wrote:
> Please don't make nested patch when there is few modules.
> We need flake8 running.
OK I'll make two patches: one for account_statement_sepa and another for the
account_sepa_message (it must be nested because it impacts two modules: the new
account_sepa_message and account_payment_sepa).
On 2015/03/19 15:32:53, nicoe wrote: > On 2015/03/19 15:30:40, ced wrote: > > Please don't ...
8 years, 6 months ago
(2015-03-19 16:04:13 UTC)
#5
On 2015/03/19 15:32:53, nicoe wrote:
> On 2015/03/19 15:30:40, ced wrote:
> > Please don't make nested patch when there is few modules.
> > We need flake8 running.
>
> OK I'll make two patches: one for account_statement_sepa and another for the
> account_sepa_message (it must be nested because it impacts two modules: the
new
> account_sepa_message and account_payment_sepa).
2 modules is not enough to use nested and drop validation tools.
New changeset 31d963d802f8 by Cédric Krier in branch 'default': Add account_statment_sepa module https://hg.tryton.org/modules/account_statement_sepa/rev/31d963d802f8
Issue 12011002: tryton-env: Add account_statment_sepa module
(Closed)
Created 8 years, 6 months ago by nicoe
Modified 1 year ago
Reviewers: Okko, reviewbot, rietveld-bot_tryton.org
Base URL:
Comments: 22