From a1b21dc20f6506987da73baecd22e1cf37eea238 Mon Sep 17 00:00:00 2001
From: Tom Blake <46909693+tom-blake@users.noreply.github.com>
Date: Thu, 12 Dec 2024 13:31:48 +1300
Subject: [PATCH] Fix On This Page links (#1147)

On this page links to anchor headings are currently broken due to
the Astro v5 upgrade renaming all instances of 'slug' to 'id'.

Test plan:

- Confirm links to anchor headings in the On this page component
  are being set to expected values for guides and connection collections
---
 src/components/TocNav.vue | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/components/TocNav.vue b/src/components/TocNav.vue
index 46e1e9207..d1af1d9c5 100644
--- a/src/components/TocNav.vue
+++ b/src/components/TocNav.vue
@@ -13,17 +13,17 @@
     <ol class="mt-4">
       <li
         v-for="heading in visibleHeadings"
-        :key="heading.id"
+        :key="heading.slug"
       >
         <a
-          :href="`#${heading.id}`"
+          :href="`#${heading.slug}`"
           class="flex truncate border-l-2 text-sm p-squish-2 hover:border-content-primary hover:text-content-primary"
           :class="[
-            visibleHeadingId === heading.id ?
+            visibleHeadingId === heading.slug ?
               'border-brand-accent text-content-primary':
               'text-content-tertiary'
           ]"
-          @click="handleTocClick(heading.id)"
+          @click="handleTocClick(heading.slug)"
         >
           <span
             :class="{