fix(auth): hide error/success messages when user is authenticated or conflicting states exist
This commit is contained in:
@@ -68,7 +68,7 @@ export default function ForgotPasswordPage({ onSubmit, onNavigate, currentUser =
|
||||
</div>
|
||||
)}
|
||||
|
||||
{error && (
|
||||
{error && !currentUser && !success && (
|
||||
<div className="mb-4 p-3 bg-red-50 border border-red-200 rounded-lg dark:bg-red-500/10 dark:border-red-500/20">
|
||||
<div className="flex items-center space-x-2">
|
||||
<div className="w-1.5 h-1.5 bg-red-500 rounded-full flex-shrink-0"></div>
|
||||
@@ -77,7 +77,7 @@ export default function ForgotPasswordPage({ onSubmit, onNavigate, currentUser =
|
||||
</div>
|
||||
)}
|
||||
|
||||
{success && (
|
||||
{success && !currentUser && (
|
||||
<div className="mb-4 p-3 bg-green-50 border border-green-200 rounded-lg dark:bg-green-500/10 dark:border-green-500/20">
|
||||
<div className="flex items-center space-x-2">
|
||||
<div className="w-1.5 h-1.5 bg-green-500 rounded-full flex-shrink-0"></div>
|
||||
|
||||
@@ -85,7 +85,7 @@ export default function LoginPage({ onSubmit, onNavigate, onSetSessionCookie, re
|
||||
</div>
|
||||
)}
|
||||
|
||||
{success && (
|
||||
{success && !currentUser && (
|
||||
<div className="mb-4 p-3 bg-green-50 border border-green-200 rounded-lg dark:bg-green-500/10 dark:border-green-500/20">
|
||||
<div className="flex items-center space-x-2">
|
||||
<div className="w-1.5 h-1.5 bg-green-500 rounded-full flex-shrink-0"></div>
|
||||
@@ -94,7 +94,7 @@ export default function LoginPage({ onSubmit, onNavigate, onSetSessionCookie, re
|
||||
</div>
|
||||
)}
|
||||
|
||||
{error && (
|
||||
{error && !currentUser && !success && (
|
||||
<div className="mb-4 p-3 bg-red-50 border border-red-200 rounded-lg dark:bg-red-500/10 dark:border-red-500/20">
|
||||
<div className="flex items-center space-x-2">
|
||||
<div className="w-1.5 h-1.5 bg-red-500 rounded-full flex-shrink-0"></div>
|
||||
|
||||
@@ -60,7 +60,7 @@ export default function LogoutPage({ onLogout, onSetSessionCookie }) {
|
||||
</div>
|
||||
)}
|
||||
|
||||
{error && (
|
||||
{error && !success && (
|
||||
<div className="mb-4 p-3 bg-red-50 border border-red-200 rounded-lg dark:bg-red-500/10 dark:border-red-500/20">
|
||||
<div className="flex items-center space-x-2">
|
||||
<div className="w-1.5 h-1.5 bg-red-500 rounded-full flex-shrink-0"></div>
|
||||
|
||||
@@ -120,7 +120,7 @@ export default function RegisterPage({ onSubmit, onNavigate, currentUser = null
|
||||
</div>
|
||||
)}
|
||||
|
||||
{error && (
|
||||
{error && !currentUser && !success && (
|
||||
<div className="mb-4 p-3 bg-red-50 border border-red-200 rounded-lg dark:bg-red-500/10 dark:border-red-500/20">
|
||||
<div className="flex items-center space-x-2">
|
||||
<div className="w-1.5 h-1.5 bg-red-500 rounded-full flex-shrink-0"></div>
|
||||
@@ -129,7 +129,7 @@ export default function RegisterPage({ onSubmit, onNavigate, currentUser = null
|
||||
</div>
|
||||
)}
|
||||
|
||||
{success && (
|
||||
{success && !currentUser && (
|
||||
<div className="mb-4 p-3 bg-green-50 border border-green-200 rounded-lg dark:bg-green-500/10 dark:border-green-500/20">
|
||||
<div className="flex items-center space-x-2">
|
||||
<div className="w-1.5 h-1.5 bg-green-500 rounded-full flex-shrink-0"></div>
|
||||
|
||||
@@ -74,7 +74,7 @@ export default function ResetPasswordPage({ onSubmit, onNavigate, email, token }
|
||||
</p>
|
||||
</div>
|
||||
|
||||
{error && (
|
||||
{error && !success && (
|
||||
<div className="mb-4 p-3 bg-red-50 border border-red-200 rounded-lg dark:bg-red-500/10 dark:border-red-500/20">
|
||||
<div className="flex items-center space-x-2">
|
||||
<div className="w-1.5 h-1.5 bg-red-500 rounded-full flex-shrink-0"></div>
|
||||
|
||||
Reference in New Issue
Block a user