vassili/seitime-frappe
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 4

fix: show remove button conditionally

Browse source
This commit is contained in:
Rohan Bansal 2020-12-29 16:11:03 +05:30
parent ec9329b50c
commit 921fdffa72
2 changed files with 9 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
frappe/core/page/background_jobs/background_jobs.js
View file

@ -40,6 +40,13 @@ frappe.pages.background_jobs.refresh_jobs = function() {
page.body.find('.list-jobs').remove();
$(frappe.render_template('background_jobs', { jobs: r.message || [] })).appendTo(page.content);
let $remove_failed_btn = page.body.find('.remove-failed');
if (r.message && r.message.length > 0) {
$remove_failed_btn.show();
} else {
$remove_failed_btn.hide();
}
if (frappe.get_route()[0] === 'background_jobs') {
frappe.background_jobs_timeout = setTimeout(frappe.pages.background_jobs.refresh_jobs, 2000);
}

6
frappe/core/page/background_jobs/background_jobs_outer.html
View file

@ -6,10 +6,8 @@
</label>
</div>
<div>
<button type="button" class="btn btn-primary remove-failed"> {{ __("Remove failed jobs") }}</button>
<button type="button" class="btn btn-primary remove-failed" hidden> {{ __("Remove failed jobs") }}</button>
</div>
</p>
<div class="table-area">
</div>
<div class="table-area"></div>
</div>