Merge ~pfsmorigo/ubuntu-cve-tracker:pfsmorigo/is_visible into ubuntu-cve-tracker:master

Proposed by Paulo Flabiano Smorigo
Status: Merged
Merge reported by: Emilia Torino
Merged at revision: 43654820b6c585c18bc031ccf79d14194c30d00b
Proposed branch: ~pfsmorigo/ubuntu-cve-tracker:pfsmorigo/is_visible
Merge into: ubuntu-cve-tracker:master
Diff against target: 30 lines (+5/-1)
1 file modified
scripts/publish-usn-to-website-api.py (+5/-1)
Reviewer Review Type Date Requested Status
Emilia Torino Approve
Review via email: mp+402360@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Emilia Torino (emitorino) wrote :

LGTM but let's wait for the web team before merging so we can confirm this will be the new key name

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/scripts/publish-usn-to-website-api.py b/scripts/publish-usn-to-website-api.py
2index d876eef..9b1be4b 100755
3--- a/scripts/publish-usn-to-website-api.py
4+++ b/scripts/publish-usn-to-website-api.py
5@@ -175,7 +175,7 @@ for notice_id, notice in payload:
6 }
7 )
8
9- for name, info in packages["binaries"].items():
10+ for name, info in packages["allbinaries"].items():
11 if "sources" in packages:
12 source_link, version_link = guess_binary_links(
13 name, info["version"], packages["sources"]
14@@ -183,12 +183,16 @@ for notice_id, notice in payload:
15 else:
16 source_link = ""
17 version_link = ""
18+ is_visible = "false"
19+ if name in packages["binaries"]:
20+ is_visible = "true"
21 release_package_info = {
22 "name": name,
23 "version": info["version"],
24 "is_source": "false",
25 "source_link": source_link,
26 "version_link": version_link,
27+ "is_visible": is_visible,
28 }
29 if "pocket" in info:
30 release_package_info["pocket"] = info["pocket"]

Subscribers

People subscribed via source and target branches