Show public profile prompt on own leaderboard record

Author: alexeygrigorev

courses/templates/courses/course.html

@@ -24,7 +24,7 @@ <h1 class="text-3xl font-semibold leading-tight tracking-normal md:text-4xl">
   </div>
 
   <div class="course-actions mt-5 flex flex-wrap gap-3 md:mt-8 md:gap-4">
-    {% if course.registration_url %}
+    {% if course.home_registration_open %}
       <a href="{{ course.registration_url }}" class="primer-button primer-button-secondary whitespace-nowrap">
         <i class="fas fa-user-plus"></i> Register
       </a>

courses/templates/courses/course_list.html

@@ -116,7 +116,7 @@ <h3 class="mt-2 text-xl font-semibold md:text-2xl">
                                     <p class="mt-1 font-semibold app-heading">TBA</p>
                                 {% endif %}
                             </div>
-                            {% if course.registration_url %}
+                            {% if course.home_registration_open %}
                                 <div class="flex flex-wrap gap-3 md:justify-end">
                                     <a href="{{ course.registration_url }}"
                                        class="primer-button primer-button-secondary whitespace-nowrap">

courses/templates/courses/leaderboard_score_breakdown.html

@@ -46,6 +46,12 @@ <h1 class="text-2xl font-semibold app-heading md:text-3xl">{{ enrollment.display
           <i class="fas fa-cog"></i> Admin
         </a>
       {% endif %}
+      {% if show_public_profile_settings_link %}
+        <a href="{% url 'enrollment' enrollment.course.slug %}"
+           class="primer-button">
+          Show public profile
+        </a>
+      {% endif %}
       <button type="button"
               class="leaderboard-flag-button primer-button primer-button-secondary"
               data-flag-url="{% url 'leaderboard_complaint' course_slug=enrollment.course.slug enrollment_id=enrollment.id %}"

courses/tests/test_course.py

@@ -169,6 +169,9 @@ def test_course_detail_unauthenticated_user(self):
             self.assertFalse(hasattr(hw, "submitted_at"))
 
     def test_course_detail_shows_registration_url(self):
+        self.course.start_date = timezone.localdate() + timezone.timedelta(
+            days=7
+        )
         self.course.registration_url = (
             "https://courses.datatalks.club/test-course/register"
         )
@@ -491,6 +494,58 @@ def test_score_breakdown_has_flag_button(self):
         self.assertEqual(response.status_code, 200)
         self.assertContains(response, "Flag this record")
 
+    def test_score_breakdown_prompts_owner_to_show_public_profile(self):
+        self.client.force_login(self.user)
+
+        url = reverse(
+            "leaderboard_score_breakdown",
+            kwargs={
+                "course_slug": self.course.slug,
+                "enrollment_id": self.enrollment.id,
+            },
+        )
+        response = self.client.get(url)
+
+        self.assertEqual(response.status_code, 200)
+        self.assertContains(response, "Show public profile")
+        self.assertContains(
+            response,
+            f'href="{reverse("enrollment", args=[self.course.slug])}"',
+        )
+
+    def test_score_breakdown_does_not_prompt_for_other_record(self):
+        target = self.create_enrollment("e1", 100, 1)
+        self.client.force_login(self.user)
+
+        url = reverse(
+            "leaderboard_score_breakdown",
+            kwargs={
+                "course_slug": self.course.slug,
+                "enrollment_id": target.id,
+            },
+        )
+        response = self.client.get(url)
+
+        self.assertEqual(response.status_code, 200)
+        self.assertNotContains(response, "Show public profile")
+
+    def test_score_breakdown_does_not_prompt_when_profile_public(self):
+        self.enrollment.display_public_profile = True
+        self.enrollment.save()
+        self.client.force_login(self.user)
+
+        url = reverse(
+            "leaderboard_score_breakdown",
+            kwargs={
+                "course_slug": self.course.slug,
+                "enrollment_id": self.enrollment.id,
+            },
+        )
+        response = self.client.get(url)
+
+        self.assertEqual(response.status_code, 200)
+        self.assertNotContains(response, "Show public profile")
+
     def test_score_breakdown_shows_score_equations(self):
         self.submission1.questions_score = 7
         self.submission1.faq_score = 2
@@ -1026,7 +1081,7 @@ def test_course_list_shows_active_course_metadata(self):
         self.assertContains(response, "Apr 15, 2026")
         self.assertContains(response, "13 weeks")
         self.assertContains(response, "Submitted Homework")
-        self.assertContains(
+        self.assertNotContains(
             response,
             "https://courses.datatalks.club/test-course/register",
         )

courses/views/course.py

@@ -359,6 +359,7 @@ def update_project_with_additional_info(project: Project) -> None:
 
 def course_view(request: HttpRequest, course_slug: str) -> HttpResponse:
     course = get_object_or_404(Course, slug=course_slug)
+    add_course_homepage_info(course, timezone.now())
 
     user = request.user
     homeworks = get_homeworks_for_course(course, user)
@@ -689,11 +690,20 @@ def leaderboard_score_breakdown_view(
         enrollment=enrollment
     ).order_by("project__id")
 
+    is_own_record = (
+        request.user.is_authenticated
+        and request.user.id == enrollment.student_id
+    )
+    public_profile = (
+        enrollment.student if enrollment.display_public_profile else None
+    )
+
     context = {
         "enrollment": enrollment,
-        "public_profile": enrollment.student
-        if enrollment.display_public_profile
-        else None,
+        "public_profile": public_profile,
+        "show_public_profile_settings_link": (
+            is_own_record and public_profile is None
+        ),
         "submissions": homework_submissions,
         "project_submissions": project_submissions,
     }