Merge pull request #36429 from KerollesFathy/add-title-to-get-print-settings

refactor: add title param to get_print_settings
This commit is contained in:
Ejaaz Khan 2026-01-29 11:38:53 +05:30 committed by GitHub
commit bbf25d41fd
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 5 additions and 3 deletions

View file

@ -3,7 +3,8 @@ frappe.ui.get_print_settings = function (
callback,
letter_head,
pick_columns,
has_filters = false
has_filters = false,
title = null
) {
var print_settings = locals[":Print Settings"]["Print Settings"];
@ -115,7 +116,7 @@ frappe.ui.get_print_settings = function (
settings.print_format = null;
}
},
__("Print Settings")
title ? __(title) : __("Print Settings")
);
};

View file

@ -1915,7 +1915,8 @@ frappe.views.QueryReport = class QueryReport extends frappe.views.BaseList {
(print_settings) => this.pdf_report(print_settings),
this.report_doc.letter_head,
this.get_visible_columns(),
true
true,
"PDF Settings"
);
this.add_portrait_warning(dialog);
},