openerp-dev-web team mailing list archive
-
openerp-dev-web team
-
Mailing list archive
-
Message #04269
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