From 95a926454b970c9afc7afcc6d739be91459484a5 Mon Sep 17 00:00:00 2001
From: Chandan-walker <chandan@whozzat.com>
Date: Wed, 21 Oct 2026 19:40:24 +0530
Subject: [PATCH] fixed error to disable buttons once decision api is success

---
 .../src/app/otp/widget/widget.component.html  | 19 +++++++++++++++++--
 .../src/app/otp/widget/widget.component.ts    | 14 +++++++++-----
 2 files changed, 26 insertions(+), 7 deletions(-)

diff --git a/apps/36-blocks-widget/src/app/otp/widget/widget.component.html b/apps/36-blocks-widget/src/app/otp/widget/widget.component.html
index f3111046..b217a144 100644
--- a/apps/36-blocks-widget/src/app/otp/widget/widget.component.html
+++ b/apps/36-blocks-widget/src/app/otp/widget/widget.component.html
@@ -112,7 +112,22 @@ <h2 id="oauth-consent-title" class="w-dialog-title flex-1">
                         [disabled]="oauthConsentLoading()"
                         (click)="submitOauthConsent(false)"
                     >
-                        Deny
+                        @if (oauthConsentLoading() && oauthConsentAction() === 'deny') {
+                            <svg class="w-spinner" aria-hidden="true" fill="none" viewBox="0 0 24 24">
+                                <circle
+                                    class="opacity-25"
+                                    cx="12"
+                                    cy="12"
+                                    r="10"
+                                    stroke="currentColor"
+                                    stroke-width="4"
+                                />
+                                <path class="opacity-75" fill="currentColor" d="M4 12a8 8 0 018-8v8z" />
+                            </svg>
+                            Please wait...
+                        } @else {
+                            Deny
+                        }
                     </button>
                     <button
                         type="button"
@@ -120,7 +135,7 @@ <h2 id="oauth-consent-title" class="w-dialog-title flex-1">
                         [disabled]="oauthConsentLoading() || !oauthAuthorizeData"
                         (click)="submitOauthConsent(true)"
                     >
-                        @if (oauthConsentLoading()) {
+                        @if (oauthConsentLoading() && oauthConsentAction() === 'approve') {
                             <svg class="w-spinner" aria-hidden="true" fill="none" viewBox="0 0 24 24">
                                 <circle
                                     class="opacity-25"
diff --git a/apps/36-blocks-widget/src/app/otp/widget/widget.component.ts b/apps/36-blocks-widget/src/app/otp/widget/widget.component.ts
index 77629032..17eb1803 100644
--- a/apps/36-blocks-widget/src/app/otp/widget/widget.component.ts
+++ b/apps/36-blocks-widget/src/app/otp/widget/widget.component.ts
@@ -121,6 +121,8 @@ export class ProxyAuthWidgetComponent extends BaseComponent implements OnInit, O
     private oauthProxyAuthToken: string | null = null;
     public readonly showOauthConsent = signal<boolean>(false);
     public readonly oauthConsentLoading = signal<boolean>(false);
+    /** Which consent button is in flight, so only that one shows the spinner. */
+    public readonly oauthConsentAction = signal<'approve' | 'deny' | null>(null);
     public readonly oauthConsentError = signal<string>('');
     /** Static error when GET /oauth/authorize fails validation (invalid client_id, etc.). */
     public readonly showOauthError = signal<boolean>(false);
@@ -607,7 +609,9 @@ export class ProxyAuthWidgetComponent extends BaseComponent implements OnInit, O
             return;
         }
         this.oauthConsentLoading.set(true);
+        this.oauthConsentAction.set(approved ? 'approve' : 'deny');
         this.oauthConsentError.set('');
+        this.cdr.markForCheck();
 
         const data = this.oauthAuthorizeData || {};
         const scopes = data.scopes;
@@ -643,21 +647,21 @@ export class ProxyAuthWidgetComponent extends BaseComponent implements OnInit, O
             .pipe(takeUntil(this.destroy$))
             .subscribe({
                 next: (res: any) => {
-                    this.oauthConsentLoading.set(false);
                     const dest = res?.data?.redirect_to;
-                    if (dest && typeof dest === 'string') {
-                        window.location.href = dest;
+                    if (typeof dest === 'string' && dest) {
+                        window.location.replace(dest);
                         return;
                     }
-                    console.error('[36Blocks] OAuth decision response missing redirect_to:', res);
+                    this.oauthConsentLoading.set(false);
+                    this.oauthConsentAction.set(null);
                     this.oauthConsentError.set('Consent succeeded but no redirect URL was returned.');
                     this.cdr.markForCheck();
                 },
                 error: (err) => {
                     console.error('[36Blocks] OAuth decision failed:', err);
                     this.oauthConsentLoading.set(false);
+                    this.oauthConsentAction.set(null);
                     const body = err?.error ?? err;
-                    // OAuth error body: { error, error_description } — prefer API message over HttpClient text.
                     const oauthError = body?.error || body?.error_description || body?.errors?.[0] || err?.errors?.[0];
                     this.oauthConsentError.set(
                         typeof oauthError === 'string' && oauthError.trim()