openerp-dev-web team mailing list archive

[xrg <xrg@xxxxxxxxx>]

On Wednesday 02 March 2011, you wrote:
> Jay Vora (OpenERP) has proposed merging
> lp:~openerp-dev/openobject-addons/6.0-bug-718741-jvo into
> lp:openobject-addons/6.0.
> 

Some comments, after reading /only/ the diff:
at ~491, I can read "To Basic Paysble Salary / " . It has a typo, and is not 
translated.. the "By basic salary" line, above, too.

~67: "will be Paid,to be used for" .. Get used to having a space after any 
punctuation marks. Not a real mistake, but a habit.
Also, what I personally do, is always to put a space between "code" snippets 
and punctuation, or between special punctuation that would result in smileys.
Example:
  Please go to /var/spool/mail . (see, the dot is not connected to the path, 
for clarity)
  Put an explanation in a parenthesis (explanation: ). see, break the smiley



-- 
Say NO to spam and viruses. Stop using Microsoft Windows!

https://code.launchpad.net/~openerp-dev/openobject-addons/6.0-bug-718741-jvo/+merge/51861
Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openobject-addons/6.0-bug-718741-jvo.