Skip to content

Commit

Permalink
Merge branch 'issue-1712-add-status-information' of github.com:Aiven-…
Browse files Browse the repository at this point in the history
…Open/klaw into issue-1712-add-status-information
  • Loading branch information
aindriu-aiven committed Sep 26, 2023
2 parents 65637a7 + b084f90 commit 1455587
Show file tree
Hide file tree
Showing 40 changed files with 188 additions and 125 deletions.
28 changes: 26 additions & 2 deletions .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
@@ -1,4 +1,28 @@
About this change - What it does
# Linked issue

Resolves: #xxxxx
Why this way

# What kind of change does this PR introduce?

- [ ] Bug fix
- [ ] New feature
- [ ] Refactor
- [ ] Docs update
- [ ] CI update

# What is the current behavior?

_Describe the state of the application before this PR. Illustrations appreciated (videos, gifs, screenshots)._

# What is the new behavior?

_Describe the state of the application after this PR. Illustrations appreciated (videos, gifs, screenshots)._

# Other information:

_Additional changes, explanations of the approach taken, unresolved issues, necessary follow ups, etc._

# Please check if the PR fulfills these requirements

- [ ] The commit message follows [our guidelines](https://github.com/Aiven-Open/klaw/blob/main/CONTRIBUTING.md#guideline-commit-messages)
- [ ] Tests for the changes have been added (for bug fixes / features)
5 changes: 3 additions & 2 deletions core/src/main/resources/static/js/analytics.js
Original file line number Diff line number Diff line change
Expand Up @@ -134,7 +134,7 @@ app.controller("showAnalyticsCtrl", function($scope, $http, $location, $window)
closeOnConfirm: true,
closeOnCancel: true
}).then(function(isConfirm) {
if (isConfirm.dismiss !== "cancel") {
if (isConfirm.value) {
$http({
method: "POST",
url: "user/updateTeam",
Expand All @@ -158,6 +158,7 @@ app.controller("showAnalyticsCtrl", function($scope, $http, $location, $window)
}
);
} else {
$scope.checkSwitchTeams($scope.dashboardDetails.canSwitchTeams, $scope.dashboardDetails.teamId, $scope.userlogged);
return;
}
});
Expand Down Expand Up @@ -197,7 +198,7 @@ app.controller("showAnalyticsCtrl", function($scope, $http, $location, $window)
closeOnConfirm: true,
closeOnCancel: true
}).then(function(isConfirm){
if (isConfirm.dismiss != "cancel") {
if (isConfirm.value) {
$window.location.href = $window.location.origin + $scope.dashboardDetails.contextPath + "/"+redirectPage;
} else {
return;
Expand Down
15 changes: 8 additions & 7 deletions core/src/main/resources/static/js/browseAcls.js
Original file line number Diff line number Diff line change
Expand Up @@ -144,7 +144,7 @@ app.controller("browseAclsCtrl", function($scope, $http, $location, $window) {
closeOnConfirm: true,
closeOnCancel: true
}).then(function(isConfirm) {
if (isConfirm.dismiss !== "cancel") {
if (isConfirm.value) {
$http({
method: "POST",
url: "user/updateTeam",
Expand All @@ -168,6 +168,7 @@ app.controller("browseAclsCtrl", function($scope, $http, $location, $window) {
}
);
} else {
$scope.checkSwitchTeams($scope.dashboardDetails.canSwitchTeams, $scope.dashboardDetails.teamId, $scope.userlogged);
return;
}
});
Expand Down Expand Up @@ -207,7 +208,7 @@ app.controller("browseAclsCtrl", function($scope, $http, $location, $window) {
closeOnConfirm: true,
closeOnCancel: true
}).then(function(isConfirm){
if (isConfirm.dismiss != "cancel") {
if (isConfirm.value) {
$window.location.href = $window.location.origin + $scope.dashboardDetails.contextPath + "/"+redirectPage;
} else {
return;
Expand Down Expand Up @@ -260,7 +261,7 @@ app.controller("browseAclsCtrl", function($scope, $http, $location, $window) {
closeOnConfirm: true,
closeOnCancel: true
}).then(function(isConfirm){
if (isConfirm.dismiss !== "cancel") {
if (isConfirm.value) {
swal({
title: "Delete schema ?",
text: "Delete associated schema (all versions) of this topic, if it exists ?",
Expand All @@ -272,7 +273,7 @@ app.controller("browseAclsCtrl", function($scope, $http, $location, $window) {
closeOnConfirm: true,
closeOnCancel: true
}).then(function(isConfirm){
if (isConfirm.dismiss !== "cancel") {
if (isConfirm.value) {
$scope.createTopicDeleteRequestHttpCall(env, true); // delete schema
} else {
$scope.createTopicDeleteRequestHttpCall(env, false); // don't delete schema
Expand Down Expand Up @@ -330,7 +331,7 @@ app.controller("browseAclsCtrl", function($scope, $http, $location, $window) {
closeOnConfirm: true,
closeOnCancel: true
}).then(function(isConfirm){
if (isConfirm.dismiss !== "cancel") {
if (isConfirm.value) {
$http({
method: "POST",
url: "createClaimTopicRequest",
Expand Down Expand Up @@ -390,7 +391,7 @@ app.controller("browseAclsCtrl", function($scope, $http, $location, $window) {
closeOnConfirm: true,
closeOnCancel: true
}).then(function(isConfirm){
if (isConfirm.dismiss !== "cancel") {
if (isConfirm.value) {
$http({
method: "POST",
url: "createDeleteAclSubscriptionRequest",
Expand Down Expand Up @@ -543,7 +544,7 @@ app.controller("browseAclsCtrl", function($scope, $http, $location, $window) {
closeOnConfirm: true,
closeOnCancel: true
}).then(function(isConfirm){
if (isConfirm.dismiss !== "cancel") {
if (isConfirm.value) {
$http({
method: "POST",
url: "saveTopicDocumentation",
Expand Down
5 changes: 3 additions & 2 deletions core/src/main/resources/static/js/browseTopics.js
Original file line number Diff line number Diff line change
Expand Up @@ -107,7 +107,7 @@ app.controller("browseTopicsCtrl", function($scope, $http, $location, $window) {
closeOnConfirm: true,
closeOnCancel: true
}).then(function(isConfirm) {
if (isConfirm.dismiss !== "cancel") {
if (isConfirm.value) {
$http({
method: "POST",
url: "user/updateTeam",
Expand All @@ -131,6 +131,7 @@ app.controller("browseTopicsCtrl", function($scope, $http, $location, $window) {
}
);
} else {
$scope.checkSwitchTeams($scope.dashboardDetails.canSwitchTeams, $scope.dashboardDetails.teamId, $scope.userlogged);
return;
}
});
Expand Down Expand Up @@ -170,7 +171,7 @@ app.controller("browseTopicsCtrl", function($scope, $http, $location, $window) {
closeOnConfirm: true,
closeOnCancel: true
}).then(function(isConfirm){
if (isConfirm.dismiss != "cancel") {
if (isConfirm.value) {
$window.location.href = $window.location.origin + $scope.dashboardDetails.contextPath + "/"+redirectPage;
} else {
return;
Expand Down
13 changes: 7 additions & 6 deletions core/src/main/resources/static/js/connectorOverview.js
Original file line number Diff line number Diff line change
Expand Up @@ -134,7 +134,7 @@ app.controller("connectorOverviewCtrl", function($scope, $http, $location, $wind
closeOnConfirm: true,
closeOnCancel: true
}).then(function(isConfirm) {
if (isConfirm.dismiss !== "cancel") {
if (isConfirm.value) {
$http({
method: "POST",
url: "user/updateTeam",
Expand All @@ -158,6 +158,7 @@ app.controller("connectorOverviewCtrl", function($scope, $http, $location, $wind
}
);
} else {
$scope.checkSwitchTeams($scope.dashboardDetails.canSwitchTeams, $scope.dashboardDetails.teamId, $scope.userlogged);
return;
}
});
Expand Down Expand Up @@ -197,7 +198,7 @@ app.controller("connectorOverviewCtrl", function($scope, $http, $location, $wind
closeOnConfirm: true,
closeOnCancel: true
}).then(function(isConfirm){
if (isConfirm.dismiss != "cancel") {
if (isConfirm.value) {
$window.location.href = $window.location.origin + $scope.dashboardDetails.contextPath + "/"+redirectPage;
} else {
return;
Expand Down Expand Up @@ -245,7 +246,7 @@ app.controller("connectorOverviewCtrl", function($scope, $http, $location, $wind
closeOnConfirm: true,
closeOnCancel: true
}).then(function(isConfirm){
if (isConfirm.dismiss != "cancel") {
if (isConfirm.value) {
$http({
method: "POST",
url: "createConnectorDeleteRequest",
Expand Down Expand Up @@ -294,7 +295,7 @@ app.controller("connectorOverviewCtrl", function($scope, $http, $location, $wind
closeOnConfirm: true,
closeOnCancel: true
}).then(function(isConfirm){
if (isConfirm.dismiss != "cancel") {
if (isConfirm.value) {
$http({
method: "POST",
url: "createClaimConnectorRequest",
Expand Down Expand Up @@ -362,7 +363,7 @@ app.controller("connectorOverviewCtrl", function($scope, $http, $location, $wind
closeOnConfirm: true,
closeOnCancel: true
}).then(function(isConfirm){
if (isConfirm.dismiss !== "cancel") {
if (isConfirm.value) {
$http({
method: "POST",
url: "createConnector",
Expand Down Expand Up @@ -451,7 +452,7 @@ app.controller("connectorOverviewCtrl", function($scope, $http, $location, $wind
closeOnConfirm: true,
closeOnCancel: true
}).then(function(isConfirm){
if (isConfirm.dismiss !== "cancel") {
if (isConfirm.value) {
$http({
method: "POST",
url: "saveConnectorDocumentation",
Expand Down
5 changes: 3 additions & 2 deletions core/src/main/resources/static/js/docs.js
Original file line number Diff line number Diff line change
Expand Up @@ -173,7 +173,7 @@ app.controller("docsCtrl", function($scope, $http, $location, $window) {
closeOnConfirm: true,
closeOnCancel: true
}).then(function(isConfirm) {
if (isConfirm.dismiss !== "cancel") {
if (isConfirm.value) {
$http({
method: "POST",
url: "user/updateTeam",
Expand All @@ -197,6 +197,7 @@ app.controller("docsCtrl", function($scope, $http, $location, $window) {
}
);
} else {
$scope.checkSwitchTeams($scope.dashboardDetails.canSwitchTeams, $scope.dashboardDetails.teamId, $scope.userlogged);
return;
}
});
Expand Down Expand Up @@ -236,7 +237,7 @@ app.controller("docsCtrl", function($scope, $http, $location, $window) {
closeOnConfirm: true,
closeOnCancel: true
}).then(function(isConfirm){
if (isConfirm.dismiss != "cancel") {
if (isConfirm.value) {
$window.location.href = $window.location.origin + $scope.dashboardDetails.contextPath + "/"+redirectPage;
} else {
return;
Expand Down
5 changes: 3 additions & 2 deletions core/src/main/resources/static/js/editAclRequest.js
Original file line number Diff line number Diff line change
Expand Up @@ -120,7 +120,7 @@ app.controller("editAclRequestCtrl", function($scope, $http, $location, $window)
closeOnConfirm: true,
closeOnCancel: true
}).then(function(isConfirm) {
if (isConfirm.dismiss !== "cancel") {
if (isConfirm.value) {
$http({
method: "POST",
url: "user/updateTeam",
Expand All @@ -144,6 +144,7 @@ app.controller("editAclRequestCtrl", function($scope, $http, $location, $window)
}
);
} else {
$scope.checkSwitchTeams($scope.dashboardDetails.canSwitchTeams, $scope.dashboardDetails.teamId, $scope.userlogged);
return;
}
});
Expand Down Expand Up @@ -183,7 +184,7 @@ app.controller("editAclRequestCtrl", function($scope, $http, $location, $window)
closeOnConfirm: true,
closeOnCancel: true
}).then(function(isConfirm){
if (isConfirm.dismiss !== "cancel") {
if (isConfirm.value) {
$window.location.href = $window.location.origin + $scope.dashboardDetails.contextPath + "/"+redirectPage;
} else {
return;
Expand Down
7 changes: 4 additions & 3 deletions core/src/main/resources/static/js/editTopicRequest.js
Original file line number Diff line number Diff line change
Expand Up @@ -290,7 +290,7 @@ app.controller("editTopicRequestCtrl", function($scope, $http, $location, $windo
} else {
serviceInput['remarks'] += " Warning To decrease partitions of a topic the topic has to be deleted";
}
if (isConfirm.dismiss !== "cancel") {
if (isConfirm.value) {
$scope.httpCreateUpdateTopicReq(serviceInput);
}
});
Expand Down Expand Up @@ -532,7 +532,7 @@ app.controller("editTopicRequestCtrl", function($scope, $http, $location, $windo
closeOnConfirm: true,
closeOnCancel: true
}).then(function(isConfirm) {
if (isConfirm.dismiss !== "cancel") {
if (isConfirm.value) {
$http({
method: "POST",
url: "user/updateTeam",
Expand All @@ -556,6 +556,7 @@ app.controller("editTopicRequestCtrl", function($scope, $http, $location, $windo
}
);
} else {
$scope.checkSwitchTeams($scope.dashboardDetails.canSwitchTeams, $scope.dashboardDetails.teamId, $scope.userlogged);
return;
}
});
Expand Down Expand Up @@ -595,7 +596,7 @@ app.controller("editTopicRequestCtrl", function($scope, $http, $location, $windo
closeOnConfirm: true,
closeOnCancel: true
}).then(function(isConfirm){
if (isConfirm.dismiss != "cancel") {
if (isConfirm.value) {
$window.location.href = $window.location.origin + $scope.dashboardDetails.contextPath + "/"+redirectPage;
} else {
return;
Expand Down
11 changes: 6 additions & 5 deletions core/src/main/resources/static/js/envs.js
Original file line number Diff line number Diff line change
Expand Up @@ -354,7 +354,7 @@ app.controller("envsCtrl", function($scope, $http, $location, $window) {
closeOnConfirm: true,
closeOnCancel: true
}).then(function(isConfirm){
if (isConfirm.dismiss !== "cancel") {
if (isConfirm.value) {
$http({
method: "POST",
url: "deleteTenant",
Expand Down Expand Up @@ -601,7 +601,7 @@ app.controller("envsCtrl", function($scope, $http, $location, $window) {
closeOnConfirm: true,
closeOnCancel: true
}).then(function(isConfirm){
if (isConfirm.dismiss != "cancel") {
if (isConfirm.value) {
$http({
method: "POST",
url: "deleteCluster",
Expand Down Expand Up @@ -645,7 +645,7 @@ app.controller("envsCtrl", function($scope, $http, $location, $window) {
closeOnConfirm: true,
closeOnCancel: true
}).then(function(isConfirm){
if (isConfirm.dismiss != "cancel") {
if (isConfirm.value) {
$http({
method: "POST",
url: "deleteEnvironmentRequest",
Expand Down Expand Up @@ -1179,7 +1179,7 @@ app.controller("envsCtrl", function($scope, $http, $location, $window) {
closeOnConfirm: true,
closeOnCancel: true
}).then(function(isConfirm) {
if (isConfirm.dismiss !== "cancel") {
if (isConfirm.value) {
$http({
method: "POST",
url: "user/updateTeam",
Expand All @@ -1203,6 +1203,7 @@ app.controller("envsCtrl", function($scope, $http, $location, $window) {
}
);
} else {
$scope.checkSwitchTeams($scope.dashboardDetails.canSwitchTeams, $scope.dashboardDetails.teamId, $scope.userlogged);
return;
}
});
Expand Down Expand Up @@ -1242,7 +1243,7 @@ app.controller("envsCtrl", function($scope, $http, $location, $window) {
closeOnConfirm: true,
closeOnCancel: true
}).then(function(isConfirm){
if (isConfirm.dismiss != "cancel") {
if (isConfirm.value) {
$window.location.href = $window.location.origin + $scope.dashboardDetails.contextPath + "/"+redirectPage;
} else {
return;
Expand Down
9 changes: 5 additions & 4 deletions core/src/main/resources/static/js/execAcls.js
Original file line number Diff line number Diff line change
Expand Up @@ -97,7 +97,7 @@ app.controller("execAclsCtrl", function($scope, $http, $location, $window) {
closeOnConfirm: true,
closeOnCancel: true
}).then(function(isConfirm) {
if (isConfirm.dismiss !== "cancel") {
if (isConfirm.value) {
$http({
method: "POST",
url: "user/updateTeam",
Expand All @@ -121,6 +121,7 @@ app.controller("execAclsCtrl", function($scope, $http, $location, $window) {
}
);
} else {
$scope.checkSwitchTeams($scope.dashboardDetails.canSwitchTeams, $scope.dashboardDetails.teamId, $scope.userlogged);
return;
}
});
Expand Down Expand Up @@ -160,7 +161,7 @@ app.controller("execAclsCtrl", function($scope, $http, $location, $window) {
closeOnConfirm: true,
closeOnCancel: true
}).then(function(isConfirm){
if (isConfirm.dismiss != "cancel") {
if (isConfirm.value) {
$window.location.href = $window.location.origin + $scope.dashboardDetails.contextPath + "/"+redirectPage;
} else {
return;
Expand Down Expand Up @@ -264,7 +265,7 @@ app.controller("execAclsCtrl", function($scope, $http, $location, $window) {
closeOnConfirm: true,
closeOnCancel: true
}).then(function(isConfirm){
if (isConfirm.dismiss != "cancel") {
if (isConfirm.value) {
$http({
method: "POST",
url: "execAclRequest",
Expand Down Expand Up @@ -345,7 +346,7 @@ app.controller("execAclsCtrl", function($scope, $http, $location, $window) {
closeOnConfirm: true,
closeOnCancel: true
}).then(function(isConfirm){
if (isConfirm.dismiss != "cancel") {
if (isConfirm.value) {
$http({
method: "POST",
url: "execAclRequestDecline",
Expand Down
Loading

0 comments on commit 1455587

Please sign in to comment.