Skip to content

Commit

Permalink
Merge pull request vaibhavpandeyvpz#5 from felixble/bugfix/vaibhavpan…
Browse files Browse the repository at this point in the history
…deyvpz#3-delete-multiple-reports

Delete multiple reports at once - fixes vaibhavpandeyvpz#3
  • Loading branch information
vaibhavpandeyvpz authored Jan 19, 2017
2 parents e7514c6 + 51fcffb commit 6379176
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 6379176

Please sign in to comment.