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

Proposed by Sebastien Bacher
Status: Merged
Merged at revision: 298
Proposed branch: lp:~seb128/ubuntu-archive-scripts/noncandid-migrate-after
Merge into: lp:ubuntu-archive-scripts
Diff against target: 25 lines (+4/-5)
1 file modified
templates/team-report.html (+4/-5)
To merge this branch: bzr merge lp:~seb128/ubuntu-archive-scripts/noncandid-migrate-after
Reviewer Review Type Date Requested Status
Steve Langasek Approve
Review via email: mp+391117@code.launchpad.net

Commit message

The migrate after information is not restricted to candidate items

Description of the change

example report on

https://people.canonical.com/~seb128/reportafter.html

the xdg-desktop-portal has a rational there when it has no information on the current report

To post a comment you must log in.
Revision history for this message
Steve Langasek (vorlon) :
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-09-17 19:50:01 +0000
3+++ templates/team-report.html 2020-09-22 10:16:47 +0000
4@@ -138,17 +138,16 @@
5 {% endfor %}
6 {% endif %}
7 {% if d["is-candidate"] %}
8+ <li><b>candidate</b></li>
9+ {% endif %}
10+ {% endif %}
11 {% if d.get('dependencies', {}).get('migrate-after', []) %}
12- <li><b>candidate</b> but also requires
13+ <li>Requires
14 {% for depmigrate in d.get('dependencies', {}).get('migrate-after', []) %}
15 <a href="http://people.canonical.com/~ubuntu-archive/proposed-migration/update_excuses.html#{{ depmigrate }}">{{ depmigrate }}</a>
16 {%- if not loop.last %}, {% endif %}
17 {% endfor %}
18 to migrate</li>
19- {% else %}
20- <li><b>candidate</b></li>
21- {% endif %}
22- {% endif %}
23 {% endif %}
24 </ul>
25 </li>

Subscribers

People subscribed via source and target branches