Merge lp:~sylvain-pineau/checkbox/gui_export_xlsx_with_all_options into lp:checkbox

Proposed by Sylvain Pineau
Status: Merged
Approved by: Zygmunt Krynicki
Approved revision: 3017
Merged at revision: 3017
Proposed branch: lp:~sylvain-pineau/checkbox/gui_export_xlsx_with_all_options
Merge into: lp:checkbox
Diff against target: 12 lines (+1/-1)
1 file modified
checkbox-gui/checkbox-gui/qml/SubmissionDialog.qml (+1/-1)
To merge this branch: bzr merge lp:~sylvain-pineau/checkbox/gui_export_xlsx_with_all_options
Reviewer Review Type Date Requested Status
Zygmunt Krynicki (community) Approve
Review via email: mp+220453@code.launchpad.net

Description of the change

Enable all the options supported by the xlsx exporter in checkbox-gui to get the same kind of report that checkbox-certification/oem provided.

To post a comment you must log in.
Revision history for this message
Zygmunt Krynicki (zyga) wrote :

+1

On Wed, May 21, 2014 at 3:25 PM, Sylvain Pineau <
<email address hidden>> wrote:

> Sylvain Pineau has proposed merging
> lp:~sylvain-pineau/checkbox/gui_export_xlsx_with_all_options into
> lp:checkbox.
>
> Requested reviews:
> Checkbox Developers (checkbox-dev)
>
> For more details, see:
>
> https://code.launchpad.net/~sylvain-pineau/checkbox/gui_export_xlsx_with_all_options/+merge/220453
>
> Enable all the options supported by the xlsx exporter in checkbox-gui to
> get the same kind of report that checkbox-certification/oem provided.
> --
>
> https://code.launchpad.net/~sylvain-pineau/checkbox/gui_export_xlsx_with_all_options/+merge/220453
> Your team Checkbox Developers is requested to review the proposed merge of
> lp:~sylvain-pineau/checkbox/gui_export_xlsx_with_all_options into
> lp:checkbox.
>
> === modified file 'checkbox-gui/checkbox-gui/qml/SubmissionDialog.qml'
> --- checkbox-gui/checkbox-gui/qml/SubmissionDialog.qml 2014-05-16
> 10:10:18 +0000
> +++ checkbox-gui/checkbox-gui/qml/SubmissionDialog.qml 2014-05-21
> 13:24:21 +0000
> @@ -147,7 +147,7 @@
> else if (reportTypeSelect.selectedIndex == 1) {
> var path = guiEngine.GetSaveFileName('submission.xlsx',
> i18n.tr("XLSX files (*.xlsx)"))
> - success = guiEngine.GuiExportSessionToFileAsXLSX(path,
> []);
> + success = guiEngine.GuiExportSessionToFileAsXLSX(path,
> ["with-sys-info", "with-summary", "with-job-description",
> "with-text-attachments"]);
> }
>
> if (success) {
>
>
>

Revision history for this message
Zygmunt Krynicki (zyga) wrote :

+1

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'checkbox-gui/checkbox-gui/qml/SubmissionDialog.qml'
2--- checkbox-gui/checkbox-gui/qml/SubmissionDialog.qml 2014-05-16 10:10:18 +0000
3+++ checkbox-gui/checkbox-gui/qml/SubmissionDialog.qml 2014-05-21 13:27:20 +0000
4@@ -147,7 +147,7 @@
5 else if (reportTypeSelect.selectedIndex == 1) {
6 var path = guiEngine.GetSaveFileName('submission.xlsx',
7 i18n.tr("XLSX files (*.xlsx)"))
8- success = guiEngine.GuiExportSessionToFileAsXLSX(path, []);
9+ success = guiEngine.GuiExportSessionToFileAsXLSX(path, ["with-sys-info", "with-summary", "with-job-description", "with-text-attachments"]);
10 }
11
12 if (success) {

Subscribers

People subscribed via source and target branches