Skip to content

Commit

Permalink
Delete multiple reports at once - fixes vaibhavpandeyvpz#3
Browse files Browse the repository at this point in the history
It was not possible to delete multiple reports or applications because there was an invalid check for the length of the selected rows.
  • Loading branch information
felixble committed Jan 9, 2017
1 parent 2294367 commit 51fcffb
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions assets/js/app.js
Original file line number Diff line number Diff line change
Expand Up @@ -132,7 +132,7 @@
});
switch (action) {
case 'delete:application':
if (selected.length == 1) {
if (selected.length >= 1) {
bootbox.confirm('Deleting selected applications will also delete associated crash reports. Continue?', function (result) {
if (result) {
$.post($el.data('url'), {
Expand All @@ -150,7 +150,7 @@
}
break;
case 'delete:report':
if (selected.length == 1) {
if (selected.length >= 1) {
bootbox.confirm('Are you sure you wish to delete selected crash reports? This cannot be undone.', function (result) {
if (result) {
$.post($el.data('url'), {
Expand Down

0 comments on commit 51fcffb

Please sign in to comment.