Code review comment for lp:~savoirfairelinux-openerp/banking-addons/loose-coupling

Revision history for this message
Virgil Dupras (hsoft) wrote :

Alright, thanks for the explanations. I guess I'll make the state field read-only, even if I don't really get the point of it since there's not view for the model, but if it's a convention...

There's only #2 which I don't plan to address since it's a purely subjective matter. I'm a big fan of "Sparse is better than dense" and I find 3 sparse lines more readable than 1 dense line.

« Back to merge proposal