Launchpad itself

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

  1. bug-652149
  2. Merge into devel
Revision history for this message
Jeroen T. Vermeulen (jtv) wrote :

Looks fine, apart from a few things we discussed on IRC:
 * Render & search views in your tests using BeautifulSoup(view())
 * Multi-line strings would be nicer than ("line" "line") in tests.
 * The view inherits from both LaunchpadFormView and, indirectly, LaunchpadView.

review: Approve (code)

« Back to merge proposal