Merge lp:~seb128/ubuntu-archive-scripts/report_blocked_hints into lp:ubuntu-archive-scripts

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

Commit message

generate-team-p-m: indicates when something is blocked by an hint

Includes also the hint owner

Description of the change

Example of the current report with the change applied, see the first items in the foundations section

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

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

Ok, I think that makes sense. Thanks for providing example output!

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-04-07 09:39:31 +0000
3+++ templates/team-report.html 2020-04-08 15:51:49 +0000
4@@ -111,6 +111,19 @@
5 {% endfor %}
6 </li>
7 {% endif %}
8+ {% if d["hints"] %}
9+ {% for hint in d["hints"] %}
10+ {% if hint["hint-type"] %}
11+ {% if hint["hint-type"] == "block" %}
12+ <li>Blocked by hint
13+ {% if hint["hint-from"] %}
14+ from {{ hint["hint-from"] }}
15+ {% endif %}
16+ </li>
17+ {% endif %}
18+ {% endif %}
19+ {% endfor %}
20+ {% endif %}
21 {% if d["is-candidate"] %}
22 {% if d.get('dependencies', {}).get('migrate-after', []) %}
23 <li><b>candidate</b> but also requires

Subscribers

People subscribed via source and target branches