Code review comment for lp:~pedro.baeza/account-financial-tools/7.0-account-renumber-menu-fix

Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :

Thanks! Looks good, in general.

Do you mean to say that members of the accountant group should be able to perform a renumbering? I'd say this should be restricted to accounting managers at least. Don't you think so?

And pleasee take the opportunity to change the 'o' in line 79 to 'or'.

review: Needs Information

« Back to merge proposal