Merge lp:~ivo-kracht/launchpad/bug-824292 into lp:launchpad

Proposed by Ivo Kracht
Status: Merged
Approved by: Abel Deuring
Approved revision: no longer in the source branch.
Merged at revision: 15517
Proposed branch: lp:~ivo-kracht/launchpad/bug-824292
Merge into: lp:launchpad
Diff against target: 39 lines (+2/-7)
3 files modified
lib/lp/soyuz/stories/soyuz/xx-queue-pages-motu.txt (+1/-1)
lib/lp/soyuz/stories/soyuz/xx-queue-pages.txt (+1/-1)
lib/lp/soyuz/templates/distroseries-portlet-latestuploads.pt (+0/-5)
To merge this branch: bzr merge lp:~ivo-kracht/launchpad/bug-824292
Reviewer Review Type Date Requested Status
Raphaël Badin (community) Approve
Francesco Banconi (community) code* Approve
Review via email: mp+112352@code.launchpad.net

Commit message

Only one "show uploads" link on distro series pages

Description of the change

Deleted some lines in the template that implement the link. No tests check it so I didn't have to change anything.

Pre-imp call with adeuring

test:
./bin/test soyuz -vvt xx-distroseries-index.txt

= Launchpad lint =

Checking for conflicts and issues in changed files.

Linting changed files:
  lib/lp/soyuz/templates/distroseries-portlet-latestuploads.pt

To post a comment you must log in.
Revision history for this message
Francesco Banconi (frankban) wrote :

Looks good Ivo, thanks for deleting a duplicated link.

review: Approve (code*)
Revision history for this message
Raphaël Badin (rvb) wrote :

Looks good.

[0]

I think you forgot to attach the related bug to that MP.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'lib/lp/soyuz/stories/soyuz/xx-queue-pages-motu.txt'
--- lib/lp/soyuz/stories/soyuz/xx-queue-pages-motu.txt 2011-12-30 06:14:56 +0000
+++ lib/lp/soyuz/stories/soyuz/xx-queue-pages-motu.txt 2012-06-28 12:59:59 +0000
@@ -38,7 +38,7 @@
3838
39 >>> motu_browser.open(39 >>> motu_browser.open(
40 ... "http://launchpad.dev/ubuntu/breezy-autotest/")40 ... "http://launchpad.dev/ubuntu/breezy-autotest/")
41 >>> motu_browser.getLink("Show uploads").click()41 >>> motu_browser.getLink("All uploads").click()
42 >>> print_queue(motu_browser.contents)42 >>> print_queue(motu_browser.contents)
43 Package Version Component Section Priority Sets Pocket When43 Package Version Component Section Priority Sets Pocket When
44 netapplet...ddtp... - Release 2006...44 netapplet...ddtp... - Release 2006...
4545
=== modified file 'lib/lp/soyuz/stories/soyuz/xx-queue-pages.txt'
--- lib/lp/soyuz/stories/soyuz/xx-queue-pages.txt 2012-01-06 11:08:30 +0000
+++ lib/lp/soyuz/stories/soyuz/xx-queue-pages.txt 2012-06-28 12:59:59 +0000
@@ -56,7 +56,7 @@
5656
57 >>> anon_browser.open(57 >>> anon_browser.open(
58 ... "http://launchpad.dev/ubuntu/breezy-autotest/")58 ... "http://launchpad.dev/ubuntu/breezy-autotest/")
59 >>> anon_browser.getLink("Show uploads").click()59 >>> anon_browser.getLink("All uploads").click()
6060
61 >>> anon_browser.getControl(61 >>> anon_browser.getControl(
62 ... name="queue_state", index=0).displayValue62 ... name="queue_state", index=0).displayValue
6363
=== modified file 'lib/lp/soyuz/templates/distroseries-portlet-latestuploads.pt'
--- lib/lp/soyuz/templates/distroseries-portlet-latestuploads.pt 2012-06-11 00:03:25 +0000
+++ lib/lp/soyuz/templates/distroseries-portlet-latestuploads.pt 2012-06-28 12:59:59 +0000
@@ -28,9 +28,4 @@
28 There are no recent source package uploads.28 There are no recent source package uploads.
29 </p>29 </p>
3030
31 <ul class="horizontal">
32 <li>
33 <a tal:replace="structure context/menu:overview/queue/fmt:link" />
34 </li>
35 </ul>
36</div>31</div>