Code review comment for lp:~jamalta/launchpad/destination-ppa-498643

Revision history for this message
Jamal Fanaian (jamalta) wrote :

= 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:
  lib/lp/soyuz/browser/archive.py
  lib/lp/soyuz/stories/ppa/xx-copy-packages.txt

« Back to merge proposal