Merge lp:~robru/queuebot/no-more-requestid into lp:queuebot

Proposed by Robert Bruce Park
Status: Merged
Merged at revision: 110
Proposed branch: lp:~robru/queuebot/no-more-requestid
Merge into: lp:queuebot
Diff against target: 21 lines (+2/-2)
1 file modified
plugins/landing.py (+2/-2)
To merge this branch: bzr merge lp:~robru/queuebot/no-more-requestid
Reviewer Review Type Date Requested Status
Stéphane Graber Pending
Review via email: mp+309302@code.launchpad.net

Description of the change

"request_id" field has been renamed to "id"

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'plugins/landing.py'
2--- plugins/landing.py 2016-10-25 19:36:18 +0000
3+++ plugins/landing.py 2016-10-25 22:46:52 +0000
4@@ -8,7 +8,7 @@
5 ROOT = "https://bileto.ubuntu.com/"
6 URL = ROOT + "v1/active"
7
8-PERMALINK = "{}#/ticket/{{request_id}}".format(ROOT)
9+PERMALINK = "{}#/ticket/{{id}}".format(ROOT)
10 TESTED = "{{landers}}, {} QA Signoff: {{qa_signoff}}".format(PERMALINK)
11 STATUS = "{{landers}}, {} {{status}}".format(PERMALINK)
12
13@@ -27,7 +27,7 @@
14 return
15
16 request_list = data.get('requests', [])
17- request_dict = {req['request_id']: req for req in request_list}
18+ request_dict = {req['id']: req for req in request_list}
19 for rid, req in request_dict.items():
20 req['status'] = req['status'].strip('. ')
21 old_req = self.requests.get(rid)

Subscribers

People subscribed via source and target branches