From d3ef35c1b384dfc0b9ad73879e15cfe19b733113 Mon Sep 17 00:00:00 2001
From: Sam Bland <sam.bland@york.ac.uk>
Date: Wed, 14 Oct 2026 14:12:20 +0100
Subject: [PATCH 01/13] Add json download button

---
 .../snippets/export_user_skill_profile.html   | 29 +++++++++++++++++--
 .../share_skill_profile_dropdown.html         |  4 +++
 2 files changed, 31 insertions(+), 2 deletions(-)

diff --git a/main/templates/main/snippets/export_user_skill_profile.html b/main/templates/main/snippets/export_user_skill_profile.html
index 0665c0e2..70527de8 100644
--- a/main/templates/main/snippets/export_user_skill_profile.html
+++ b/main/templates/main/snippets/export_user_skill_profile.html
@@ -7,8 +7,8 @@
 
 -->
 <script>
-    const exportSkillProfileButton = document.querySelector('.export-skill-profile');
-    exportSkillProfileButton.addEventListener('click', function () {
+    const exportSkillProfileCsvButton = document.querySelector('.export-skill-profile-csv');
+    exportSkillProfileCsvButton.addEventListener('click', function () {
       // The skill levels and chart data are passed from the Django view
       // We assume we receive the same data as the skill wheel.
       const skillLevels = {{ skill_levels|safe }};
@@ -30,4 +30,29 @@
       link.click();
       document.body.removeChild(link);
     });
+    const exportSkillProfileJSONButton = document.querySelector('.export-skill-profile-json');
+    exportSkillProfileJSONButton.addEventListener('click', function () {
+      // The skill levels and chart data are passed from the Django view
+      // We assume we receive the same data as the skill wheel.
+      const skillLevels = {{ skill_levels|safe }};
+      const charts = {{ chart_data|safe }};
+
+      const jsonData = charts.map(({user_data, user_id}) => {
+        return user_data.map(({category, skill, skill_level, subcategory}) => {
+          return {
+            category,
+            subcategory,
+            skill,
+            level: skill_level
+          };
+        });
+      }).flat();
+      const jsonContent = "data:text/json;charset=utf-8," + encodeURIComponent(JSON.stringify(jsonData, null, 2));
+      const link = document.createElement("a");
+      link.setAttribute("href", encodedUri);
+      link.setAttribute("download", "skill_profile.json");
+      document.body.appendChild(link);
+      link.click();
+      document.body.removeChild(link);
+    });
 </script>
diff --git a/main/templates/main/snippets/share_skill_profile_dropdown.html b/main/templates/main/snippets/share_skill_profile_dropdown.html
index 77effa1d..b5c6249e 100644
--- a/main/templates/main/snippets/share_skill_profile_dropdown.html
+++ b/main/templates/main/snippets/share_skill_profile_dropdown.html
@@ -29,6 +29,10 @@
                         href="#">Export as CSV</button>
                 {% include "main/snippets/export_user_skill_profile.html" %}
             </li>
+            <li>
+                <button class="dropdown-item export-skill-profile-json" href="#">Export as JSON</button>
+            </li>
+            {% include "main/snippets/export_user_skill_profile.html" %}
         </ul>
     </li>
 </div>

From 6c3f13d23371d152625e4d8595f06f61950b8600 Mon Sep 17 00:00:00 2001
From: Sam Bland <sam.bland@york.ac.uk>
Date: Wed, 14 Oct 2026 14:12:21 +0100
Subject: [PATCH 02/13] Update skill share inline docs

---
 .../main/snippets/export_user_skill_profile.html          | 8 ++++----
 .../main/snippets/share_skill_profile_dropdown.html       | 7 ++++++-
 2 files changed, 10 insertions(+), 5 deletions(-)

diff --git a/main/templates/main/snippets/export_user_skill_profile.html b/main/templates/main/snippets/export_user_skill_profile.html
index 70527de8..5dc95212 100644
--- a/main/templates/main/snippets/export_user_skill_profile.html
+++ b/main/templates/main/snippets/export_user_skill_profile.html
@@ -1,9 +1,9 @@
-<!-- This template implements an export button to export the current user's skill profile as a CSV file.
- It works on the skill profile page.
+<!-- This template implements an export button to export the current user's skill profile as a CSV or json file.
 
 
-To use add a button with the class "export-skill-profile" to the page and include the following script at the end of the page.
-"{ % include "main/snippets/export_user_skill_profile.html" %}"
+To use add a button with the class `export-skill-profile-csv` and `export-skill-profile-json`
+ to the page and include the following script at the end of the page.
+`{ % include "main/snippets/export_user_skill_profile.html" %}`
 
 -->
 <script>
diff --git a/main/templates/main/snippets/share_skill_profile_dropdown.html b/main/templates/main/snippets/share_skill_profile_dropdown.html
index b5c6249e..5387ae9b 100644
--- a/main/templates/main/snippets/share_skill_profile_dropdown.html
+++ b/main/templates/main/snippets/share_skill_profile_dropdown.html
@@ -1,4 +1,9 @@
-<!-- Share dropdown -->
+<!-- Share dropdown
+
+This template implements a share dropdown menu for the skill profile page.
+It includes options to open a shareable link, copy the link, and export the skill profile as CSV or JSON.
+
+-->
 <div class="dropdown nav d-none d-sm-block order-lg-3 justify-content-end">
     <li class="nav-item d-flex align-items-center justify-content-end">
         <a class="nav-link dropdown-toggle justify-content-end"

From 643bd14b7d93c53f0739c23dbaf3ecfcafeecd3c Mon Sep 17 00:00:00 2001
From: Sam Bland <sam.bland@york.ac.uk>
Date: Wed, 14 Oct 2026 14:12:21 +0100
Subject: [PATCH 03/13] Split export skills js

---
 main/templates/main/snippets/export_user_skill_profile.html | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/main/templates/main/snippets/export_user_skill_profile.html b/main/templates/main/snippets/export_user_skill_profile.html
index 5dc95212..d3b724ef 100644
--- a/main/templates/main/snippets/export_user_skill_profile.html
+++ b/main/templates/main/snippets/export_user_skill_profile.html
@@ -6,7 +6,7 @@
 `{ % include "main/snippets/export_user_skill_profile.html" %}`
 
 -->
-<script>
+<script id="export-skills-data-csv">
     const exportSkillProfileCsvButton = document.querySelector('.export-skill-profile-csv');
     exportSkillProfileCsvButton.addEventListener('click', function () {
       // The skill levels and chart data are passed from the Django view
@@ -30,6 +30,8 @@
       link.click();
       document.body.removeChild(link);
     });
+</script>
+<script id="export-skills-data-json">
     const exportSkillProfileJSONButton = document.querySelector('.export-skill-profile-json');
     exportSkillProfileJSONButton.addEventListener('click', function () {
       // The skill levels and chart data are passed from the Django view

From 6e43ec3554c9ebbd09c1dd840095cb912b74d542 Mon Sep 17 00:00:00 2001
From: Sam Bland <sam.bland@york.ac.uk>
Date: Wed, 14 Oct 2026 14:12:57 +0100
Subject: [PATCH 04/13] Add client logged in fixture

---
 tests/main/conftest.py | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/tests/main/conftest.py b/tests/main/conftest.py
index dfca97b4..ddd0e7d3 100644
--- a/tests/main/conftest.py
+++ b/tests/main/conftest.py
@@ -119,3 +119,9 @@ def event() -> Event:
         blog="https://example.com/blog",
         contributors="Test Contributor",
     )
+
+@pytest.fixture
+def client_logged_in(client, user):
+    """Fixture for a client that is logged in as a user."""
+    client.force_login(user)
+    return client

From 1ccfae5ac50a85de2cf7286c4553ea3a88d15e51 Mon Sep 17 00:00:00 2001
From: Sam Bland <sam.bland@york.ac.uk>
Date: Wed, 14 Oct 2026 14:12:59 +0100
Subject: [PATCH 05/13] Add download user skills data api endpoints

Added the csv and json download data views that can be accessed from:
- http://localhost:8000/accounts/download-skills-data/csv
- http://localhost:8000/accounts/download-skills-data/json

Also included an optional parameter to the json download to export anonymous json
---
 main/urls.py                  | 10 +++++
 main/views/data_views.py      | 84 ++++++++++++++++++++++++++++++++++-
 tests/main/test_data_views.py | 78 ++++++++++++++++++++++++++++++++
 3 files changed, 171 insertions(+), 1 deletion(-)

diff --git a/main/urls.py b/main/urls.py
index 1ffca584..6b0560a3 100644
--- a/main/urls.py
+++ b/main/urls.py
@@ -46,6 +46,16 @@
     path("skills-profile/", views.SkillProfileView.as_view(), name="skills_profile"),
     path("profile/", views.UserUpdateView.as_view(), name="profile"),
     path("overview/", views.AccountOverviewView.as_view(), name="account-overview"),
+    path(
+        "download-skills-data/csv/",
+        views.DownloadUserSkillDataCsvView.as_view(),
+        name="download-csv-data-link",
+    ),
+    path(
+        "download-skills-data/json/",
+        views.DownloadUserSkillDataJSONView.as_view(),
+        name="download-json-data-link",
+    ),
 ]
 
 policies_patterns = [
diff --git a/main/views/data_views.py b/main/views/data_views.py
index 85d03ecd..4faa5b4e 100644
--- a/main/views/data_views.py
+++ b/main/views/data_views.py
@@ -1,9 +1,12 @@
 """Views for getting data from the framework database."""
 
-from django.http import HttpRequest, JsonResponse
+import csv
+
+from django.http import HttpRequest, HttpResponse, JsonResponse
 from django.views import View
 
 from ..io_resources import export_framework
+from ..models import UserSkill
 
 
 class FrameworkView(View):
@@ -23,3 +26,82 @@ def get(self, request: HttpRequest) -> JsonResponse:
                 - skill_levels: A list of the Skill Levels
         """
         return JsonResponse(export_framework(), json_dumps_params=dict(indent=2))
+
+
+class DownloadUserSkillDataCsvView(View):
+    """A view that returns the user skill data as a CSV file."""
+
+    def get(self, request: HttpRequest) -> HttpResponse:
+        """Define the GET response.
+
+        Args:
+            request: A GET request from the user.
+
+        Returns:
+            A HttpResponse containing the user skill data in CSV format.
+        """
+        user_skills = UserSkill.objects.filter(user=request.user.pk)
+        user_skills_data = [
+            {
+                "skill": user_skill.skill.name,
+                "category": user_skill.skill.competency.competency_domain.name,
+                "subcategory": user_skill.skill.competency.name,
+                "skill_level": user_skill.skill_level.level,
+            }
+            for user_skill in user_skills
+        ]
+        context = {}
+        context["chart_data"] = [
+            {
+                "user_id": "root",
+                "user_data": user_skills_data,
+            }
+        ]
+
+        response = HttpResponse(
+            content_type="text/csv",
+            headers={"Content-Disposition": 'attachment; filename="downloaded_.csv"'},
+        )
+        writer = csv.writer(response)
+        writer.writerow(["skill", "category", "subcategory", "skill_level"])
+        for user_skill in user_skills_data:
+            writer.writerow(
+                [
+                    user_skill["skill"],
+                    user_skill["category"],
+                    user_skill["subcategory"],
+                    user_skill["skill_level"],
+                ]
+            )
+        return response
+
+
+class DownloadUserSkillDataJSONView(View):
+    """A view that returns the user skill data as a JSON file."""
+
+    def get(self, request: HttpRequest) -> JsonResponse:
+        """Define the GET response.
+
+        Args:
+            request: A GET request from the user.
+
+        Returns:
+            A JsonResponse containing the user skill data in JSON format.
+        """
+        user_skills = UserSkill.objects.filter(user=request.user.pk)
+        # get_anonymous url param
+        get_anonymous = request.GET.get("anonymous", "false").lower() == "true"
+        user_skills_data = [
+            {
+                "skill": user_skill.skill.name,
+                "category": user_skill.skill.competency.competency_domain.name,
+                "subcategory": user_skill.skill.competency.name,
+                "skill_level": user_skill.skill_level.level,
+            }
+            for user_skill in user_skills
+        ]
+        returned_data = {
+            "username": "anonymous" if get_anonymous else request.user.username,
+            "user_data": user_skills_data,
+        }
+        return JsonResponse(returned_data, json_dumps_params=dict(indent=2))
diff --git a/tests/main/test_data_views.py b/tests/main/test_data_views.py
index d4823396..9a641108 100644
--- a/tests/main/test_data_views.py
+++ b/tests/main/test_data_views.py
@@ -39,3 +39,81 @@ def test_framework_view_can_only_get(self, client):
         assert response.status_code == HTTPStatus.METHOD_NOT_ALLOWED
         response = client.trace(self._get_url())
         assert response.status_code == HTTPStatus.METHOD_NOT_ALLOWED
+
+
+class TestDownloadCSVDataLinkView:
+    """Test suite for the DownloadCSVDataLinkView."""
+
+    def _get_url(self):
+        return reverse("download-csv-data-link")
+
+    def test_download_csv_data_link_view_get(self, client_logged_in):
+        """Test the GET returns the correct CSV data link."""
+        response = client_logged_in.get(self._get_url())
+        assert response.status_code == HTTPStatus.OK
+
+    def test_download_csv_data_link_view_returns_a_csv_file(self, client_logged_in):
+        """Test that the GET request returns a CSV file."""
+        response = client_logged_in.get(self._get_url())
+        assert response.status_code == HTTPStatus.OK
+        assert response["Content-Type"] == "text/csv"
+
+    def test_download_csv_data_link_view_csv_content(
+        self, client_logged_in, user_skill
+    ):
+        """Test that the CSV content is correct."""
+        response = client_logged_in.get(self._get_url())
+        assert response.status_code == HTTPStatus.OK
+        content = response.content.decode("utf-8")
+        assert content.startswith("skill")
+        assert "category" in content
+        assert "subcategory" in content
+        assert "skill_level" in content
+        assert user_skill.skill.name in content
+        assert (
+            content.count("\n") > 1
+        )  # Ensure there is at least one row of data besides the header
+
+
+class TestDownloadUserSkillDataJSONView:
+    """Test suite for the DownloadUserSkillDataJSONView."""
+
+    def _get_url(self):
+        return reverse("download-json-data-link")
+
+    def test_download_json_data_link_view_get(self, client_logged_in):
+        """Test the GET returns the correct JSON data link."""
+        response = client_logged_in.get(self._get_url())
+        assert response.status_code == HTTPStatus.OK
+
+    def test_download_json_data_link_view_returns_a_json_file(self, client_logged_in):
+        """Test that the GET request returns a JSON file."""
+        response = client_logged_in.get(self._get_url())
+        assert response.status_code == HTTPStatus.OK
+        assert response["Content-Type"] == "application/json"
+
+    def test_download_json_data_link_view_json_content(self, client_logged_in):
+        """Test that the JSON content is correct."""
+        response = client_logged_in.get(self._get_url())
+        assert response.status_code == HTTPStatus.OK
+        content = response.json()
+        assert isinstance(content, dict)
+        assert "username" in content
+        assert "user_data" in content
+        assert isinstance(content["user_data"], list)
+        if content["user_data"]:
+            assert "skill" in content["user_data"][0]
+            assert "category" in content["user_data"][0]
+            assert "subcategory" in content["user_data"][0]
+            assert "skill_level" in content["user_data"][0]
+
+    def test_returns_anonymous_username_when_anonymous_param_is_true(
+        self, client_logged_in
+    ):
+        """Assert username is 'anonymous' when anonymous URL para is set to true."""
+        response = client_logged_in.get(self._get_url() + "?anonymous=true")
+        assert response.status_code == HTTPStatus.OK
+        content = response.json()
+        assert content["username"] == "anonymous"
+        # Also check that the user_data is still present
+        assert "user_data" in content

From 29b6b9544d65c741605be9932ca7094ca2a0896d Mon Sep 17 00:00:00 2001
From: Sam Bland <sam.bland@york.ac.uk>
Date: Wed, 14 Oct 2026 14:13:28 +0100
Subject: [PATCH 06/13] Update download json and csv links in skill share
 dropdown

Also made this togglable as they are no longer valid for the shared skills data view
---
 .../share_skill_profile_dropdown.html         | 24 ++++++++++---------
 main/views/account_views.py                   |  4 ++++
 2 files changed, 17 insertions(+), 11 deletions(-)

diff --git a/main/templates/main/snippets/share_skill_profile_dropdown.html b/main/templates/main/snippets/share_skill_profile_dropdown.html
index 5387ae9b..45493203 100644
--- a/main/templates/main/snippets/share_skill_profile_dropdown.html
+++ b/main/templates/main/snippets/share_skill_profile_dropdown.html
@@ -26,18 +26,20 @@
                     Copy Link <i aria-label="Copy sharable link"
     class="fa-solid fa-copy"></i>
                 </button>
-                {% include "main/snippets/create_user_skill_profile_link.html" %}
             </li>
-            <li>
-                <button type="button"
-                        class="dropdown-item export-skill-profile"
-                        href="#">Export as CSV</button>
-                {% include "main/snippets/export_user_skill_profile.html" %}
-            </li>
-            <li>
-                <button class="dropdown-item export-skill-profile-json" href="#">Export as JSON</button>
-            </li>
-            {% include "main/snippets/export_user_skill_profile.html" %}
+            {% include "main/snippets/create_user_skill_profile_link.html" %}
+            {% if allow_export %}
+                <li>
+                    <a class="dropdown-item export-skill-profile-csv"
+                       href="{% url 'download-csv-data-link' %}"
+                       id="copy-link-button">Export as CSV</a>
+                </li>
+                <li>
+                    <a class="dropdown-item export-skill-profile-json"
+                       href="{% url 'download-json-data-link' %}"
+                       id="copy-link-button">Export as JSON</a>
+                </li>
+            {% endif %}
         </ul>
     </li>
 </div>
diff --git a/main/views/account_views.py b/main/views/account_views.py
index fa26b89e..706413bc 100644
--- a/main/views/account_views.py
+++ b/main/views/account_views.py
@@ -104,6 +104,10 @@ def get_context_data(self, **kwargs: Any) -> dict[str, Any]:
         context["skill_levels"] = dumps(
             list(SkillLevel.objects.values("level", "name"))
         )
+
+        context["allow_export"] = (
+            True  # Allow exporting skill profile data as CSV or JSON
+        )
         return context
 
 

From 0c796d3e9998ae572b1ae5b1477b30d6828b938d Mon Sep 17 00:00:00 2001
From: Sam Bland <sam.bland@york.ac.uk>
Date: Wed, 14 Oct 2026 14:13:34 +0100
Subject: [PATCH 07/13] Fix exported data field names

Changed from
category - competency_domain
subcategory - competency
---
 main/views/data_views.py      | 14 +++++++-------
 tests/main/test_data_views.py |  8 ++++----
 2 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/main/views/data_views.py b/main/views/data_views.py
index 4faa5b4e..a1d9ea1e 100644
--- a/main/views/data_views.py
+++ b/main/views/data_views.py
@@ -44,8 +44,8 @@ def get(self, request: HttpRequest) -> HttpResponse:
         user_skills_data = [
             {
                 "skill": user_skill.skill.name,
-                "category": user_skill.skill.competency.competency_domain.name,
-                "subcategory": user_skill.skill.competency.name,
+                "competency_domain": user_skill.skill.competency.competency_domain.name,
+                "competency": user_skill.skill.competency.name,
                 "skill_level": user_skill.skill_level.level,
             }
             for user_skill in user_skills
@@ -63,13 +63,13 @@ def get(self, request: HttpRequest) -> HttpResponse:
             headers={"Content-Disposition": 'attachment; filename="downloaded_.csv"'},
         )
         writer = csv.writer(response)
-        writer.writerow(["skill", "category", "subcategory", "skill_level"])
+        writer.writerow(["skill", "competency_domain", "competency", "skill_level"])
         for user_skill in user_skills_data:
             writer.writerow(
                 [
                     user_skill["skill"],
-                    user_skill["category"],
-                    user_skill["subcategory"],
+                    user_skill["competency_domain"],
+                    user_skill["competency"],
                     user_skill["skill_level"],
                 ]
             )
@@ -94,8 +94,8 @@ def get(self, request: HttpRequest) -> JsonResponse:
         user_skills_data = [
             {
                 "skill": user_skill.skill.name,
-                "category": user_skill.skill.competency.competency_domain.name,
-                "subcategory": user_skill.skill.competency.name,
+                "competency_domain": user_skill.skill.competency.competency_domain.name,
+                "competency": user_skill.skill.competency.name,
                 "skill_level": user_skill.skill_level.level,
             }
             for user_skill in user_skills
diff --git a/tests/main/test_data_views.py b/tests/main/test_data_views.py
index 9a641108..6f101228 100644
--- a/tests/main/test_data_views.py
+++ b/tests/main/test_data_views.py
@@ -66,8 +66,8 @@ def test_download_csv_data_link_view_csv_content(
         assert response.status_code == HTTPStatus.OK
         content = response.content.decode("utf-8")
         assert content.startswith("skill")
-        assert "category" in content
-        assert "subcategory" in content
+        assert "competency_domain" in content
+        assert "competency" in content
         assert "skill_level" in content
         assert user_skill.skill.name in content
         assert (
@@ -103,8 +103,8 @@ def test_download_json_data_link_view_json_content(self, client_logged_in):
         assert isinstance(content["user_data"], list)
         if content["user_data"]:
             assert "skill" in content["user_data"][0]
-            assert "category" in content["user_data"][0]
-            assert "subcategory" in content["user_data"][0]
+            assert "competency_domain" in content["user_data"][0]
+            assert "competency" in content["user_data"][0]
             assert "skill_level" in content["user_data"][0]
 
     def test_returns_anonymous_username_when_anonymous_param_is_true(

From cc3824c3f8cf5a8464b6dd18e6c793cfa3f1d6a0 Mon Sep 17 00:00:00 2001
From: Sam Bland <sam.bland@york.ac.uk>
Date: Wed, 14 Oct 2026 14:13:34 +0100
Subject: [PATCH 08/13] Moved account data views to separate files and
 implemented mixins

---
 main/views/__init__.py                |   1 +
 main/views/account_data_views.py      | 129 ++++++++++++++++++++++++++
 main/views/data_views.py              |  84 +----------------
 tests/main/test_account_data_views.py | 100 ++++++++++++++++++++
 tests/main/test_data_views.py         |  78 ----------------
 5 files changed, 231 insertions(+), 161 deletions(-)
 create mode 100644 main/views/account_data_views.py
 create mode 100644 tests/main/test_account_data_views.py

diff --git a/main/views/__init__.py b/main/views/__init__.py
index 751bc2f0..d67fe4ae 100644
--- a/main/views/__init__.py
+++ b/main/views/__init__.py
@@ -1,5 +1,6 @@
 """Init in views."""
 
+from .account_data_views import *  # noqa:  F403
 from .account_views import *  # noqa:  F403
 from .data_views import *  # noqa:  F403
 from .page_views import *  # noqa:  F403
diff --git a/main/views/account_data_views.py b/main/views/account_data_views.py
new file mode 100644
index 00000000..f1bdb97b
--- /dev/null
+++ b/main/views/account_data_views.py
@@ -0,0 +1,129 @@
+"""Views for getting account-related data from the framework database."""
+
+import csv
+import logging
+from typing import TYPE_CHECKING, Any
+
+from django.http import (
+    HttpRequest,
+    HttpResponse,
+    HttpResponseForbidden,
+    JsonResponse,
+)
+from django.http.response import HttpResponseBase
+from django.views import View
+
+from ..models import User, UserSkill
+
+logger = logging.getLogger(__name__)
+
+
+if TYPE_CHECKING:  # pragma: no cover
+    from ..models import User as UserType
+
+
+class _AuthenticatedHttpRequest(HttpRequest):
+    """Custom HttpRequest type for authenticated users."""
+
+    user: "UserType"
+
+
+class _TermsAcceptedMixin:
+    """Require authenticated users to accept terms before accessing endpoints."""
+
+    terms_acceptance_url_name = "terms_acceptance"
+
+    def dispatch(
+        self, request: HttpRequest, *args: Any, **kwargs: Any
+    ) -> HttpResponseBase:
+        """Redirect to terms acceptance page when the user has not accepted terms."""
+        if not request.user.is_authenticated:
+            return HttpResponseForbidden(
+                "User must be authenticated to access this resource."
+            )
+
+        if not request.user.agreed_to_tos:
+            return HttpResponseForbidden(
+                "User must accept the terms of service to access this resource."
+            )
+        return super().dispatch(request, *args, **kwargs)  # type: ignore
+
+
+class AuthenticatedDataView(_TermsAcceptedMixin, View):
+    """A base view that ensures the user is authenticated formats user data."""
+
+    request: _AuthenticatedHttpRequest
+
+
+class DownloadUserSkillDataMixin:
+    """Mixin to provide functionality for downloading user skill data."""
+
+    def _format_user_skill_data_for_download(
+        self, user: User
+    ) -> list[dict[str, str | int]]:
+        user_skills = UserSkill.objects.filter(user=user.pk)
+        return [
+            {
+                "skill": user_skill.skill.name,
+                "competency_domain": user_skill.skill.competency.competency_domain.name,
+                "competency": user_skill.skill.competency.name,
+                "skill_level": user_skill.skill_level.level,
+            }
+            for user_skill in user_skills
+        ]
+
+
+class DownloadUserSkillDataCsvView(AuthenticatedDataView, DownloadUserSkillDataMixin):
+    """A view that returns the user skill data as a CSV file."""
+
+    def get(self, request: _AuthenticatedHttpRequest) -> HttpResponse:
+        """Handle GET request to download user skill data as CSV.
+
+        Args:
+            request: The HTTP request object.
+
+        Returns:
+            An HttpResponse containing the user skill data as a CSV file
+        """
+        user_skills_data = self._format_user_skill_data_for_download(request.user)
+        response = HttpResponse(
+            content_type="text/csv",
+            headers={"Content-Disposition": 'attachment; filename="downloaded_.csv"'},
+        )
+        writer = csv.writer(response)
+        writer.writerow(["skill", "competency_domain", "competency", "skill_level"])
+        for user_skill in user_skills_data:
+            writer.writerow(
+                [
+                    user_skill["skill"],
+                    user_skill["competency_domain"],
+                    user_skill["competency"],
+                    user_skill["skill_level"],
+                ]
+            )
+        return response
+
+
+class DownloadUserSkillDataJSONView(AuthenticatedDataView, DownloadUserSkillDataMixin):
+    """A view that returns the user skill data as a JSON file."""
+
+    def get(self, request: _AuthenticatedHttpRequest) -> JsonResponse:
+        """Define the GET response.
+
+        Args:
+            request: The HTTP request object.
+
+        Returns:
+            A JsonResponse containing the user skill data in JSON format.
+        """
+        user_skills_data = self._format_user_skill_data_for_download(request.user)
+        get_anonymous = request.GET.get("anonymous", "false").lower() == "true"
+        returned_data = {
+            "username": "anonymous" if get_anonymous else request.user.username,
+            "user_data": user_skills_data,
+        }
+        return JsonResponse(
+            returned_data,
+            json_dumps_params=dict(indent=2),
+            headers={"Content-Disposition": 'attachment; filename="downloaded_.json"'},
+        )
diff --git a/main/views/data_views.py b/main/views/data_views.py
index a1d9ea1e..85d03ecd 100644
--- a/main/views/data_views.py
+++ b/main/views/data_views.py
@@ -1,12 +1,9 @@
 """Views for getting data from the framework database."""
 
-import csv
-
-from django.http import HttpRequest, HttpResponse, JsonResponse
+from django.http import HttpRequest, JsonResponse
 from django.views import View
 
 from ..io_resources import export_framework
-from ..models import UserSkill
 
 
 class FrameworkView(View):
@@ -26,82 +23,3 @@ def get(self, request: HttpRequest) -> JsonResponse:
                 - skill_levels: A list of the Skill Levels
         """
         return JsonResponse(export_framework(), json_dumps_params=dict(indent=2))
-
-
-class DownloadUserSkillDataCsvView(View):
-    """A view that returns the user skill data as a CSV file."""
-
-    def get(self, request: HttpRequest) -> HttpResponse:
-        """Define the GET response.
-
-        Args:
-            request: A GET request from the user.
-
-        Returns:
-            A HttpResponse containing the user skill data in CSV format.
-        """
-        user_skills = UserSkill.objects.filter(user=request.user.pk)
-        user_skills_data = [
-            {
-                "skill": user_skill.skill.name,
-                "competency_domain": user_skill.skill.competency.competency_domain.name,
-                "competency": user_skill.skill.competency.name,
-                "skill_level": user_skill.skill_level.level,
-            }
-            for user_skill in user_skills
-        ]
-        context = {}
-        context["chart_data"] = [
-            {
-                "user_id": "root",
-                "user_data": user_skills_data,
-            }
-        ]
-
-        response = HttpResponse(
-            content_type="text/csv",
-            headers={"Content-Disposition": 'attachment; filename="downloaded_.csv"'},
-        )
-        writer = csv.writer(response)
-        writer.writerow(["skill", "competency_domain", "competency", "skill_level"])
-        for user_skill in user_skills_data:
-            writer.writerow(
-                [
-                    user_skill["skill"],
-                    user_skill["competency_domain"],
-                    user_skill["competency"],
-                    user_skill["skill_level"],
-                ]
-            )
-        return response
-
-
-class DownloadUserSkillDataJSONView(View):
-    """A view that returns the user skill data as a JSON file."""
-
-    def get(self, request: HttpRequest) -> JsonResponse:
-        """Define the GET response.
-
-        Args:
-            request: A GET request from the user.
-
-        Returns:
-            A JsonResponse containing the user skill data in JSON format.
-        """
-        user_skills = UserSkill.objects.filter(user=request.user.pk)
-        # get_anonymous url param
-        get_anonymous = request.GET.get("anonymous", "false").lower() == "true"
-        user_skills_data = [
-            {
-                "skill": user_skill.skill.name,
-                "competency_domain": user_skill.skill.competency.competency_domain.name,
-                "competency": user_skill.skill.competency.name,
-                "skill_level": user_skill.skill_level.level,
-            }
-            for user_skill in user_skills
-        ]
-        returned_data = {
-            "username": "anonymous" if get_anonymous else request.user.username,
-            "user_data": user_skills_data,
-        }
-        return JsonResponse(returned_data, json_dumps_params=dict(indent=2))
diff --git a/tests/main/test_account_data_views.py b/tests/main/test_account_data_views.py
new file mode 100644
index 00000000..9dbb0fc2
--- /dev/null
+++ b/tests/main/test_account_data_views.py
@@ -0,0 +1,100 @@
+"""Test suite for the account-related data views.
+
+This test module includes tests for data views of the app.
+"""
+
+from http import HTTPStatus
+
+from django.urls import reverse
+
+
+class ViewRequiresAuthenticationMixin:
+    """Mixin to test that views require authentication."""
+
+    def test_unauthenticated_user_gets_403(self, client):
+        """Test that an unauthenticated user receives a 403 Forbidden response."""
+        response = client.get(self._get_url())
+        assert response.status_code == HTTPStatus.FORBIDDEN
+
+
+class DownloadUserSkillDataMixin(ViewRequiresAuthenticationMixin):
+    """Test suite for the GetUserSkillsDataMixin."""
+
+    _url: str
+    _content_type: str
+
+    def test_authenticated_user_can_access(self, client_logged_in):
+        """Test that an authenticated user can access the view."""
+        response = client_logged_in.get(self._get_url())
+        assert response.status_code == HTTPStatus.OK
+
+    def test_returns_correct_content_type(self, client_logged_in):
+        """Test that the response returns the correct content type."""
+        response = client_logged_in.get(self._get_url())
+        assert response.status_code == HTTPStatus.OK
+        assert response["Content-Type"] == self._content_type
+
+    def test_return_type_should_be_file_attachement(self, client_logged_in):
+        """Test that the response has the correct file attachment header."""
+        response = client_logged_in.get(self._get_url())
+        assert response.status_code == HTTPStatus.OK
+        content_disposition = response["Content-Disposition"]
+        assert content_disposition.startswith("attachment;")
+
+    def _get_url(self):
+        return reverse(self._url)
+
+
+class TestDownloadUserSkillDataCsvView(DownloadUserSkillDataMixin):
+    """Test suite for the DownloadCSVDataLinkView."""
+
+    _url = "download-csv-data-link"
+    _content_type = "text/csv"
+
+    def test_download_csv_data_link_view_csv_content(
+        self, client_logged_in, user_skill
+    ):
+        """Test that the CSV content is correct."""
+        response = client_logged_in.get(self._get_url())
+        content = response.content.decode("utf-8")
+        assert content.startswith("skill")
+        assert "competency_domain" in content
+        assert "competency" in content
+        assert "skill_level" in content
+        assert user_skill.skill.name in content
+        assert (
+            content.count("\n") > 1
+        )  # Ensure there is at least one row of data besides the header
+
+
+class TestDownloadUserSkillDataJSONView(DownloadUserSkillDataMixin):
+    """Test suite for the DownloadUserSkillDataJSONView."""
+
+    _url = "download-json-data-link"
+    _content_type = "application/json"
+
+    def test_download_json_data_link_view_json_content(self, client_logged_in):
+        """Test that the JSON content is correct."""
+        response = client_logged_in.get(self._get_url())
+        assert response.status_code == HTTPStatus.OK
+        content = response.json()
+        assert isinstance(content, dict)
+        assert "username" in content
+        assert "user_data" in content
+        assert isinstance(content["user_data"], list)
+        if content["user_data"]:
+            assert "skill" in content["user_data"][0]
+            assert "competency_domain" in content["user_data"][0]
+            assert "competency" in content["user_data"][0]
+            assert "skill_level" in content["user_data"][0]
+
+    def test_returns_anonymous_username_when_anonymous_param_is_true(
+        self, client_logged_in
+    ):
+        """Assert username is 'anonymous' when anonymous URL para is set to true."""
+        response = client_logged_in.get(self._get_url() + "?anonymous=true")
+        assert response.status_code == HTTPStatus.OK
+        content = response.json()
+        assert content["username"] == "anonymous"
+        # Also check that the user_data is still present
+        assert "user_data" in content
diff --git a/tests/main/test_data_views.py b/tests/main/test_data_views.py
index 6f101228..d4823396 100644
--- a/tests/main/test_data_views.py
+++ b/tests/main/test_data_views.py
@@ -39,81 +39,3 @@ def test_framework_view_can_only_get(self, client):
         assert response.status_code == HTTPStatus.METHOD_NOT_ALLOWED
         response = client.trace(self._get_url())
         assert response.status_code == HTTPStatus.METHOD_NOT_ALLOWED
-
-
-class TestDownloadCSVDataLinkView:
-    """Test suite for the DownloadCSVDataLinkView."""
-
-    def _get_url(self):
-        return reverse("download-csv-data-link")
-
-    def test_download_csv_data_link_view_get(self, client_logged_in):
-        """Test the GET returns the correct CSV data link."""
-        response = client_logged_in.get(self._get_url())
-        assert response.status_code == HTTPStatus.OK
-
-    def test_download_csv_data_link_view_returns_a_csv_file(self, client_logged_in):
-        """Test that the GET request returns a CSV file."""
-        response = client_logged_in.get(self._get_url())
-        assert response.status_code == HTTPStatus.OK
-        assert response["Content-Type"] == "text/csv"
-
-    def test_download_csv_data_link_view_csv_content(
-        self, client_logged_in, user_skill
-    ):
-        """Test that the CSV content is correct."""
-        response = client_logged_in.get(self._get_url())
-        assert response.status_code == HTTPStatus.OK
-        content = response.content.decode("utf-8")
-        assert content.startswith("skill")
-        assert "competency_domain" in content
-        assert "competency" in content
-        assert "skill_level" in content
-        assert user_skill.skill.name in content
-        assert (
-            content.count("\n") > 1
-        )  # Ensure there is at least one row of data besides the header
-
-
-class TestDownloadUserSkillDataJSONView:
-    """Test suite for the DownloadUserSkillDataJSONView."""
-
-    def _get_url(self):
-        return reverse("download-json-data-link")
-
-    def test_download_json_data_link_view_get(self, client_logged_in):
-        """Test the GET returns the correct JSON data link."""
-        response = client_logged_in.get(self._get_url())
-        assert response.status_code == HTTPStatus.OK
-
-    def test_download_json_data_link_view_returns_a_json_file(self, client_logged_in):
-        """Test that the GET request returns a JSON file."""
-        response = client_logged_in.get(self._get_url())
-        assert response.status_code == HTTPStatus.OK
-        assert response["Content-Type"] == "application/json"
-
-    def test_download_json_data_link_view_json_content(self, client_logged_in):
-        """Test that the JSON content is correct."""
-        response = client_logged_in.get(self._get_url())
-        assert response.status_code == HTTPStatus.OK
-        content = response.json()
-        assert isinstance(content, dict)
-        assert "username" in content
-        assert "user_data" in content
-        assert isinstance(content["user_data"], list)
-        if content["user_data"]:
-            assert "skill" in content["user_data"][0]
-            assert "competency_domain" in content["user_data"][0]
-            assert "competency" in content["user_data"][0]
-            assert "skill_level" in content["user_data"][0]
-
-    def test_returns_anonymous_username_when_anonymous_param_is_true(
-        self, client_logged_in
-    ):
-        """Assert username is 'anonymous' when anonymous URL para is set to true."""
-        response = client_logged_in.get(self._get_url() + "?anonymous=true")
-        assert response.status_code == HTTPStatus.OK
-        content = response.json()
-        assert content["username"] == "anonymous"
-        # Also check that the user_data is still present
-        assert "user_data" in content

From ac3b9086996032fbda2a81fdaf8ca7009710e3ce Mon Sep 17 00:00:00 2001
From: Sam Bland <sam.bland@york.ac.uk>
Date: Thu, 15 Oct 2026 07:44:18 +0100
Subject: [PATCH 09/13] Used Mixins from account views instead of recreating
 for data account views

---
 main/views/account_data_views.py      | 57 +++++----------------------
 tests/main/test_account_data_views.py | 11 +-----
 2 files changed, 11 insertions(+), 57 deletions(-)

diff --git a/main/views/account_data_views.py b/main/views/account_data_views.py
index f1bdb97b..3592352c 100644
--- a/main/views/account_data_views.py
+++ b/main/views/account_data_views.py
@@ -1,58 +1,15 @@
 """Views for getting account-related data from the framework database."""
 
 import csv
-import logging
-from typing import TYPE_CHECKING, Any
 
 from django.http import (
-    HttpRequest,
     HttpResponse,
-    HttpResponseForbidden,
     JsonResponse,
 )
-from django.http.response import HttpResponseBase
 from django.views import View
 
 from ..models import User, UserSkill
-
-logger = logging.getLogger(__name__)
-
-
-if TYPE_CHECKING:  # pragma: no cover
-    from ..models import User as UserType
-
-
-class _AuthenticatedHttpRequest(HttpRequest):
-    """Custom HttpRequest type for authenticated users."""
-
-    user: "UserType"
-
-
-class _TermsAcceptedMixin:
-    """Require authenticated users to accept terms before accessing endpoints."""
-
-    terms_acceptance_url_name = "terms_acceptance"
-
-    def dispatch(
-        self, request: HttpRequest, *args: Any, **kwargs: Any
-    ) -> HttpResponseBase:
-        """Redirect to terms acceptance page when the user has not accepted terms."""
-        if not request.user.is_authenticated:
-            return HttpResponseForbidden(
-                "User must be authenticated to access this resource."
-            )
-
-        if not request.user.agreed_to_tos:
-            return HttpResponseForbidden(
-                "User must accept the terms of service to access this resource."
-            )
-        return super().dispatch(request, *args, **kwargs)  # type: ignore
-
-
-class AuthenticatedDataView(_TermsAcceptedMixin, View):
-    """A base view that ensures the user is authenticated formats user data."""
-
-    request: _AuthenticatedHttpRequest
+from .account_views import AuthenticatedHttpRequest, TermsAcceptedMixin
 
 
 class DownloadUserSkillDataMixin:
@@ -73,10 +30,12 @@ def _format_user_skill_data_for_download(
         ]
 
 
-class DownloadUserSkillDataCsvView(AuthenticatedDataView, DownloadUserSkillDataMixin):
+class DownloadUserSkillDataCsvView(
+    TermsAcceptedMixin, DownloadUserSkillDataMixin, View
+):
     """A view that returns the user skill data as a CSV file."""
 
-    def get(self, request: _AuthenticatedHttpRequest) -> HttpResponse:
+    def get(self, request: AuthenticatedHttpRequest) -> HttpResponse:
         """Handle GET request to download user skill data as CSV.
 
         Args:
@@ -104,10 +63,12 @@ def get(self, request: _AuthenticatedHttpRequest) -> HttpResponse:
         return response
 
 
-class DownloadUserSkillDataJSONView(AuthenticatedDataView, DownloadUserSkillDataMixin):
+class DownloadUserSkillDataJSONView(
+    TermsAcceptedMixin, DownloadUserSkillDataMixin, View
+):
     """A view that returns the user skill data as a JSON file."""
 
-    def get(self, request: _AuthenticatedHttpRequest) -> JsonResponse:
+    def get(self, request: AuthenticatedHttpRequest) -> JsonResponse:
         """Define the GET response.
 
         Args:
diff --git a/tests/main/test_account_data_views.py b/tests/main/test_account_data_views.py
index 9dbb0fc2..f5094b8e 100644
--- a/tests/main/test_account_data_views.py
+++ b/tests/main/test_account_data_views.py
@@ -7,17 +7,10 @@
 
 from django.urls import reverse
 
+from .test_main_views import LoginRequiredMixin
 
-class ViewRequiresAuthenticationMixin:
-    """Mixin to test that views require authentication."""
 
-    def test_unauthenticated_user_gets_403(self, client):
-        """Test that an unauthenticated user receives a 403 Forbidden response."""
-        response = client.get(self._get_url())
-        assert response.status_code == HTTPStatus.FORBIDDEN
-
-
-class DownloadUserSkillDataMixin(ViewRequiresAuthenticationMixin):
+class DownloadUserSkillDataMixin(LoginRequiredMixin):
     """Test suite for the GetUserSkillsDataMixin."""
 
     _url: str

From ff557fb56c0ca13f132a57a81e1ca4fd46518b75 Mon Sep 17 00:00:00 2001
From: Sam Bland <sam.bland@york.ac.uk>
Date: Thu, 15 Oct 2026 07:58:11 +0100
Subject: [PATCH 10/13] Use function instead of mixin class for format user
 skill data in account data views

---
 main/views/account_data_views.py | 40 +++++++++++++-------------------
 1 file changed, 16 insertions(+), 24 deletions(-)

diff --git a/main/views/account_data_views.py b/main/views/account_data_views.py
index 3592352c..38f36175 100644
--- a/main/views/account_data_views.py
+++ b/main/views/account_data_views.py
@@ -12,27 +12,21 @@
 from .account_views import AuthenticatedHttpRequest, TermsAcceptedMixin
 
 
-class DownloadUserSkillDataMixin:
-    """Mixin to provide functionality for downloading user skill data."""
-
-    def _format_user_skill_data_for_download(
-        self, user: User
-    ) -> list[dict[str, str | int]]:
-        user_skills = UserSkill.objects.filter(user=user.pk)
-        return [
-            {
-                "skill": user_skill.skill.name,
-                "competency_domain": user_skill.skill.competency.competency_domain.name,
-                "competency": user_skill.skill.competency.name,
-                "skill_level": user_skill.skill_level.level,
-            }
-            for user_skill in user_skills
-        ]
+def format_user_skill_data_for_download(user: User) -> list[dict[str, str | int]]:
+    """Format the user skill data for download."""
+    user_skills = UserSkill.objects.filter(user=user.pk)
+    return [
+        {
+            "skill": user_skill.skill.name,
+            "competency_domain": user_skill.skill.competency.competency_domain.name,
+            "competency": user_skill.skill.competency.name,
+            "skill_level": user_skill.skill_level.level,
+        }
+        for user_skill in user_skills
+    ]
 
 
-class DownloadUserSkillDataCsvView(
-    TermsAcceptedMixin, DownloadUserSkillDataMixin, View
-):
+class DownloadUserSkillDataCsvView(TermsAcceptedMixin, View):
     """A view that returns the user skill data as a CSV file."""
 
     def get(self, request: AuthenticatedHttpRequest) -> HttpResponse:
@@ -44,7 +38,7 @@ def get(self, request: AuthenticatedHttpRequest) -> HttpResponse:
         Returns:
             An HttpResponse containing the user skill data as a CSV file
         """
-        user_skills_data = self._format_user_skill_data_for_download(request.user)
+        user_skills_data = format_user_skill_data_for_download(request.user)
         response = HttpResponse(
             content_type="text/csv",
             headers={"Content-Disposition": 'attachment; filename="downloaded_.csv"'},
@@ -63,9 +57,7 @@ def get(self, request: AuthenticatedHttpRequest) -> HttpResponse:
         return response
 
 
-class DownloadUserSkillDataJSONView(
-    TermsAcceptedMixin, DownloadUserSkillDataMixin, View
-):
+class DownloadUserSkillDataJSONView(TermsAcceptedMixin, View):
     """A view that returns the user skill data as a JSON file."""
 
     def get(self, request: AuthenticatedHttpRequest) -> JsonResponse:
@@ -77,7 +69,7 @@ def get(self, request: AuthenticatedHttpRequest) -> JsonResponse:
         Returns:
             A JsonResponse containing the user skill data in JSON format.
         """
-        user_skills_data = self._format_user_skill_data_for_download(request.user)
+        user_skills_data = format_user_skill_data_for_download(request.user)
         get_anonymous = request.GET.get("anonymous", "false").lower() == "true"
         returned_data = {
             "username": "anonymous" if get_anonymous else request.user.username,

From 6e89962f35514b51229339a6abed5a2fb279e40b Mon Sep 17 00:00:00 2001
From: "pre-commit-ci[bot]"
 <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Date: Thu, 15 Oct 2026 06:58:44 +0000
Subject: [PATCH 11/13] [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci
---
 tests/main/conftest.py | 1 +
 1 file changed, 1 insertion(+)

diff --git a/tests/main/conftest.py b/tests/main/conftest.py
index ddd0e7d3..d50340cb 100644
--- a/tests/main/conftest.py
+++ b/tests/main/conftest.py
@@ -120,6 +120,7 @@ def event() -> Event:
         contributors="Test Contributor",
     )
 
+
 @pytest.fixture
 def client_logged_in(client, user):
     """Fixture for a client that is logged in as a user."""

From 531c29a95fd34b024fc7e5e2db155f18e6fceb8e Mon Sep 17 00:00:00 2001
From: Sam Bland <sam.bland@york.ac.uk>
Date: Thu, 15 Oct 2026 11:01:59 +0100
Subject: [PATCH 12/13] Minor amends to download skill profile button ids and
 selectors

---
 .../snippets/create_user_skill_profile_link.html |  4 ++--
 .../main/snippets/export_user_skill_profile.html |  6 +++---
 .../snippets/share_skill_profile_dropdown.html   | 16 ++++++++--------
 3 files changed, 13 insertions(+), 13 deletions(-)

diff --git a/main/templates/main/snippets/create_user_skill_profile_link.html b/main/templates/main/snippets/create_user_skill_profile_link.html
index 099b8142..7f60e10d 100644
--- a/main/templates/main/snippets/create_user_skill_profile_link.html
+++ b/main/templates/main/snippets/create_user_skill_profile_link.html
@@ -11,8 +11,8 @@
 <script>
   // On document load we convert the skill levels and chart data into a query string and set it as the href of the export link.
   document.addEventListener('DOMContentLoaded', function () {
-    const exportSkillProfileLink = document.querySelector('.export-skill-profile-link');
-    const exportSkillProfileLinkCopyButton = document.querySelector('.export-skill-profile-link-copy');
+    const exportSkillProfileLink = document.querySelector('#export-skill-profile-link');
+    const exportSkillProfileLinkCopyButton = document.querySelector('#export-skill-profile-link-copy');
     const skillLevels = {{ skill_levels|safe }};
     const charts = {{ chart_data|safe }};
 
diff --git a/main/templates/main/snippets/export_user_skill_profile.html b/main/templates/main/snippets/export_user_skill_profile.html
index d3b724ef..4fefd483 100644
--- a/main/templates/main/snippets/export_user_skill_profile.html
+++ b/main/templates/main/snippets/export_user_skill_profile.html
@@ -7,7 +7,7 @@
 
 -->
 <script id="export-skills-data-csv">
-    const exportSkillProfileCsvButton = document.querySelector('.export-skill-profile-csv');
+    const exportSkillProfileCsvButton = document.querySelector('#export-skill-profile-csv');
     exportSkillProfileCsvButton.addEventListener('click', function () {
       // The skill levels and chart data are passed from the Django view
       // We assume we receive the same data as the skill wheel.
@@ -32,7 +32,7 @@
     });
 </script>
 <script id="export-skills-data-json">
-    const exportSkillProfileJSONButton = document.querySelector('.export-skill-profile-json');
+    const exportSkillProfileJSONButton = document.querySelector('#export-skill-profile-json');
     exportSkillProfileJSONButton.addEventListener('click', function () {
       // The skill levels and chart data are passed from the Django view
       // We assume we receive the same data as the skill wheel.
@@ -51,7 +51,7 @@
       }).flat();
       const jsonContent = "data:text/json;charset=utf-8," + encodeURIComponent(JSON.stringify(jsonData, null, 2));
       const link = document.createElement("a");
-      link.setAttribute("href", encodedUri);
+      link.setAttribute("href", jsonContent);
       link.setAttribute("download", "skill_profile.json");
       document.body.appendChild(link);
       link.click();
diff --git a/main/templates/main/snippets/share_skill_profile_dropdown.html b/main/templates/main/snippets/share_skill_profile_dropdown.html
index 45493203..1e70158e 100644
--- a/main/templates/main/snippets/share_skill_profile_dropdown.html
+++ b/main/templates/main/snippets/share_skill_profile_dropdown.html
@@ -13,16 +13,16 @@
            aria-expanded="false">Share</a>
         <ul class="dropdown-menu">
             <li>
-                <a class="dropdown-item export-skill-profile-link"
-                   id="shareable-link"
+                <a class="dropdown-item"
+                   id="export-skill-profile-link"
                    href="#"
                    target="_blank"
                    rel="noopener noreferrer">Open Link <i aria-label="Copy sharable link" class="fa-solid fa-link"></i></a>
             </li>
             <li>
                 <button type="button"
-                        class="dropdown-item export-skill-profile-link-copy"
-                        id="copy-link-button">
+                        class="dropdown-item"
+                        id="export-skill-profile-link-copy">
                     Copy Link <i aria-label="Copy sharable link"
     class="fa-solid fa-copy"></i>
                 </button>
@@ -30,14 +30,14 @@
             {% include "main/snippets/create_user_skill_profile_link.html" %}
             {% if allow_export %}
                 <li>
-                    <a class="dropdown-item export-skill-profile-csv"
+                    <a class="dropdown-item"
                        href="{% url 'download-csv-data-link' %}"
-                       id="copy-link-button">Export as CSV</a>
+                       id="download-csv-button">Export as CSV</a>
                 </li>
                 <li>
-                    <a class="dropdown-item export-skill-profile-json"
+                    <a class="dropdown-item"
                        href="{% url 'download-json-data-link' %}"
-                       id="copy-link-button">Export as JSON</a>
+                       id="download-json-button">Export as JSON</a>
                 </li>
             {% endif %}
         </ul>

From ff58c7debc4ec57a604d13ae80c9dc962cb9b85c Mon Sep 17 00:00:00 2001
From: Sam Bland <sam.bland@york.ac.uk>
Date: Thu, 15 Oct 2026 11:07:20 +0100
Subject: [PATCH 13/13] Minor amend to download file name

---
 main/views/account_data_views.py | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/main/views/account_data_views.py b/main/views/account_data_views.py
index 38f36175..3dcc5409 100644
--- a/main/views/account_data_views.py
+++ b/main/views/account_data_views.py
@@ -41,7 +41,7 @@ def get(self, request: AuthenticatedHttpRequest) -> HttpResponse:
         user_skills_data = format_user_skill_data_for_download(request.user)
         response = HttpResponse(
             content_type="text/csv",
-            headers={"Content-Disposition": 'attachment; filename="downloaded_.csv"'},
+            headers={"Content-Disposition": 'attachment; filename="skill_profile.csv"'},
         )
         writer = csv.writer(response)
         writer.writerow(["skill", "competency_domain", "competency", "skill_level"])
@@ -78,5 +78,7 @@ def get(self, request: AuthenticatedHttpRequest) -> JsonResponse:
         return JsonResponse(
             returned_data,
             json_dumps_params=dict(indent=2),
-            headers={"Content-Disposition": 'attachment; filename="downloaded_.json"'},
+            headers={
+                "Content-Disposition": 'attachment; filename="skill_profile.json"'
+            },
         )