← Back to team overview

openerp-dev-web team mailing list archive

Re: [Merge] lp:~openerp-dev/openobject-client-web/6.0-opw-4396-sma into lp:openobject-client-web/6.0

 

Review: Needs Information
* Did the customer provide the patch himself? in the email above, or was the patch re-done from scratch?
* If the patch was the customer's, he should be credited (by using —author)
* The "{0,1}" RE pattern can (and should) be replaced with "?".
* Shouldn't we add support for a positive prefix just in case?
-- 
https://code.launchpad.net/~openerp-dev/openobject-client-web/6.0-opw-4396-sma/+merge/53227
Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openobject-client-web/6.0-opw-4396-sma.



Follow ups

References