refactor: move error handling display to a new location in the Authorize component

Author: kentcdodds

epicshop/epic-me/app/routes/oauth/authorize.tsx

@@ -223,34 +223,6 @@ export default function Authorize({
 					</div>
 				</div>
 
-				{actionData?.status === 'error' && actionData?.message ? (
-					<div className="mb-8 rounded-xl border border-red-200 bg-red-50 p-6 dark:border-red-700 dark:bg-red-900/20">
-						<div className="flex">
-							<div className="flex-shrink-0">
-								<svg
-									className="h-5 w-5 text-red-400 dark:text-red-300"
-									viewBox="0 0 20 20"
-									fill="currentColor"
-								>
-									<path
-										fillRule="evenodd"
-										d="M10 18a8 8 0 100-16 8 8 0 000 16zM8.707 7.293a1 1 0 00-1.414 1.414L8.586 10l-1.293 1.293a1 1 0 101.414 1.414L10 11.414l1.293 1.293a1 1 0 001.414-1.414L11.414 10l1.293-1.293a1 1 0 00-1.414-1.414L10 8.586 8.707 7.293z"
-										clipRule="evenodd"
-									/>
-								</svg>
-							</div>
-							<div className="ml-3">
-								<h3 className="text-sm font-medium text-red-800 dark:text-red-200">
-									Authorization Failed
-								</h3>
-								<div className="mt-2 text-sm text-red-700 dark:text-red-300">
-									<p>{actionData.message}</p>
-								</div>
-							</div>
-						</div>
-					</div>
-				) : null}
-
 				<div className="mb-8 rounded-lg border border-gray-200 bg-gray-50 p-4 dark:border-gray-700 dark:bg-gray-700">
 					<h3 className="mb-2 text-sm font-medium text-gray-900 dark:text-white">
 						What happens when you authorize?
@@ -433,6 +405,34 @@ export default function Authorize({
 						</div>
 					</div>
 
+					{actionData?.status === 'error' && actionData?.message ? (
+						<div className="mb-8 rounded-xl border border-red-200 bg-red-50 p-6 dark:border-red-700 dark:bg-red-900/20">
+							<div className="flex">
+								<div className="flex-shrink-0">
+									<svg
+										className="h-5 w-5 text-red-400 dark:text-red-300"
+										viewBox="0 0 20 20"
+										fill="currentColor"
+									>
+										<path
+											fillRule="evenodd"
+											d="M10 18a8 8 0 100-16 8 8 0 000 16zM8.707 7.293a1 1 0 00-1.414 1.414L8.586 10l-1.293 1.293a1 1 0 101.414 1.414L10 11.414l1.293 1.293a1 1 0 001.414-1.414L11.414 10l1.293-1.293a1 1 0 00-1.414-1.414L10 8.586 8.707 7.293z"
+											clipRule="evenodd"
+										/>
+									</svg>
+								</div>
+								<div className="ml-3">
+									<h3 className="text-sm font-medium text-red-800 dark:text-red-200">
+										Authorization Failed
+									</h3>
+									<div className="mt-2 text-sm text-red-700 dark:text-red-300">
+										<p>{actionData.message}</p>
+									</div>
+								</div>
+							</div>
+						</div>
+					) : null}
+
 					{users.length > 0 ? (
 						<div className="grid gap-4">
 							{users.map((user) => {