Merge pull request 'refactor: remove server configuration and routes for admin module; simplify routing structure' (#2) from task/popup-for-dynamic-form into develop

Reviewed-on: #2

src/app/admin/projects/project-detail-popup/project-detail-popup.html

@@ -88,7 +88,7 @@
     </div>
     }
 
-    @if (project.categories.length) {
+    @if (project.categories?.length) {
     <div class="detail-section">
         <h4>Categories</h4>
         <div class="tag-list">

src/app/admin/projects/project-detail-popup/project-detail-popup.scss

@@ -4,7 +4,7 @@
     max-height: 80vh;
     overflow-y: auto;
     background: var(--eerie-black-2, #1e1e1e);
-    border: 1.2px solid hsla(45, 100%, 72%, 0.45);
+    border: 1px solid var(--jet, #383838);
     border-radius: 14px;
 }
 

src/app/auth/auth.service.ts

@@ -111,7 +111,7 @@ export class AuthService {
     this.http.post<void>(this.api('/api/v1/auth/logout'), {}).subscribe();
     this.accessToken = null;
     this.safeRemoveToken();
-    this.router.navigateByUrl('login');
+    this.router.navigate(['/admin/login']);
   }
 
   refreshToken(): Observable<RefreshTokenResponse> {

src/app/auth/otp/otp.component.html

@@ -21,9 +21,19 @@
         <!-- Step 2: OTP Input -->
         @if (isOtpSent() && !isVerified()) {
         <div class="otp-section">
+            <p class="info-text">
+                OTP sent to <b>{{ emailForm.value.email }}</b>
+            </p>
+
             <form [formGroup]="otpForm" (ngSubmit)="verifyOtp()">
                 <label for="otp">Enter 6-digit OTP</label>
                 <input id="otp" type="text" maxlength="6" formControlName="otp" placeholder="123456" />
+                @if (isError()) {
+                <div class="error-banner">
+                    <i class="fa-solid fa-circle-exclamation"></i>
+                    <span>{{ message() }}</span>
+                </div>
+                }
                 <button type="submit" [disabled]="otpForm.invalid">
                     Verify OTP
                 </button>
@@ -37,19 +47,5 @@
             </button>
         </div>
         }
-
-        @if (isError()) {
-        <div class="error-banner">
-            <i class="fa-solid fa-circle-exclamation"></i>
-            <span>{{ message() }}</span>
-        </div>
-        }
-
-        @if (!isError() && isOtpSent() && !isVerified()) {
-        <div class="success-banner">
-            <i class="fa-solid fa-circle-check"></i>
-            <span>{{ message() }}</span>
-        </div>
-        }
     </div>
 </div>

src/app/auth/otp/otp.component.scss

@@ -171,23 +171,3 @@ button {
     font-size: 14px;
     margin-bottom: 10px;
 }
-
-/* Success banner for OTP sent */
-.success-banner {
-    display: flex;
-    align-items: center;
-    gap: 8px;
-    background: rgba(120, 255, 140, 0.07);
-    border: 1px solid rgba(120, 255, 140, 0.25);
-    border-radius: 8px;
-    padding: 10px 14px;
-    margin: 10px 0 10px;
-    color: #78ff8c;
-    font-size: 13px;
-
-    i {
-        font-size: 15px;
-        flex-shrink: 0;
-        color: #78ff8c;
-    }
-}

src/app/auth/otp/otp.component.ts

@@ -19,7 +19,7 @@ export class OtpComponent implements OnInit {
   message = signal('');
   countdown = signal(0);
   timer: NodeJS.Timeout | undefined;
-  returnUrl = '';
+  returnUrl = '/';
   fb: FormBuilder = inject(FormBuilder);
   authService: AuthService = inject(AuthService);
   router: Router = inject(Router);
@@ -35,40 +35,28 @@ export class OtpComponent implements OnInit {
     });
 
     if(this.authService.isLoggedIn()){
-      this.router.navigate(['']);
+      this.router.navigateByUrl('/');
     }
   }
 
 
   ngOnInit() {
-    this.returnUrl = this.route.snapshot.queryParamMap.get('returnUrl') || '';
+    this.returnUrl = this.route.snapshot.queryParamMap.get('returnUrl') || '/';
   }
 
   sendOtp() {
     if (this.emailForm.invalid) return;
     const email = this.emailForm.value.email;
 
-    this.authService.sendOtp(email).subscribe({
-      next: () => {
+    this.authService.sendOtp(email).subscribe(() => {
       this.isOtpSent.set(true);
-        this.isError.set(false);
-        this.message.set(`OTP sent to ${email}.`);
+      this.message.set('OTP sent successfully!');
       this.startTimer(30); // 30 seconds countdown
-      },
-      error: (err) => {
-        this.isError.set(true);
-        if (err.status === 404) {
-          this.message.set('Email not found. Please check and try again.');
-        } else {
-          this.message.set('Failed to send OTP. Please try again.');
-        }
-      }
     });
   }
 
   resendOtp() {
     if (this.countdown() > 0) return;
-    this.otpForm.reset();
     this.sendOtp();
   }
 

src/app/guards/auth-guard.ts

@@ -16,5 +16,5 @@ export const authGuard: CanActivateFn = (route, state) => {
 
   return auth.currentToken
     ? true
-    : router.parseUrl(`login?returnUrl=${state.url}`);
+    : router.parseUrl(`/admin/login?returnUrl=${state.url}`);
 };

src/app/shared/dynamic-form/dynamic-form.scss

@@ -1,18 +1,10 @@
 /* Dynamic form spacing and material tweaks to match popup theme */
-.full-width {
-    width: 100%;
-    display: block;
-}
+.full-width { width: 100%; display: block; }
 
-mat-form-field.full-width {
-    margin-bottom: 12px;
-}
+mat-form-field.full-width { margin-bottom: 12px; }
 
 /* Reduce label size slightly to fit popup */
-mat-form-field .mat-form-field-label {
-    font-size: 0.95rem;
-    color: var(--light-gray-70);
-}
+mat-form-field .mat-form-field-label { font-size: 0.95rem; color: var(--light-gray-70); }
 
 /* Make input text contrast better */
 .mat-input-element,
@@ -23,16 +15,10 @@ textarea.mat-mdc-input-element,
 }
 
 /* Smaller helper/hint text */
-.mat-hint {
-    color: var(--light-gray-70);
-    font-size: 0.85rem;
-}
+.mat-hint { color: var(--light-gray-70); font-size: 0.85rem; }
 
 /* Error styling consistent with theme */
-mat-error {
-    color: #ff8a80;
-    font-size: 0.9rem;
-}
+mat-error { color: #ff8a80; font-size: 0.9rem; }
 
 /* Outline border always visible, highlight on focus */
 ::ng-deep .mdc-notched-outline__leading,
@@ -48,18 +34,8 @@ mat-error {
 }
 
 /* Make array item fields inline on larger screens */
-.array-item-fields {
-    display: flex;
-    gap: 12px;
-    align-items: baseline;
-    flex-wrap: wrap;
-}
-
-.array-item-fields mat-form-field {
-    flex: 1 1 auto;
-    margin-bottom: 0;
-    min-width: 120px;
-}
+.array-item-fields { display: flex; gap: 12px; align-items: baseline; flex-wrap: wrap; }
+.array-item-fields mat-form-field { flex: 1 1 auto; margin-bottom: 0; min-width: 120px; }
 
 /* Stack array item fields vertically on small screens */
 @media (max-width: 580px) {
@@ -107,7 +83,6 @@ mat-error {
 
 /* Force material label/input colors to match dark popup theme (covers MDC + legacy classes) */
 .themed-popup {
-
     ::ng-deep .mat-form-field-label,
     ::ng-deep .mat-mdc-floating-label,
     ::ng-deep .mat-form-field .mat-form-field-label {
@@ -132,7 +107,6 @@ mat-error {
         opacity: 1 !important;
     }
 }
-
 .form-array {
     margin-bottom: 20px;
     padding: 10px 0;

src/styles.scss

@@ -507,9 +507,9 @@
   .mat-mdc-text-field.mat-mdc-focused .mat-mdc-notched-outline .mat-mdc-notched-outline-path {
     stroke: var(--orange-yellow-crayola) !important;
     border-color: var(--orange-yellow-crayola) !important;
-  }
+
 /* Additional overrides: set MDC theme variables and force a gold focus ring */
-// .themed-popup {
+.themed-popup {
   /* Override MDC primary color used by many Material components */
   --mdc-theme-primary: var(--orange-yellow-crayola);
   --mdc-theme-on-primary: var(--smoky-black);
@@ -529,8 +529,9 @@
   .mat-mdc-text-field.mat-mdc-focused .mat-mdc-notched-outline-path {
     stroke: var(--orange-yellow-crayola) !important;
   }
-// }
-    // box-shadow: 0 6px 18px rgba(36,26,18,0.12) !important;
+}
+    box-shadow: 0 6px 18px rgba(36,26,18,0.12) !important;
+  }
 }
   
   .contacts-list {