Cleanup after api key

Author: AramAlsabti

src/app/admin/api-key/api-key-edit/api-key-edit.component.ts

@@ -24,12 +24,10 @@ export class ApiKeyEditComponent implements OnInit {
   };
   public title = '';
   public submitButton = '';
-  private id: number;
   public errorMessage: string;
   public errorMessages: string[];
   public errorFields: string[];
   public formFailedSubmit = false;
-  public isEditMode = false;
   public permissions: PermissionResponse[] = [];
   private organizationId: number;
 
@@ -56,30 +54,9 @@ export class ApiKeyEditComponent implements OnInit {
         this.submitButton = translations['API-KEY.EDIT.SAVE'];
       });
 
-    this.id = +this.route.snapshot.paramMap.get('api-key-id');
     this.organizationId = this.sharedVariableService.getSelectedOrganisationId();
-
-    // if (this.id > 0) {
-    //   // TODO: Fetch current api key
-    //   this.getApiKey(this.id);
-    //   this.isEditMode = true;
-    // }
   }
 
-  // private getApiKey(id: number) {
-  //   this.apiKeyService.get(id).subscribe(
-  //     (response) => {
-  //       console.log(response);
-  //       this.apiKeyRequest.id = response.id;
-  //       this.apiKeyRequest.name = response.name;
-  //       this.apiKeyRequest.permissions = response.permissions;
-  //     },
-  //     (error: HttpErrorResponse) => {
-  //       this.showError(error);
-  //     }
-  //   );
-  // }
-
   private getPermissions() {
     this.permissionService
       .getPermissions(
@@ -103,7 +80,7 @@ export class ApiKeyEditComponent implements OnInit {
   }
 
   onSubmit(): void {
-    this.id ? this.update() : this.create();
+    this.create();
   }
 
   private create(): void {
@@ -113,12 +90,6 @@ export class ApiKeyEditComponent implements OnInit {
     );
   }
 
-  private update(): void {
-    this.apiKeyService
-      .update(this.apiKeyRequest, this.id)
-      .subscribe(this.routeBack, this.showError);
-  }
-
   public compare(o1: any, o2: any): boolean {
     return o1 === o2;
   }

src/app/admin/api-key/api-key-list/api-key-table/api-key-table.component.html

@@ -50,35 +50,12 @@
     <ng-container matColumnDef="menu">
       <th mat-header-cell *matHeaderCellDef></th>
       <td mat-cell *matCellDef="let element">
-        <div class="dropdown" *ngIf="canAccess(element)">
-          <a
-            href="#"
-            role="button"
-            id="tableRowDropdown-{{ element.id }}"
-            class="applicationRow__edit dropdown-toggle"
-            data-toggle="dropdown"
-            aria-expanded="false"
-            [attr.aria-label]="'APPLICATION-TABLE-ROW.SHOW-OPTIONS' | translate"
-          ></a>
-          <ul
-            class="dropdown-menu dropdown-menu--table"
-            attr.aria-labelledby="tableRowDropdown-{{ element.id }}"
-          >
-            <!-- TODO: Delete. Edit is not part of estimate. Since it's only one option, use a delete button instead -->
-            <!-- <li class="dropdown-item">
-              <a
-                [routerLink]="['/admin/api-key', element.id, 'edit-api-key']"
-                routerLinkActive="active"
-                >{{ 'API-KEY.TABLE-ROW.EDIT' | translate }}
-              </a>
-            </li> -->
-            <li class="dropdown-item">
-              <a (click)="deleteApiKey(element.id)" [routerLink]=""
-                >{{ 'API-KEY.TABLE-ROW.DELETE' | translate }}
-              </a>
-            </li>
-          </ul>
-        </div>
+        <a
+          *ngIf="canAccess(element)"
+          (click)="deleteApiKey(element.id)"
+          [routerLink]=""
+          >{{ 'API-KEY.TABLE-ROW.DELETE' | translate }}
+        </a>
       </td>
     </ng-container>
 

src/app/admin/api-key/api-key.service.ts

@@ -25,12 +25,6 @@ export class ApiKeyService {
     });
   }
 
-  update(body: ApiKeyRequest, id: number): Observable<ApiKeyResponse> {
-    return this.restService.put(this.endpoint, body, id, {
-      observe: 'response',
-    });
-  }
-
   get(id: number): Observable<ApiKeyResponse> {
     return this.restService.get(this.endpoint, {}, id).pipe(
       map((response: ApiKeyResponse) => {