lists of jobs should be based on job.summary, not .partial_id
Bug #1314516 reported by
Zygmunt Krynicki
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Checkbox GUI (Legacy) |
Fix Released
|
Medium
|
Brendan Donegan | ||
| Next Generation Checkbox (CLI) |
Fix Released
|
Medium
|
Brendan Donegan | ||
| PlainBox (Toolkit) |
Fix Released
|
Medium
|
Brendan Donegan | ||
Bug Description
We want to offer fully translatable and human-readable testing experience. For that we should switch to job.summary in all the places that display job "name".
Currently we display a mix of job.description (for resource jobs, sometimes), job.summary, job.id and job.partial_id. Local jobs that "cat" other jobs for "categories" are also handled specially. We should have no corner cases.
Related branches
lp:~brendan-donegan/checkbox/bug1314516_checkbox-gui
- Zygmunt Krynicki (community): Approve
- Brendan Donegan (community): Needs Resubmitting
-
Diff: 87 lines (+14/-15)2 files modifiedcheckbox-gui/checkbox-gui/testitemmodel.cpp (+12/-13)
plainbox/plainbox/impl/job.py (+2/-2)
lp:~brendan-donegan/checkbox/bug1314516_cli
- Zygmunt Krynicki (community): Approve
-
Diff: 28 lines (+5/-2)2 files modifiedcheckbox-ng/checkbox_ng/commands/cli.py (+1/-1)
plainbox/plainbox/impl/job.py (+4/-1)
lp:~brendan-donegan/checkbox/bug1314516_plainbox
- Zygmunt Krynicki (community): Approve
- Brendan Donegan (community): Needs Resubmitting
-
Diff: 12 lines (+1/-1)1 file modifiedplainbox/plainbox/impl/commands/run.py (+1/-1)
| Changed in checkbox-ng: | |
| status: | New → Confirmed |
| Changed in plainbox: | |
| status: | New → Confirmed |
| description: | updated |
| tags: | added: ce-qa-concern ce-qa-providers |
Revision history for this message
| Brendan Donegan (brendan-donegan) wrote | #1 |
| Changed in checkbox-gui: | |
| status: | Confirmed → In Progress |
| importance: | Undecided → Medium |
| assignee: | nobody → Brendan Donegan (brendan-donegan) |
Revision history for this message
| Brendan Donegan (brendan-donegan) wrote | #2 |
| Changed in checkbox-ng: | |
| importance: | Undecided → Medium |
| Changed in plainbox: | |
| importance: | Undecided → Medium |
Revision history for this message
| Brendan Donegan (brendan-donegan) wrote | #3 |
| Changed in checkbox-ng: | |
| status: | Confirmed → In Progress |
| assignee: | nobody → Brendan Donegan (brendan-donegan) |
| Changed in checkbox-gui: | |
| status: | In Progress → Fix Committed |
| Changed in plainbox: | |
| status: | Confirmed → In Progress |
| Changed in checkbox-ng: | |
| status: | In Progress → Fix Committed |
| Changed in plainbox: | |
| assignee: | nobody → Brendan Donegan (brendan-donegan) |
| Changed in plainbox: | |
| status: | In Progress → Fix Committed |
| Changed in checkbox-ng: | |
| milestone: | none → 0.4 |
| Changed in plainbox: | |
| milestone: | none → 0.6 |
| Changed in checkbox-gui: | |
| milestone: | none → 0.18 |
| Changed in checkbox-ng: | |
| status: | Fix Committed → Fix Released |
| Changed in checkbox-gui: | |
| status: | Fix Committed → Fix Released |
| Changed in plainbox: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
I can take care of this in checkbox-gui, to start off with