Hi,

The patches that removes the Property fields is ready for testing. It
is passing all the tests.

https://bugs.tryton.org/issue2349

I would like to commit it ASAP to get enough time to find all bugs
before the release.
Indeed I do not expect a full review of the patches (it is quite big but
almost always based the same pattern) but more a check of the design.
The MultiValue field is designed since a long time already. The main new
thing is the choice of how to define the Value model. For example is the
grouping of value right (like the sale.configuration.sequence)? Is the
choice of standard Value or the company one right?
It will be good also to have some migration test.
It is also important to read my last messages on the bug-tracker that
describes some design choice made when facing issues.

Finally, do not forget that it is not yet the removal of company rule.
It will be done in a second step with https://bugs.tryton.org/issue4080

Thanks,
-- 
Cédric Krier - B2CK SPRL
Email/Jabber: cedric.kr...@b2ck.com
Tel: +32 472 54 46 59
Website: http://www.b2ck.com/

-- 
You received this message because you are subscribed to the Google Groups 
"tryton-dev" group.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/tryton-dev/20170328221201.GA6408%40tetsuo.

Reply via email to