openerp-dev-web team mailing list archive
-
openerp-dev-web team
-
Mailing list archive
-
Message #04321
Re: [Merge] lp:~openerp-dev/openobject-client-web/6.0-opw-4396-sma into lp:openobject-client-web/6.0
> * 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)
No, customer didn't provide any patch. Its done from scratch.
> * The "{0,1}" RE pattern can (and should) be replaced with "?".
> * Shouldn't we add support for a positive prefix just in case?
For this i have committed changes, no need to check "{0,1}" in re.
Please check and if anything missing tell me.
--
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.
References