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
1=== modified file 'templates/team-report.html'
2--- templates/team-report.html 2020-03-04 08:53:35 +0000
3+++ templates/team-report.html 2020-03-05 10:19:25 +0000
4@@ -108,9 +108,18 @@
5 </li>
6 {% endif %}
7 {% if d["is-candidate"] %}
8+ {% if d.get('dependencies', {}).get('migrate-after', []) %}
9+ <li><b>candidate</b> but also requires
10+ {% for depmigrate in d.get('dependencies', {}).get('migrate-after', []) %}
11+ <a href="http://people.canonical.com/~ubuntu-archive/proposed-migration/update_excuses.html#{{ depmigrate }}">{{ depmigrate }}</a>
12+ {%- if not loop.last %}, {% endif %}
13+ {% endfor %}
14+ to migrate</li>
15+ {% else %}
16 <li><b>candidate</b></li>
17 {% endif %}
18 {% endif %}
19+ {% endif %}
20 </ul>
21 </li>
22 {% endfor %}

Subscribers

People subscribed via source and target branches