Merge lp://qastaging/~bladernr/hwcert-tools/improve-c3_results_notify-reporting-tool into lp://qastaging/~hardware-certification/hwcert-tools/reporting-tools
Proposed by
Jeff Lane
| Status: | Merged |
|---|---|
| Approved by: | Daniel Manrique |
| Approved revision: | 155 |
| Merged at revision: | 155 |
| Proposed branch: | lp://qastaging/~bladernr/hwcert-tools/improve-c3_results_notify-reporting-tool |
| Merge into: | lp://qastaging/~hardware-certification/hwcert-tools/reporting-tools |
| Diff against target: |
279 lines (+161/-40) 1 file modified
certification_reports/c3_results_notify.py (+161/-40) |
| To merge this branch: | bzr merge lp://qastaging/~bladernr/hwcert-tools/improve-c3_results_notify-reporting-tool |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Daniel Manrique (community) | Approve | ||
|
Review via email:
|
|||
Description of the change
Overhaul of the c3_results_
It now has better error handling and some general housekeeping.
It also now has date filtering and a couple convenience options for reporting submissions or certificates over the past week or past month.
Also, it will generate a report of completed certificates as well.
The general certificates result now shows both Scheduled and In Progress certs to avoid forgetting about "Scheduled" ones that need reminders from time to time.
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
Looks OK, I made a couple of minor comments but I'll approve this.