feat: add delete of release candidates

client/src/app/components/environments/environment-list/environment-list-view.component.html

@@ -131,5 +131,3 @@
     </p-accordion-panel>
   }
 </p-accordion>
-
-<p-confirmDialog></p-confirmDialog>

client/src/app/components/environments/environment-list/environment-list-view.component.ts

@@ -17,7 +17,6 @@ import { injectMutation, injectQuery, QueryClient } from '@tanstack/angular-quer
 import { IconsModule } from 'icons.module';
 import { ConfirmationService } from 'primeng/api';
 import { ButtonModule } from 'primeng/button';
-import { ConfirmDialogModule } from 'primeng/confirmdialog';
 import { InputTextModule } from 'primeng/inputtext';
 import { TagModule } from 'primeng/tag';
 import { EnvironmentDeploymentInfoComponent } from '../deployment-info/environment-deployment-info.component';
@@ -49,7 +48,6 @@ import { EnvironmentStatusTagComponent } from '../environment-status-tag/environ
     EnvironmentStatusInfoComponent,
     LockTimeComponent,
     AvatarModule,
-    ConfirmDialogModule,
     CommonModule,
     TimeAgoPipe,
     UserAvatarComponent,

client/src/app/core/modules/openapi/@tanstack/angular-query-experimental.gen.ts

@@ -29,6 +29,8 @@ import type {
   GetGroupsWithWorkflowsData,
   GetAllRepositoriesData,
   GetRepositoryByIdData,
+  DeleteReleaseCandidateByNameData,
+  DeleteReleaseCandidateByNameResponse,
   GetReleaseCandidateByNameData,
   GetCommitsSinceLastReleaseCandidateData,
   GetAllPullRequestsData,
@@ -74,6 +76,7 @@ import {
   getGroupsWithWorkflows,
   getAllRepositories,
   getRepositoryById,
+  deleteReleaseCandidateByName,
   getReleaseCandidateByName,
   getCommitsSinceLastReleaseCandidate,
   getAllPullRequests,
@@ -560,6 +563,20 @@ export const getRepositoryByIdOptions = (options: Options<GetRepositoryByIdData>
   });
 };
 
+export const deleteReleaseCandidateByNameMutation = (options?: Partial<Options<DeleteReleaseCandidateByNameData>>) => {
+  const mutationOptions: MutationOptions<DeleteReleaseCandidateByNameResponse, DefaultError, Options<DeleteReleaseCandidateByNameData>> = {
+    mutationFn: async localOptions => {
+      const { data } = await deleteReleaseCandidateByName({
+        ...options,
+        ...localOptions,
+        throwOnError: true,
+      });
+      return data;
+    },
+  };
+  return mutationOptions;
+};
+
 export const getReleaseCandidateByNameQueryKey = (options: Options<GetReleaseCandidateByNameData>) => [createQueryKey('getReleaseCandidateByName', options)];
 
 export const getReleaseCandidateByNameOptions = (options: Options<GetReleaseCandidateByNameData>) => {

client/src/app/core/modules/openapi/sdk.gen.ts

@@ -42,6 +42,8 @@ import type {
   GetAllRepositoriesResponse,
   GetRepositoryByIdData,
   GetRepositoryByIdResponse,
+  DeleteReleaseCandidateByNameData,
+  DeleteReleaseCandidateByNameResponse,
   GetReleaseCandidateByNameData,
   GetReleaseCandidateByNameResponse,
   GetCommitsSinceLastReleaseCandidateData,
@@ -278,6 +280,13 @@ export const getRepositoryById = <ThrowOnError extends boolean = false>(options:
   });
 };
 
+export const deleteReleaseCandidateByName = <ThrowOnError extends boolean = false>(options: Options<DeleteReleaseCandidateByNameData, ThrowOnError>) => {
+  return (options?.client ?? client).delete<DeleteReleaseCandidateByNameResponse, unknown, ThrowOnError>({
+    ...options,
+    url: '/api/release-candidate/{name}',
+  });
+};
+
 export const getReleaseCandidateByName = <ThrowOnError extends boolean = false>(options: Options<GetReleaseCandidateByNameData, ThrowOnError>) => {
   return (options?.client ?? client).get<GetReleaseCandidateByNameResponse, unknown, ThrowOnError>({
     ...options,

client/src/app/core/modules/openapi/types.gen.ts

@@ -685,6 +685,24 @@ export type GetRepositoryByIdResponses = {
 
 export type GetRepositoryByIdResponse = GetRepositoryByIdResponses[keyof GetRepositoryByIdResponses];
 
+export type DeleteReleaseCandidateByNameData = {
+  body?: never;
+  path: {
+    name: string;
+  };
+  query?: never;
+  url: '/api/release-candidate/{name}';
+};
+
+export type DeleteReleaseCandidateByNameResponses = {
+  /**
+   * OK
+   */
+  200: ReleaseCandidateInfoDto;
+};
+
+export type DeleteReleaseCandidateByNameResponse = DeleteReleaseCandidateByNameResponses[keyof DeleteReleaseCandidateByNameResponses];
+
 export type GetReleaseCandidateByNameData = {
   body?: never;
   path: {

client/src/app/pages/main-layout/main-layout.component.html

@@ -37,5 +37,6 @@
       </div>
     </div>
     <p-toast></p-toast>
+    <p-confirmdialog />
   </div>
 </main>

client/src/app/pages/main-layout/main-layout.component.ts

@@ -15,6 +15,7 @@ import { ToastModule } from 'primeng/toast';
 import { TooltipModule } from 'primeng/tooltip';
 import { HeliosIconComponent } from '../../components/helios-icon/helios-icon.component';
 import { UserLockInfoComponent } from '@app/components/user-lock-info/user-lock-info.component';
+import { ConfirmDialogModule } from 'primeng/confirmdialog';
 
 @Component({
   selector: 'app-main-layout',
@@ -31,6 +32,7 @@ import { UserLockInfoComponent } from '@app/components/user-lock-info/user-lock-
     HeliosIconComponent,
     DividerModule,
     AvatarModule,
+    ConfirmDialogModule,
     CardModule,
     ProfileNavSectionComponent,
     UserLockInfoComponent,

client/src/app/pages/project-settings/project-settings.component.html

@@ -145,6 +145,4 @@ <h3 class="text-xl mb-2">Workflows</h3>
       </div>
     </div>
   </p-dialog>
-
-  <p-confirmDialog></p-confirmDialog>
 </div>

client/src/app/pages/project-settings/project-settings.component.ts

@@ -10,7 +10,6 @@ import { InputTextModule } from 'primeng/inputtext';
 import { IconsModule } from 'icons.module';
 import { DragDropModule } from 'primeng/dragdrop';
 import { ConfirmationService } from 'primeng/api';
-import { ConfirmDialogModule } from 'primeng/confirmdialog';
 import { DividerModule } from 'primeng/divider';
 
 import {
@@ -33,7 +32,6 @@ import { LockingThresholdsComponent } from '@app/components/locking-thresholds/l
 
 @Component({
   selector: 'app-project-settings',
-  standalone: true,
   imports: [
     FormsModule,
     TableModule,
@@ -47,7 +45,6 @@ import { LockingThresholdsComponent } from '@app/components/locking-thresholds/l
     InputTextModule,
     IconsModule,
     DragDropModule,
-    ConfirmDialogModule,
     DividerModule,
   ],
   templateUrl: './project-settings.component.html',

client/src/app/pages/release-candidate-details/release-candidate-details.component.html

@@ -14,6 +14,11 @@ <h2 class="text-2xl">
             <i-tabler name="git-commit" class="!size-4" />
             {{ releaseCandidate.commit.sha | slice: 0 : 7 }}
           </p-tag>
+          @if (permissionService.isAtLeastMaintainer()) {
+            <p-button text (onClick)="deleteReleaseCandidate(releaseCandidate)">
+              <i-tabler name="trash" class="!size-5 text-red-500 hover:text-red-700" />
+            </p-button>
+          }
         </h2>
         <div class="flex gap-1 items-center text-sm">
           <div>created by</div>

client/src/app/pages/release-candidate-details/release-candidate-details.component.ts

@@ -1,5 +1,10 @@
 import { Component, inject, input } from '@angular/core';
-import { evaluateMutation, getReleaseCandidateByNameOptions, getReleaseCandidateByNameQueryKey } from '@app/core/modules/openapi/@tanstack/angular-query-experimental.gen';
+import {
+  deleteReleaseCandidateByNameMutation,
+  evaluateMutation,
+  getReleaseCandidateByNameOptions,
+  getReleaseCandidateByNameQueryKey,
+} from '@app/core/modules/openapi/@tanstack/angular-query-experimental.gen';
 import { injectMutation, injectQuery, QueryClient } from '@tanstack/angular-query-experimental';
 import { ButtonModule } from 'primeng/button';
 import { ButtonGroupModule } from 'primeng/buttongroup';
@@ -11,9 +16,11 @@ import { TimeAgoPipe } from '@app/pipes/time-ago.pipe';
 import { TooltipModule } from 'primeng/tooltip';
 import { SlicePipe } from '@angular/common';
 import { TagModule } from 'primeng/tag';
-import { MessageService } from 'primeng/api';
+import { ConfirmationService, MessageService } from 'primeng/api';
 import { KeycloakService } from '@app/core/services/keycloak/keycloak.service';
 import { PermissionService } from '@app/core/services/permission.service';
+import { ReleaseCandidateDetailsDto } from '@app/core/modules/openapi';
+import { ActivatedRoute, Router } from '@angular/router';
 
 @Component({
   selector: 'app-release-candidate-details',
@@ -24,7 +31,10 @@ export class ReleaseCandidateDetailsComponent {
   private messageService = inject(MessageService);
   private keycloakService = inject(KeycloakService);
   permissionService = inject(PermissionService);
+  private confirmationService = inject(ConfirmationService);
   private queryClient = inject(QueryClient);
+  private router = inject(Router);
+  private route = inject(ActivatedRoute);
 
   name = input.required<string>();
   releaseCandidateQuery = injectQuery(() => getReleaseCandidateByNameOptions({ path: { name: this.name() } }));
@@ -37,10 +47,28 @@ export class ReleaseCandidateDetailsComponent {
     },
   }));
 
+  deleteReleaseCandidateMutation = injectMutation(() => ({
+    ...deleteReleaseCandidateByNameMutation(),
+    onSuccess: () => {
+      this.messageService.add({ severity: 'success', summary: 'Release Candidate Deletion', detail: 'Release candidate has been deleted successfully' });
+      this.router.navigate(['..'], { relativeTo: this.route });
+    },
+  }));
+
   evaluateReleaseCandidate = (isWorking: boolean) => {
     this.evaluateReleaseCandidateMutation.mutate({ path: { name: this.name(), isWorking } });
   };
 
+  deleteReleaseCandidate = (rc: ReleaseCandidateDetailsDto) => {
+    this.confirmationService.confirm({
+      header: 'Delete Release Candidate',
+      message: `Are you sure you want to delete release candidate ${rc.name}? This cannot be undone.`,
+      accept: () => {
+        this.deleteReleaseCandidateMutation.mutate({ path: { name: rc.name } });
+      },
+    });
+  };
+
   hasUserEvaluatedTo(isWorking: boolean) {
     const evaluations = this.releaseCandidateQuery.data()?.evaluations;
     if (!evaluations) return false;

client/src/icons.module.ts

@@ -54,6 +54,7 @@ import {
   IconCircle,
   IconClock,
   IconTag,
+  IconTrash,
   IconX,
 } from 'angular-tabler-icons/icons';
 
@@ -111,6 +112,7 @@ const icons = {
   IconCircle,
   IconClock,
   IconTag,
+  IconTrash,
   IconX,
 };
 

server/application-server/openapi.yaml

@@ -474,6 +474,23 @@ paths:
             application/json:
               schema:
                 $ref: "#/components/schemas/ReleaseCandidateDetailsDto"
+    delete:
+      tags:
+      - release-candidate-controller
+      operationId: deleteReleaseCandidateByName
+      parameters:
+      - name: name
+        in: path
+        required: true
+        schema:
+          type: string
+      responses:
+        "200":
+          description: OK
+          content:
+            application/json:
+              schema:
+                $ref: "#/components/schemas/ReleaseCandidateInfoDto"
   /api/release-candidate/newcommits:
     get:
       tags:

server/application-server/src/main/java/de/tum/cit/aet/helios/releasecandidate/ReleaseCandidate.java

@@ -33,7 +33,7 @@ public class ReleaseCandidate {
 
   @Id private String name;
 
-  @OneToOne(optional = false, cascade = CascadeType.ALL)
+  @OneToOne(optional = false)
   private Commit commit;
 
   @ManyToOne(fetch = FetchType.LAZY)

server/application-server/src/main/java/de/tum/cit/aet/helios/releasecandidate/ReleaseCandidateController.java

@@ -4,6 +4,7 @@
 import java.util.List;
 import lombok.RequiredArgsConstructor;
 import org.springframework.http.ResponseEntity;
+import org.springframework.web.bind.annotation.DeleteMapping;
 import org.springframework.web.bind.annotation.GetMapping;
 import org.springframework.web.bind.annotation.PathVariable;
 import org.springframework.web.bind.annotation.PostMapping;
@@ -48,4 +49,10 @@ public ResponseEntity<Void> evaluate(@PathVariable String name, @PathVariable bo
     releaseCandidateService.evaluateReleaseCandidate(name, isWorking);
     return ResponseEntity.ok().build();
   }
+
+  @DeleteMapping("/{name}")
+  public ResponseEntity<ReleaseCandidateInfoDto> deleteReleaseCandidateByName(
+      @PathVariable String name) {
+    return ResponseEntity.ok(releaseCandidateService.deleteReleaseCandidateByName(name));
+  }
 }

server/application-server/src/main/java/de/tum/cit/aet/helios/releasecandidate/ReleaseCandidateRepository.java

@@ -19,4 +19,6 @@ Optional<ReleaseCandidate> findByRepositoryRepositoryIdAndCommitSha(
       Long repositoryId, String commitSha);
 
   boolean existsByRepositoryRepositoryIdAndName(Long repositoryId, String name);
+
+  Optional<ReleaseCandidate> deleteByRepositoryRepositoryIdAndName(Long repositoryId, String name);
 }

server/application-server/src/main/java/de/tum/cit/aet/helios/releasecandidate/ReleaseCandidateService.java

@@ -183,4 +183,18 @@ public void evaluateReleaseCandidate(String name, boolean isWorking) {
     evaluation.setWorking(isWorking);
     releaseCandidateEvaluationRepository.save(evaluation);
   }
+
+  public ReleaseCandidateInfoDto deleteReleaseCandidateByName(String name) {
+    final Long repositoryId = RepositoryContext.getRepositoryId();
+
+    ReleaseCandidateInfoDto rc = releaseCandidateRepository
+        .findByRepositoryRepositoryIdAndName(repositoryId, name)
+        .map(ReleaseCandidateInfoDto::fromReleaseCandidate)
+        .orElseThrow(() -> new ReleaseCandidateException("ReleaseCandidate could not be found."));
+
+    releaseCandidateRepository
+        .deleteByRepositoryRepositoryIdAndName(repositoryId, name);
+
+    return rc;
+  }
 }