PlainBox (Toolkit)

Merge ~sylvain-pineau/plainbox:strip_ns_json into plainbox:master

  1. Git
  2. lp:~sylvain-pineau/plainbox
  3. strip_ns_json
  4. Merge into master
Proposed by Sylvain Pineau
Status: Merged
Approved by: Sylvain Pineau
Approved revision: 0bd36bc3a42b3efe058b7af7289f9e84afb9294b
Merged at revision: 5bf9ed724600f5a689a8fe33e89cd9ce02b90fa4
Proposed branch: ~sylvain-pineau/plainbox:strip_ns_json
Merge into: plainbox:master
Diff against target: 13 lines (+1/-1)
1 file modified
plainbox/impl/providers/exporters/data/checkbox.json (+1/-1)
Reviewer Review Type Date Requested Status
Sylvain Pineau (community) Approve
Review via email: mp+321970@code.launchpad.net

Description of the change

small patch to strip job ns from json reports (as we do for xml)

To post a comment you must log in.
Revision history for this message
Sylvain Pineau (sylvain-pineau) wrote :

self-approved

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/plainbox/impl/providers/exporters/data/checkbox.json b/plainbox/impl/providers/exporters/data/checkbox.json
2index 4925fe5..3a77b84 100644
3--- a/plainbox/impl/providers/exporters/data/checkbox.json
4+++ b/plainbox/impl/providers/exporters/data/checkbox.json
5@@ -38,7 +38,7 @@
6 "results": [
7 {%- for job_id, job_state in job_state_map|dictsort if job_state.result.outcome != None and job_state.job.plugin not in ("resource", "local", "attachment") %}
8 {
9- "id": "{{ job_state.job.id }}",
10+ "id": "{{ job_id|strip_ns }}",
11 "name": "{{ job_state.job.tr_summary() }}",
12 "certification_status": "{{ job_state.effective_certification_status }}",
13 "category": "{{ job_state.effective_category_id }}",

Subscribers

People subscribed via source and target branches