← 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

 

> * 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