Merge pull request #5093 from kateryna-tarelkina-dd/confirm-named-server-deletion

Add confirmation dialog for named server deletion
This commit is contained in:
Min RK
2025-08-06 21:33:49 -07:00
committed by GitHub

View File

@@ -71,15 +71,21 @@ require(["jquery", "moment", "jhapi"], function ($, moment, JHAPI) {
var row = getRow($(this)); var row = getRow($(this));
var serverName = row.data("server-name"); var serverName = row.data("server-name");
// before request if (
disableRow(row); window.confirm(
`Are you sure you want to delete your server "${serverName}"?`,
)
) {
// before request
disableRow(row);
// request // request
api.delete_named_server(user, serverName, { api.delete_named_server(user, serverName, {
success: function () { success: function () {
row.remove(); row.remove();
}, },
}); });
}
} }
// initial state: hook up click events // initial state: hook up click events