Merge lp:~mwhudson/launchpad/vostok-add-root into lp:launchpad
Proposed by
Michael Hudson-Doyle
Status: | Merged |
---|---|
Approved by: | Tim Penhey |
Approved revision: | no longer in the source branch. |
Merged at revision: | 11299 |
Proposed branch: | lp:~mwhudson/launchpad/vostok-add-root |
Merge into: | lp:launchpad |
Prerequisite: | lp:~mwhudson/launchpad/vostok-add-layer |
Diff against target: |
238 lines (+140/-5) 10 files modified
lib/lp/vostok/browser/__init__.py (+4/-0) lib/lp/vostok/browser/configure.zcml (+22/-0) lib/lp/vostok/browser/root.py (+15/-0) lib/lp/vostok/browser/tests/__init__.py (+4/-0) lib/lp/vostok/browser/tests/request.py (+19/-0) lib/lp/vostok/browser/tests/test_root.py (+38/-0) lib/lp/vostok/configure.zcml (+7/-0) lib/lp/vostok/publisher.py (+12/-4) lib/lp/vostok/templates/root.pt (+3/-0) lib/lp/vostok/tests/test_publisher.py (+16/-1) |
To merge this branch: | bzr merge lp:~mwhudson/launchpad/vostok-add-root |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Tim Penhey (community) | Approve | ||
Review via email: mp+31239@code.launchpad.net |
Description of the change
Hi,
This branch adds a custom root object for the vostok vhost. This makes us more
separate from the other Launchpad publication stuff, which given the goal of
vostok is a good thing (tm).
Cheers,
mwh
To post a comment you must log in.
> class VostokRootView( LaunchpadView) :
> pass
Shouldn't you at least have
__used_for__ = IVostokRoot
Instead of:
(VostokRoot( ), VostokTestReque st()), name='+index') initialize( )
view = getMultiAdapter(
view.
Can you alter the get_initialized _view function?
> class IVostokRoot( Interface) : # might need to inherit from some IRoot thing
You can probably lose the comment.
Also... the root.pt needs some work :-)
Why is the IOpenLaunchBag) .clear( )
getUtility(
needed in test_root_object?