Merge lp:~jamalta/launchpad/destination-ppa-498643 into lp:launchpad
Proposed by
Jamal Fanaian
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Brad Crittenden | ||||
Approved revision: | not available | ||||
Merged at revision: | not available | ||||
Proposed branch: | lp:~jamalta/launchpad/destination-ppa-498643 | ||||
Merge into: | lp:launchpad | ||||
Diff against target: |
58 lines (+6/-6) 3 files modified
lib/lp/soyuz/browser/archive.py (+1/-1) lib/lp/soyuz/browser/tests/archive-views.txt (+3/-3) lib/lp/soyuz/stories/ppa/xx-copy-packages.txt (+2/-2) |
||||
To merge this branch: | bzr merge lp:~jamalta/launchpad/destination-ppa-498643 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Brad Crittenden (community) | code | Approve | |
Review via email: mp+17628@code.launchpad.net |
Commit message
Changed the Destination PPA options to read 'PPA Name (username/ppa)' in copy packages page to help distinguish between similarly named PPAs.
To post a comment you must log in.
= Summary =
Add PPA url to the Destination PPA widget when copying contents from a PPA, to make each option unique (bug 498643).
== Proposed fix ==
Appended the PPA url to each widget option:
Name of PPA (user/ppaname)
== Pre-implementation notes ==
Following style suggestion by Jullian Edwards in the bug comments.
== Tests ==
% bin/test -vvct xx-copy-packages
== Demo and Q/A ==
Must login as user with an existing PPA (u: <email address hidden> p: test).
* https:/ /launchpad. dev/~cprov/ +archive/ ppa/+copy- packages
== Launchpad lint ==
Checking for conflicts. and issues in doctests and templates.
Running jslint, xmllint, pyflakes, and pylint.
Using normal rules.
Linting changed files: soyuz/browser/ archive. py soyuz/stories/ ppa/xx- copy-packages. txt
lib/lp/
lib/lp/