For me, it doesn't look like a good addition for 2 reasons:
- using year of sequence in invoice sequence is wrong because it is not the accounting date. The right solution is to hardcode the year in the sequence template.
- it doesn't improve the current code.
Instead the code should be improved to allow customization of the parameters by third modules.
Cédric Krieradded 1 deleted label and removed 1 deleted label
what do you mean?
No notification when a review is unassigned?
"""
Instead the code should be improved to allow customization of the parameters by third modules.
"""
Is what is happening in the quoted review.
"""
Change the issue for just customisation.
"""
is what i did, you changed the title back to "improve..."
New changeset d4c31394f786 by Sergi Almacellas Abellana in branch 'default':
Allow to customize the substitutions used on sequence
http://hg.tryton.org/trytond/rev/d4c31394f786
Roundup Robotadded 1 deleted label and removed 1 deleted label