Hide profile overlay for now

i-dot-ai · Sep 18, 2024 · 125cfdf · 125cfdf
1 parent 545263d
commit 125cfdf
Show file tree

Hide file tree

Showing 5 changed files with 32 additions and 16 deletions.
diff --git a/django_app/redbox_app/redbox_core/views/auth_views.py b/django_app/redbox_app/redbox_core/views/auth_views.py
@@ -25,7 +25,7 @@ def sign_in_view(request: HttpRequest):
 
             try:
                 user = models.User.objects.get(email=email)
-                link = MagicLink.objects.create(user=user, redirect_to="/chats")
+                link = MagicLink.objects.create(user=user, redirect_to="/check-demographics") # Switch this to "/chats" once profile overlay is added to Chats page
                 full_link = request.build_absolute_uri(link.get_absolute_url())
                 body = render_to_string("email/verification.txt", {"url": full_link})
                 send_mail(

diff --git a/django_app/redbox_app/redbox_core/views/demographics_views.py b/django_app/redbox_app/redbox_core/views/demographics_views.py
@@ -14,6 +14,17 @@
 logger = logging.getLogger(__name__)
 
 
+# This can be removed once profile overlay added to /chats
+class CheckDemographicsView(View):
+    @method_decorator(login_required)
+    def get(self, request: HttpRequest) -> HttpResponse:
+        user: User = request.user
+        if all([user.name, user.ai_experience, user.grade, user.business_unit, user.profession]):
+            return redirect("chats")
+        else:
+            return redirect("demographics")
+
+
 class DemographicsView(LoginRequiredMixin, UpdateView):
     model = User
     template_name = "demographics.html"

diff --git a/django_app/redbox_app/templates/chats.html b/django_app/redbox_app/templates/chats.html
@@ -6,9 +6,11 @@
 
 {% block content %}
 
+{# To be added back in once profile overlay is required
 {% if not request.user.name %}
   {{ profile_overlay(csrf_token, request.user, True) }}
 {% endif %}
+#}
 
 <div class="govuk-width-container">
 
@@ -28,21 +30,21 @@ <h1 class="govuk-visually-hidden">Chats</h1>
           New chat
         </a>
 
-      <div class="iai-panel govuk-!-margin-top-5 govuk-!-padding-top-3">
-        <h2 class="govuk-body govuk-!-font-weight-bold">Recent chats</h2>
-        <div class="iai-panel__scrollable">
-          {% for date_group, chats in chat_grouped_by_date_group %}
-          <h3 class="rb-chat-history__date_group govuk-body-xs govuk-!-font-weight-bold govuk-!-margin-bottom-1">{{ date_group }}</h3>
-          <ul class="govuk-list govuk-list--spaced">
-            {% for chat in chats %}
-              <li class="govuk-!-margin-bottom-1">
-                <a class="govuk-link govuk-body-s" href="{{url('chats', chat.id)}}">{{ chat.name }}</a>
-              </li>
+        <div class="iai-panel govuk-!-margin-top-5 govuk-!-padding-top-3">
+          <h2 class="govuk-body govuk-!-font-weight-bold">Recent chats</h2>
+          <div class="iai-panel__scrollable">
+            {% for date_group, chats in chat_grouped_by_date_group %}
+            <h3 class="rb-chat-history__date_group govuk-body-xs govuk-!-font-weight-bold govuk-!-margin-bottom-1">{{ date_group }}</h3>
+            <ul class="govuk-list govuk-list--spaced">
+              {% for chat in chats %}
+                <li class="govuk-!-margin-bottom-1">
+                  <a class="govuk-link govuk-body-s" href="{{url('chats', chat.id)}}">{{ chat.name }}</a>
+                </li>
+              {% endfor %}
+            </ul>
             {% endfor %}
-          </ul>
-          {% endfor %}
+          </div>
         </div>
-      </div>
 
         <document-selector class="iai-panel govuk-!-margin-top-5 govuk-!-padding-top-3">
           <fieldset class="govuk-fieldset">

diff --git a/django_app/redbox_app/templates/demographics.html b/django_app/redbox_app/templates/demographics.html
@@ -48,6 +48,7 @@ <h1 class="govuk-heading-l">My details</h1>
         </fieldset>
       </div>
 
+      {# To be added in when profile overlay ready to go live
       <div class="govuk-form-group">
         <label class="govuk-label" for="{{ form.info_about_user.id_for_label }}">
           {{ form.info_about_user.label }}
@@ -61,6 +62,7 @@ <h1 class="govuk-heading-l">My details</h1>
         </label>
         {{ form.redbox_response_preferences }}
       </div>
+      #}
 
       <div class="govuk-button-group">
         {{ govukButton(text="Update") }}

diff --git a/tests/test_journey.py b/tests/test_journey.py
@@ -52,8 +52,9 @@ def test_user_journey(page: Page, email_address: str):
     my_details_page = chats_page.navigate_my_details()
     my_details_page.name = "Roland Hamilton-Jones"
     my_details_page.ai_experience = "Enthusiastic Experimenter"
-    my_details_page.info_about_user = "Information about user"
-    my_details_page.redbox_response_preferences = "Respond concisely"
+    # Add these in once profile overlay is added
+    #my_details_page.info_about_user = "Information about user"
+    #my_details_page.redbox_response_preferences = "Respond concisely"
     chats_page = my_details_page.update()
 
     # Documents page