Launchpad itself

Code review comment for lp:~bac/launchpad/bug-607733

  1. bug-607733
  2. Merge into devel
Revision history for this message
Curtis Hovey (sinzui) wrote :

I Brad.

As I said on IRC. I see a permission change on the model, but a test of the view, which I know has its own permission settings. I expected to see a test of the model permissions, perhaps check_permission('zope.Public', subscription).

review: Needs Fixing (code)

« Back to merge proposal