openerp-community-reviewer team mailing list archive

Thread
Date

Re: [Merge] lp:~numerigraphe-team/report-print-send/7.0-user-browse-lp1116672-ls into lp:report-print-send

To: mp+216356@xxxxxxxxxxxxxxxxxx
From: Pedro Manuel Baeza <pedro.baeza@xxxxxxxxx>
Date: Thu, 17 Apr 2014 16:38:44 -0000
In-reply-to: <20140417160431.5995.98972.launchpad@ackee.canonical.com>
Reply-to: mp+216356@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Approve code review

Uhm, I'm not aware about that deprecation, but it can be. I always avoid single ID on read, write, and so on overrides, but I see no problem with browse method. Do you know anyone that overrides this method?

In summary, I have no hard feeling about any form, so please let it as you want.

I find this discussion very instructive :)

Regards.
-- 
https://code.launchpad.net/~numerigraphe-team/report-print-send/7.0-user-browse-lp1116672-ls/+merge/216356
Your team Report Printing and Sending Core Editors is subscribed to branch lp:report-print-send.

References

[Merge] lp:~numerigraphe-team/report-print-send/7.0-user-browse-lp1116672-ls into lp:report-print-send
From: Lionel Sausin - Numérigraphe, 2014-04-17