Tryton - Issues



Title Manage chart of account update per account
Priority feature Status chatting
Superseder Nosy List Timitos, ced, pokoli, reviewbot, roundup-bot
Type feature request Components account, account_eu
Assigned To pokoli Keywords review
Reviews 44501002,43911002,43241002
View: 44501002, 43911002, 43241002

Created on 2017-11-05.20:58:45 by ced, last changed by ced.

msg42816 (view) Author: [hidden] (ced) (Tryton committer) (Tryton translator) Date: 2018-08-09.09:31:33
I re-open it because the TaxRule and TaxRuleLine are missing the template_override mechanism.
New changeset 7438a9ede821 by Sergi Almacellas Abellana in branch 'default':
Remove readonly by template on children fields
review43241002 updated at
msg38961 (view) Author: [hidden] (pokoli) (Tryton committer) (Tryton translator) Date: 2018-03-13.12:33:42
Here is review43241002 that removes the children readonly
msg38936 (view) Author: [hidden] (ced) (Tryton committer) (Tryton translator) Date: 2018-03-10.19:53:02
I re-open it because the children fields should not be readonly by template.
New changeset 19c8d4af92f3 by Sergi Almacellas Abellana in branch 'default':
Manage template overriden field
New changeset 622fea89afd3 by Sergi Almacellas Abellana in branch 'default':
Manage chart update per record
review44501002 updated at
review43911002 updated at
review44501002 updated at
review44501002 updated at
review44501002 updated at
review44501002 updated at
review44501002 updated at
review43911002 updated at
review44501002 updated at
New review44501002 at
msg37295 (view) Author: [hidden] (pokoli) (Tryton committer) (Tryton translator) Date: 2017-12-14.10:53:12
I added review44501002 which implements it on the account model. 

I will complete to other models, once I get some feedback.
msg37293 (view) Author: [hidden] (Timitos) Date: 2017-12-14.10:32:47
Nice feature. I agree that we should do it for all models.

There should be a detailed help text for the checkbox to disable the readonly as the user needs to know about the implication of setting the value to True.
msg37292 (view) Author: [hidden] (pokoli) (Tryton committer) (Tryton translator) Date: 2017-12-14.10:23:10
Indeed, I'm wondering if we should not apply only to account, but for all models that are linked to a template: Account Type, Tax, Rule and line.
msg36719 (view) Author: [hidden] (ced) (Tryton committer) (Tryton translator) Date: 2017-11-05.20:58:44
It will be better to make it explicit when the user wants to change an account linked to a template.
I think the best improvement will be:

- Make field readonly when there is a template filled
- Add a checkbox to disable the readonly to allow edition
- if the checkbox is checked the update wizard will skip this account (type, tax etc.)
Date User Action Args
2018-08-09 09:31:33cedsetstatus: resolved -> chatting
messages: + msg42816
2018-03-26 10:22:58roundup-botsetstatus: testing -> resolved
messages: + msg39370
2018-03-13 12:50:45reviewbotsetmessages: + msg38963
2018-03-13 12:33:46pokolisetreviews: 44501002,43911002 -> 44501002,43911002,43241002
2018-03-13 12:33:42pokolisetstatus: chatting -> testing
messages: + msg38961
2018-03-10 19:53:02cedsetstatus: resolved -> chatting
messages: + msg38936
2018-03-09 11:42:55roundup-botsetmessages: + msg38887
2018-03-09 11:42:44roundup-botsetstatus: testing -> resolved
nosy: + roundup-bot
messages: + msg38886
2018-03-06 15:46:47reviewbotsetmessages: + msg38821
2018-03-06 15:46:45reviewbotsetmessages: + msg38820

Showing 10 items. Show all history (warning: this could be VERY long)