Merge lp:~seb128/ubuntu-archive-scripts/display-migrate-after into lp:ubuntu-archive-scripts

Proposed by Sebastien Bacher
Status: Merged
Approved by: Łukasz Zemczak
Approved revision: 269
Merged at revision: 269
Proposed branch: lp:~seb128/ubuntu-archive-scripts/display-migrate-after
Merge into: lp:ubuntu-archive-scripts
Diff against target: 22 lines (+9/-0)
1 file modified
templates/team-report.html (+9/-0)
To merge this branch: bzr merge lp:~seb128/ubuntu-archive-scripts/display-migrate-after
Reviewer Review Type Date Requested Status
Łukasz Zemczak Approve
Review via email: mp+380276@code.launchpad.net

Commit message

team-report: displays the migrate-after information for the candidates

Description of the change

If a candidate package has migrate-after items mention it as
'candidate but also requires <items> to migrate'
which items is a space separate list of package (url pointing to the main report)

To post a comment you must log in.
Revision history for this message
Łukasz Zemczak (sil2100) wrote :

Ok, I think this looks okay, also on the example report that got generated. +1

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'templates/team-report.html'
--- templates/team-report.html 2020-03-04 08:53:35 +0000
+++ templates/team-report.html 2020-03-05 10:19:25 +0000
@@ -108,9 +108,18 @@
108 </li>108 </li>
109 {% endif %}109 {% endif %}
110 {% if d["is-candidate"] %}110 {% if d["is-candidate"] %}
111 {% if d.get('dependencies', {}).get('migrate-after', []) %}
112 <li><b>candidate</b> but also requires
113 {% for depmigrate in d.get('dependencies', {}).get('migrate-after', []) %}
114 <a href="http://people.canonical.com/~ubuntu-archive/proposed-migration/update_excuses.html#{{ depmigrate }}">{{ depmigrate }}</a>
115 {%- if not loop.last %}, {% endif %}
116 {% endfor %}
117 to migrate</li>
118 {% else %}
111 <li><b>candidate</b></li>119 <li><b>candidate</b></li>
112 {% endif %}120 {% endif %}
113 {% endif %}121 {% endif %}
122 {% endif %}
114 </ul>123 </ul>
115 </li>124 </li>
116 {% endfor %}125 {% endfor %}

Subscribers

People subscribed via source and target branches