Remove chatbot link and related access logic from home page (#1714)

* Remove chatbot link and related access logic from home page

Eliminated the chatbot link from the home page template and removed associated access logic from the HomeView. Corresponding tests for chatbot link visibility were also deleted, simplifying the code and UI.

* Remove Chatbot link from trial page

Eliminates the Chatbot navigation link for SaaS deployments from the trial.html template. This may be due to a change in feature availability or access requirements.

Author: mabashian

ansible_ai_connect/main/tests/test_views.py

@@ -318,48 +318,6 @@ def tearDown(self):
         self.non_rh_user_with_subscription.delete()
         Organization.objects.filter(id=12345).delete()
 
-    def test_chatbot_link_with_anonymous_user(self):
-        r = self.client.get(reverse("home"))
-        self.assertEqual(r.status_code, HTTPStatus.OK)
-        self.assertContains(r, TestChatbotView.DOCUMENT_URL)
-        self.assertNotContains(r, "Chatbot")
-
-    def test_chatbot_link_with_non_rh_user_and_subscription(self):
-        self.client.force_login(user=self.non_rh_user_with_subscription)
-        r = self.client.get(reverse("home"))
-        self.assertEqual(r.status_code, HTTPStatus.OK)
-        self.assertContains(r, TestChatbotView.DOCUMENT_URL)
-        self.assertContains(r, "Chatbot")
-
-    def test_chatbot_link_with_non_rh_user(self):
-        self.client.force_login(user=self.non_rh_user)
-        r = self.client.get(reverse("home"))
-        self.assertEqual(r.status_code, HTTPStatus.OK)
-        self.assertContains(r, TestChatbotView.DOCUMENT_URL)
-        self.assertNotContains(r, "Chatbot")
-
-    def test_chatbot_link_with_non_rh_test_user(self):
-        self.client.force_login(user=self.non_rh_test_user)
-        r = self.client.get(reverse("home"))
-        self.assertEqual(r.status_code, HTTPStatus.OK)
-        self.assertContains(r, TestChatbotView.DOCUMENT_URL)
-        self.assertContains(r, "Chatbot")
-
-    @override_settings(CHATBOT_DEFAULT_PROVIDER="")
-    def test_chatbot_link_with_rh_user_but_chatbot_disabled(self):
-        self.client.force_login(user=self.rh_user)
-        r = self.client.get(reverse("home"))
-        self.assertEqual(r.status_code, HTTPStatus.OK)
-        self.assertContains(r, TestChatbotView.DOCUMENT_URL)
-        self.assertNotContains(r, "Chatbot")
-
-    def test_chatbot_link_with_rh_user(self):
-        self.client.force_login(user=self.rh_user)
-        r = self.client.get(reverse("home"))
-        self.assertEqual(r.status_code, HTTPStatus.OK)
-        self.assertContains(r, TestChatbotView.DOCUMENT_URL)
-        self.assertContains(r, "Chatbot")
-
     def test_chatbot_view_with_anonymous_user(self):
         r = self.client.get(reverse("chatbot"))
         self.assertEqual(r.status_code, HTTPStatus.FOUND)

ansible_ai_connect/users/templates/users/home.html

@@ -131,9 +131,6 @@ <h1 class="pf-c-title pf-m-lg">{{ project_name }}</h1>
                   {% if deployment_mode == 'saas' and user.is_authenticated and user.rh_user_is_org_admin %}
                   <a class="pf-l-level__item" href="/console"><span class="fas fa-solid fa-cog"></span> Admin Portal</a>
                   {% endif %}
-                  {% if chatbot_enabled and deployment_mode == 'saas' and user.is_authenticated and can_access_chatbot %}
-                  <a class="pf-l-level__item" href="/chatbot"><span class="fas fa-solid fa-comments"></span> Chatbot</a>
-                  {% endif %}
                 </div>
               </div>
             </div>

ansible_ai_connect/users/templates/users/trial.html

@@ -143,9 +143,6 @@ <h1 class="pf-c-title pf-m-lg">{{ project_name }}</h1>
                   {% if deployment_mode == 'saas' and user.is_authenticated and user.rh_user_is_org_admin %}
                     <a class="pf-l-level__item" href="/console"><span class="fas fa-solid fa-cog"></span> Admin Portal</a>
                   {% endif %}
-                  {% if chatbot_enabled and deployment_mode == 'saas' and user.is_authenticated and rh_internal_user_or_test_user %}
-                    <a class="pf-l-level__item" href="/chatbot"><span class="fas fa-solid fa-comments"></span> Chatbot</a>
-                  {% endif %}
                 </div>
               </div>
             </div>

ansible_ai_connect/users/views.py

@@ -112,13 +112,6 @@ def get_context_data(self, **kwargs):
 
         context["documentation_url"] = settings.COMMERCIAL_DOCUMENTATION_URL
 
-        user = self.request.user
-        context["can_access_chatbot"] = user.is_authenticated and (
-            user.rh_internal
-            or user.groups.filter(name="test").exists()
-            or user.rh_org_has_subscription
-        )
-
         # Show chatbot link when the chatbot service is configured.
         llm: ModelPipelineStreamingChatBot = apps.get_app_config("ai").get_model_pipeline(
             ModelPipelineStreamingChatBot