fix(billing): use onSuccess callback for Back to Billing navigation

The ManualTransferForm's internal submitted state was being reset by Server
Component re-render errors, causing the Back to Billing button to reset the
form instead of navigating. Now uses a dedicated onSuccess prop that always
navigates to the billing page after successful submission.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: guillermoscript

app/[locale]/dashboard/admin/billing/upgrade/upgrade-page-client.tsx

@@ -29,7 +29,6 @@ export function UpgradePageClient({ plans, currentPlan }: UpgradePageClientProps
   const locale = useLocale()
   const t = useTranslations('dashboard.admin.billing.upgrade')
   const [loading, setLoading] = useState(false)
-  const [manualSubmitted, setManualSubmitted] = useState(false)
   const [manualTransfer, setManualTransfer] = useState<{
     planId: string
     planName: string
@@ -75,26 +74,24 @@ export function UpgradePageClient({ plans, currentPlan }: UpgradePageClientProps
     if (!manualTransfer) return
     try {
       await requestManualPlanUpgrade(manualTransfer.planId, manualTransfer.interval, bankReference, notes)
-      setManualSubmitted(true)
     } catch (e: any) {
       toast.error(e.message || t('submitError'))
     }
   }
 
+  const navigateToBilling = () => {
+    router.push(`/${locale}/dashboard/admin/billing`)
+  }
+
   if (manualTransfer) {
     return (
       <ManualTransferForm
         planName={manualTransfer.planName}
         amount={manualTransfer.amount}
         interval={manualTransfer.interval}
         onSubmit={handleManualSubmit}
-        onCancel={() => {
-          if (manualSubmitted) {
-            router.push(`/${locale}/dashboard/admin/billing`)
-          } else {
-            setManualTransfer(null)
-          }
-        }}
+        onSuccess={navigateToBilling}
+        onCancel={() => setManualTransfer(null)}
       />
     )
   }

components/admin/manual-transfer-form.tsx

@@ -15,6 +15,7 @@ interface ManualTransferFormProps {
   interval: string
   onSubmit: (bankReference: string, notes: string) => Promise<void>
   onProofUpload?: (file: File) => Promise<void>
+  onSuccess?: () => void
   onCancel: () => void
 }
 
@@ -24,6 +25,7 @@ export function ManualTransferForm({
   interval,
   onSubmit,
   onProofUpload,
+  onSuccess,
   onCancel,
 }: ManualTransferFormProps) {
   const [bankReference, setBankReference] = useState('')
@@ -55,7 +57,7 @@ export function ManualTransferForm({
               Your upgrade request has been submitted. We&apos;ll send you bank transfer instructions shortly.
             </p>
           </div>
-          <Button variant="outline" onClick={onCancel}>Back to Billing</Button>
+          <Button variant="outline" onClick={onSuccess || onCancel}>Back to Billing</Button>
         </CardContent>
       </Card>
     )