New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
DRILL-6243: Added alert box to confirm shutdown of drillbit after clicking shutdown button #1169
Conversation
@sohami Please review |
…cking shutdown button
complete: function(data) { | ||
alert(data.responseJSON["response"]); | ||
button.prop('disabled',true).css('opacity',0.5); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please fix indentation here and below. Also add the error:
callback for Ajax request.
Like alert with received error ?
button.prop('disabled',true).css('opacity',0.5); | ||
} | ||
}); | ||
if (confirm("Click ok to shutdown")) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Message should be more of like "Are you sure you want to shutdown Drillbit running on + location.host + node ?"
+1 LGTM. Thanks for making the changes. |
…cking shutdown button closes apache#1169
…cking shutdown button