credativ team mailing list archive
-
credativ team
-
Mailing list archive
-
Message #05284
Re: [Merge] lp:~sylvain-legal/openupgrade-addons/migration-analytic into lp:openupgrade-addons
Thanks for your quick review !
1/ You're right.
--> changed commited.
2/ I'm not sure. To test I did the following :
A : in a 6.1 : "update account_analytic_account set type=null; "
B : migration -> 7.0
C : the 'type' values are 'normal' ; --> OK
A' in a 6.1 : "update account_analytic_account set type='wrong_value'; "
B' : migration -> 7.0
C' : the 'type' value still is 'wrong_value' ; --> FAILED
Conclusion, if the database values are OK, the migration will do the following :
- Null, 'normal' -> 'normal'
- 'view' -> 'view'
3/ You're right too !
--> changed commited.
--
https://code.launchpad.net/~sylvain-legal/openupgrade-addons/migration-analytic/+merge/174751
Your team OpenUpgrade Committers is subscribed to branch lp:openupgrade-addons.
References