launchpad-reviewers team mailing list archive
-
launchpad-reviewers team
-
Mailing list archive
-
Message #02177
[Merge] lp:~wallyworld/launchpad/mp-link-not-visible into lp:launchpad
The proposal to merge lp:~wallyworld/launchpad/mp-link-not-visible into lp:launchpad has been updated.
Description changed to:
= Summary =
Small fix to always show the Propose for Merging link on the branch page even if the user is not logged in.
They will just be prompted to login before being redirected to the right place. This is how other similar
links on that page work eg Link to Bug
= Implementation =
Remove @enabled_with_permission('launchpad.AnyPerson') decorator from register_merge()
= Tests =
Added to xx-propose-for-merging doc test to check link is there even if not logged in.
= Launchpad lint =
Checking for conflicts and issues in changed files.
Linting changed files:
lib/lp/code/browser/branch.py
lib/lp/code/stories/branches/xx-propose-for-merging.txt
--
https://code.launchpad.net/~wallyworld/launchpad/mp-link-not-visible/+merge/43850
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~wallyworld/launchpad/mp-link-not-visible into lp:launchpad.
References