From 54d64f5927dddaa5b191f51290a6a1c5563c4ddb Mon Sep 17 00:00:00 2001 From: Claude Date: Tue, 5 Aug 2025 09:22:13 +0530 Subject: [PATCH] =?UTF-8?q?=E2=9C=A8=20Streamline=20user=20experience=20wi?= =?UTF-8?q?th=20cleaner=20UI=20and=20improved=20authentication=20flow?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Remove special styling for career navigator button, unify all Try Now buttons - Clean up authentication messaging system to prevent message confusion - Replace immediate success messages with direct redirections for smoother UX - Add session-based post-login messaging for better redirect handling - Improve login page with verification resend button and cleaner message display - Remove redundant payment success messages for free agents 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude --- agents/templates/agents/marketplace.html | 24 ++---------- agents/views.py | 48 ++++++++++++------------ authentication/views.py | 23 +++++++++--- templates/authentication/login.html | 40 +++++++++++++++++++- 4 files changed, 83 insertions(+), 52 deletions(-) diff --git a/agents/templates/agents/marketplace.html b/agents/templates/agents/marketplace.html index d17c1a8..1361da7 100644 --- a/agents/templates/agents/marketplace.html +++ b/agents/templates/agents/marketplace.html @@ -7,23 +7,7 @@ {% endblock %} @@ -92,12 +76,12 @@
{% if user.is_authenticated %} {% if agent.slug == 'cybersec-career-navigator' %} - - 🎓 Try Now → + + Try Now → {% elif agent.slug == 'ai-brand-strategist' %} - 🧠 Try Now → + Try Now → {% else %} Try Now → diff --git a/agents/views.py b/agents/views.py index ad8e21b..6860146 100644 --- a/agents/views.py +++ b/agents/views.py @@ -196,10 +196,12 @@ def execute_agent(request): def career_navigator_access(request): """Handle Try Now button click - charge wallet and redirect to form""" if not request.user.is_authenticated: - # Clear any existing messages to prevent confusion + # Clear all existing messages before adding login message storage = messages.get_messages(request) - storage.used = True - messages.error(request, 'Please login to access the Career Navigator.') + for _ in storage: + pass # Consume all messages + # Add login message to session for after login redirect + request.session['post_login_message'] = 'Please complete your login to access the Career Navigator.' return redirect('authentication:login') # Get the career navigator agent @@ -240,21 +242,19 @@ def career_navigator_access(request): completed_at=timezone.now() ) - # Success message and redirect to form - if agent.price > 0: - messages.success(request, f'Welcome to your {agent.name} consultation.') - else: - messages.success(request, f'Welcome to your {agent.name} consultation.') + # Redirect directly to form - no message needed return redirect('agents:career_navigator') def career_navigator_view(request): """Display the career navigator form page""" if not request.user.is_authenticated: - # Clear any existing messages to prevent confusion + # Clear all existing messages before adding login message storage = messages.get_messages(request) - storage.used = True - messages.error(request, 'Please login to access the Career Navigator.') + for _ in storage: + pass # Consume all messages + # Add login message to session for after login redirect + request.session['post_login_message'] = 'Please complete your login to access the Career Navigator.' return redirect('authentication:login') # Get the career navigator agent @@ -292,10 +292,12 @@ def career_navigator_view(request): def ai_brand_strategist_view(request): """Display the AI Brand Strategist form page""" if not request.user.is_authenticated: - # Clear any existing messages to prevent confusion + # Clear all existing messages before adding login message storage = messages.get_messages(request) - storage.used = True - messages.error(request, 'Please login to access the AI Brand Strategist.') + for _ in storage: + pass # Consume all messages + # Add login message to session for after login redirect + request.session['post_login_message'] = 'Please complete your login to access the AI Brand Strategist.' return redirect('authentication:login') # Get the AI Brand Strategist agent @@ -333,10 +335,12 @@ def ai_brand_strategist_view(request): def ai_brand_strategist_access(request): """Handle Try Now button click - charge wallet and redirect to form""" if not request.user.is_authenticated: - # Clear any existing messages to prevent confusion + # Clear all existing messages before adding login message storage = messages.get_messages(request) - storage.used = True - messages.error(request, 'Please login to access the AI Brand Strategist.') + for _ in storage: + pass # Consume all messages + # Add login message to session for after login redirect + request.session['post_login_message'] = 'Please complete your login to access the AI Brand Strategist.' return redirect('authentication:login') # Get the AI Brand Strategist agent @@ -377,11 +381,7 @@ def ai_brand_strategist_access(request): completed_at=timezone.now() ) - # Success message and redirect to form - if agent.price > 0: - messages.success(request, f'Welcome to your {agent.name} consultation.') - else: - messages.success(request, f'Welcome to your {agent.name} consultation.') + # Redirect directly to form - no message needed return redirect('agents:ai_brand_strategist') @@ -1123,13 +1123,11 @@ def direct_access_handler(request, slug): description=f'Payment for {agent.name}', agent_slug=agent.slug ) - messages.success(request, f'Payment of {agent.price} AED processed successfully.') except Exception as e: messages.error(request, 'Payment processing failed. Please try again.') return redirect('agents:agent_detail', slug=slug) - # Grant access - redirect to display page - messages.success(request, f'Access granted to {agent.name}. Redirecting to consultation form...') + # Grant access - redirect directly to display page return redirect('agents:direct_access_display', slug=slug) diff --git a/authentication/views.py b/authentication/views.py index 5b9130b..5f6d8ac 100644 --- a/authentication/views.py +++ b/authentication/views.py @@ -89,10 +89,17 @@ def handle_ratelimited(request, exception): @ratelimit(key='ip', rate='5/m', method=UNSAFE, block=False) def login_view(request): """User login view with rate limiting (5 attempts per minute per IP)""" + # Handle post-login session messages + if 'post_login_message' in request.session: + messages.info(request, request.session.pop('post_login_message')) + # Check if rate limited if getattr(request, 'limited', False): - logger.warning(f"Login rate limit exceeded for IP {request.META.get('REMOTE_ADDR')}") - messages.error(request, 'Too many login attempts. Please try again in a few minutes.') + # Use session to avoid repeated rate limit messages + if not request.session.get('rate_limit_shown'): + logger.warning(f"Login rate limit exceeded for IP {request.META.get('REMOTE_ADDR')}") + messages.error(request, 'Too many login attempts. Please wait before trying again.') + request.session['rate_limit_shown'] = True return render(request, 'authentication/login.html') if request.method == 'POST': @@ -103,12 +110,14 @@ def login_view(request): if user is not None: # Check if email is verified (only if email verification is required) if settings.REQUIRE_EMAIL_VERIFICATION and not user.email_verified: - from django.urls import reverse - resend_url = reverse('authentication:resend_verification') - messages.warning(request, f'Please verify your email address before logging in. Check your inbox for the verification link, or resend verification email.') - return render(request, 'authentication/login.html') + messages.warning(request, 'Please verify your email address before logging in. Check your inbox for the verification link.') + # Store resend URL in context for template + context = {'show_resend_verification': True} + return render(request, 'authentication/login.html', context) login(request, user) + # Clear rate limit flag on successful login + request.session.pop('rate_limit_shown', None) # Redirect to 'next' parameter if provided, otherwise homepage next_url = request.GET.get('next') or request.POST.get('next') if next_url: @@ -116,6 +125,8 @@ def login_view(request): return redirect('core:homepage') else: messages.error(request, 'Invalid email or password') + # Clear rate limit flag on any POST attempt (failed login) + request.session.pop('rate_limit_shown', None) return render(request, 'authentication/login.html') diff --git a/templates/authentication/login.html b/templates/authentication/login.html index ece36ea..34b1670 100644 --- a/templates/authentication/login.html +++ b/templates/authentication/login.html @@ -104,6 +104,37 @@ text-decoration: underline; } + .verification-actions { + text-align: center; + margin-bottom: var(--spacing-lg); + padding: var(--spacing-md); + background: var(--surface-variant); + border-radius: var(--radius-sm); + border: 1px solid var(--outline); + } + + .verification-text { + margin: 0 0 var(--spacing-sm) 0; + color: var(--on-surface-variant); + font-size: 14px; + } + + .btn-outline { + background: transparent; + border: 1px solid var(--primary); + color: var(--primary); + } + + .btn-outline:hover { + background: var(--primary); + color: var(--on-primary); + } + + .btn-sm { + padding: var(--spacing-xs) var(--spacing-md); + font-size: 14px; + } + /* Responsive */ @media (max-width: 480px) { .login-page { @@ -132,11 +163,18 @@ {% if messages %}
{% for message in messages %} -
{{ message|safe }}
+
{{ message }}
{% endfor %}
{% endif %} + {% if show_resend_verification %} +
+

Need to resend verification email?

+ Resend Verification +
+ {% endif %} +