← Back to team overview

credativ team mailing list archive

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