Code review comment for lp:~numerigraphe-team/report-print-send/7.0-user-browse-lp1116672-ls
Revision history for this message
| Pedro Manuel Baeza (pedro.baeza) wrote | # |
review:
Approve
(code review)
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
f33e652
(Get the code!)
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.