From e661161eac3fed4ab92bb835873740d3063a436a Mon Sep 17 00:00:00 2001
From: Jaanu423 <puja.mourya575@gmail.com>
Date: Wed, 17 Jun 2026 11:07:01 +0530
Subject: [PATCH] feat(clerk): enhance disbursement review and my-requests UI
 components

- Update disbursement review and my-disbursement-requests components
  with improved layout and additional UI enhancements

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 .../disbursement-review.component.html        | 33 +++++++++++++++++++
 .../disbursement-review.component.scss        | 27 +++++++++++++++
 .../disbursement-review.component.ts          |  4 ++-
 .../my-disbursement-requests.component.html   |  8 +++++
 .../my-disbursement-requests.component.scss   | 18 ++++++++++
 .../my-disbursement-requests.component.ts     | 11 +++++--
 6 files changed, 97 insertions(+), 4 deletions(-)

diff --git a/src/app/clerk/disbursement-review/disbursement-review.component.html b/src/app/clerk/disbursement-review/disbursement-review.component.html
index 8d89420ec..29b14614d 100644
--- a/src/app/clerk/disbursement-review/disbursement-review.component.html
+++ b/src/app/clerk/disbursement-review/disbursement-review.component.html
@@ -135,6 +135,17 @@
         </table>
       </div>
 
+      <!-- Processing indicator -->
+      @if (request.status === 'PROCESSING') {
+        <div class="dri-card dri-card--processing">
+          <div class="dri-section-title">
+            <mat-icon>hourglass_top</mat-icon>
+            Processing
+          </div>
+          <p class="dri-processing-text">Transactions are being posted. Please refresh in a few moments.</p>
+        </div>
+      }
+
       <!-- Rejection reason (if rejected) -->
       @if (request.status === 'REJECTED' && request.rejection_reason) {
         <div class="dri-card dri-card--rejected">
@@ -150,6 +161,28 @@
         </div>
       }
 
+      <!-- Processing failed (system error — transactions reversed) -->
+      @if (request.status === 'FAILED') {
+        <div class="dri-card dri-card--failed">
+          <div class="dri-section-title">
+            <mat-icon>error</mat-icon>
+            Processing Failed — Transactions Reversed
+          </div>
+          <p class="dri-rejection-text">
+            {{ request.error_message || 'An unexpected error occurred during approval.' }}
+          </p>
+          <p class="dri-processing-text" style="margin-top: 6px; font-size: 12px">
+            All partial transactions have been automatically reversed. The request can be re-submitted by the clerk.
+          </p>
+          @if (request.manager_name || request.manager_username) {
+            <div class="dri-meta-item" style="margin-top: 8px">
+              <mat-icon>manage_accounts</mat-icon>
+              Attempted by: {{ request.manager_name || request.manager_username }}
+            </div>
+          }
+        </div>
+      }
+
       <!-- Txn refs (if approved) -->
       @if (request.status === 'APPROVED' && request.txn_refs) {
         <div class="dri-card dri-card--approved">
diff --git a/src/app/clerk/disbursement-review/disbursement-review.component.scss b/src/app/clerk/disbursement-review/disbursement-review.component.scss
index 5fbb8c52f..4364d9c8b 100644
--- a/src/app/clerk/disbursement-review/disbursement-review.component.scss
+++ b/src/app/clerk/disbursement-review/disbursement-review.component.scss
@@ -61,6 +61,16 @@
   &--approved {
     border-top: 3px solid #22c55e;
   }
+
+  &--failed {
+    border-top: 3px solid #dc2626;
+    background: #fff5f5;
+  }
+
+  &--processing {
+    border-top: 3px solid #f59e0b;
+    background: #fffbeb;
+  }
 }
 
 .dri-card-header {
@@ -114,6 +124,23 @@
     background: #fee2e2;
     color: #991b1b;
   }
+
+  &--processing {
+    background: #fef3c7;
+    color: #78350f;
+  }
+
+  &--failed {
+    background: #fee2e2;
+    color: #7f1d1d;
+  }
+}
+
+.dri-processing-text {
+  padding: 6px 20px 14px;
+  margin: 0;
+  font-size: 13px;
+  color: #78350f;
 }
 
 .dri-meta-item {
diff --git a/src/app/clerk/disbursement-review/disbursement-review.component.ts b/src/app/clerk/disbursement-review/disbursement-review.component.ts
index 8b4fd670d..54ab4c282 100644
--- a/src/app/clerk/disbursement-review/disbursement-review.component.ts
+++ b/src/app/clerk/disbursement-review/disbursement-review.component.ts
@@ -134,8 +134,10 @@ export class DisbursementReviewComponent implements OnInit {
   statusClass(status: string): string {
     const map: Record<string, string> = {
       PENDING: 'dri-status--pending',
+      PROCESSING: 'dri-status--processing',
       APPROVED: 'dri-status--approved',
-      REJECTED: 'dri-status--rejected'
+      REJECTED: 'dri-status--rejected',
+      FAILED: 'dri-status--failed'
     };
     return map[status] ?? '';
   }
diff --git a/src/app/clerk/my-disbursement-requests/my-disbursement-requests.component.html b/src/app/clerk/my-disbursement-requests/my-disbursement-requests.component.html
index bf2b1b46a..a5dc03299 100644
--- a/src/app/clerk/my-disbursement-requests/my-disbursement-requests.component.html
+++ b/src/app/clerk/my-disbursement-requests/my-disbursement-requests.component.html
@@ -189,6 +189,14 @@
                 </td>
               </tr>
             }
+            @if (r.status === 'FAILED' && r.error_message) {
+              <tr class="mdr-reason-row mdr-reason-row--failed">
+                <td colspan="8" class="mdr-reason-cell mdr-reason-cell--failed">
+                  <mat-icon class="mdr-reason-icon">error_outline</mat-icon>
+                  Processing failed &amp; transactions reversed: {{ r.error_message }}
+                </td>
+              </tr>
+            }
           }
         </tbody>
       </table>
diff --git a/src/app/clerk/my-disbursement-requests/my-disbursement-requests.component.scss b/src/app/clerk/my-disbursement-requests/my-disbursement-requests.component.scss
index 3593f134a..0557af9b1 100644
--- a/src/app/clerk/my-disbursement-requests/my-disbursement-requests.component.scss
+++ b/src/app/clerk/my-disbursement-requests/my-disbursement-requests.component.scss
@@ -312,6 +312,16 @@
     background: #fee2e2;
     color: #991b1b;
   }
+
+  &--processing {
+    background: #fef3c7;
+    color: #78350f;
+  }
+
+  &--failed {
+    background: #fee2e2;
+    color: #7f1d1d;
+  }
 }
 
 .cell-mono {
@@ -432,6 +442,10 @@
   background: #fff7f7;
 }
 
+.mdr-reason-row--failed td {
+  background: #fff1f1;
+}
+
 .mdr-reason-cell {
   display: flex;
   align-items: center;
@@ -442,6 +456,10 @@
   padding: 6px 14px 10px;
 }
 
+.mdr-reason-cell--failed {
+  color: #7f1d1d;
+}
+
 .mdr-reason-icon {
   font-size: 14px;
   width: 14px;
diff --git a/src/app/clerk/my-disbursement-requests/my-disbursement-requests.component.ts b/src/app/clerk/my-disbursement-requests/my-disbursement-requests.component.ts
index 207f96317..c75d6cac9 100644
--- a/src/app/clerk/my-disbursement-requests/my-disbursement-requests.component.ts
+++ b/src/app/clerk/my-disbursement-requests/my-disbursement-requests.component.ts
@@ -74,7 +74,8 @@ export class MyDisbursementRequestsComponent implements OnInit {
       { value: '', label: 'All' },
       { value: 'PENDING', label: 'Pending' },
       { value: 'APPROVED', label: 'Approved' },
-      { value: 'REJECTED', label: 'Rejected' }
+      { value: 'REJECTED', label: 'Rejected' },
+      { value: 'FAILED', label: 'Failed' }
     ];
   }
 
@@ -189,8 +190,10 @@ export class MyDisbursementRequestsComponent implements OnInit {
   statusClass(status: string): string {
     const map: Record<string, string> = {
       PENDING: 'mdr-status--pending',
+      PROCESSING: 'mdr-status--processing',
       APPROVED: 'mdr-status--approved',
-      REJECTED: 'mdr-status--rejected'
+      REJECTED: 'mdr-status--rejected',
+      FAILED: 'mdr-status--failed'
     };
     return map[status] ?? '';
   }
@@ -198,8 +201,10 @@ export class MyDisbursementRequestsComponent implements OnInit {
   statusLabel(status: string): string {
     const map: Record<string, string> = {
       PENDING: 'Pending Approval',
+      PROCESSING: 'Processing…',
       APPROVED: 'Approved',
-      REJECTED: 'Rejected'
+      REJECTED: 'Rejected',
+      FAILED: 'Processing Failed'
     };
     return map[status] ?? status;
   }