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
=== modified file 'templates/team-report.html'
--- templates/team-report.html 2020-09-17 19:50:01 +0000
+++ templates/team-report.html 2020-09-22 10:16:47 +0000
@@ -138,17 +138,16 @@
138 {% endfor %}138 {% endfor %}
139 {% endif %}139 {% endif %}
140 {% if d["is-candidate"] %}140 {% if d["is-candidate"] %}
141 <li><b>candidate</b></li>
142 {% endif %}
143 {% endif %}
141 {% if d.get('dependencies', {}).get('migrate-after', []) %}144 {% if d.get('dependencies', {}).get('migrate-after', []) %}
142 <li><b>candidate</b> but also requires 145 <li>Requires
143 {% for depmigrate in d.get('dependencies', {}).get('migrate-after', []) %}146 {% for depmigrate in d.get('dependencies', {}).get('migrate-after', []) %}
144 <a href="http://people.canonical.com/~ubuntu-archive/proposed-migration/update_excuses.html#{{ depmigrate }}">{{ depmigrate }}</a>147 <a href="http://people.canonical.com/~ubuntu-archive/proposed-migration/update_excuses.html#{{ depmigrate }}">{{ depmigrate }}</a>
145 {%- if not loop.last %}, {% endif %}148 {%- if not loop.last %}, {% endif %}
146 {% endfor %}149 {% endfor %}
147 to migrate</li>150 to migrate</li>
148 {% else %}
149 <li><b>candidate</b></li>
150 {% endif %}
151 {% endif %}
152 {% endif %}151 {% endif %}
153 </ul>152 </ul>
154 </li>153 </li>

Subscribers

People subscribed via source and target branches