From 7350d75f0209d9abf4960bec3e9aca7a81e2f169 Mon Sep 17 00:00:00 2001
From: Stephanie Smith <stephanie.c.smith@jpl.nasa.gov>
Date: Tue, 19 Nov 2024 10:33:38 -0800
Subject: [PATCH 1/2] fixing link logic in HomepageTeaserBlock

---
 .changeset/slimy-hairs-rush.md                |  5 +++
 .../HomepageTeaserBlock.vue                   | 33 ++++++++++---------
 packages/vue/src/utils/mixins.ts              |  4 +--
 3 files changed, 25 insertions(+), 17 deletions(-)
 create mode 100644 .changeset/slimy-hairs-rush.md

diff --git a/.changeset/slimy-hairs-rush.md b/.changeset/slimy-hairs-rush.md
new file mode 100644
index 00000000..b1ad117a
--- /dev/null
+++ b/.changeset/slimy-hairs-rush.md
@@ -0,0 +1,5 @@
+---
+"@explorer-1/vue": patch
+---
+
+Fixing link logic in HomepageTeaserBlock
diff --git a/packages/vue/src/components/HomepageTeaserBlock/HomepageTeaserBlock.vue b/packages/vue/src/components/HomepageTeaserBlock/HomepageTeaserBlock.vue
index c4ceed8d..0f01906a 100644
--- a/packages/vue/src/components/HomepageTeaserBlock/HomepageTeaserBlock.vue
+++ b/packages/vue/src/components/HomepageTeaserBlock/HomepageTeaserBlock.vue
@@ -33,14 +33,14 @@
             class="lg:pr-14"
             :text="data.paragraph"
           />
-          <template v-if="data.link && theLink">
+          <template v-if="data.link || theExternalLink">
             <BaseLink
               variant="primary"
               class="mt-5"
               link-class="inline-block"
               caret-wrapper-class="py-2"
-              :href="theLink"
-              :to="data.link.page ? data.link.page.url : null"
+              :href="theExternalLink"
+              :to="data.link.page ? data.link.page.url : undefined"
               external-target-blank
             >
               {{ data.link.text }}
@@ -60,13 +60,13 @@
               v-if="theCard.image"
               class="relative h-auto"
             >
-              <template v-if="theCard.link && theLink">
+              <template v-if="theCardExternalLink || theCard.link">
                 <BaseLink
                   variant="none"
                   link-class="block"
                   :aria-label="theCard.link.text || 'Learn more'"
-                  :href="theLink"
-                  :to="theCard.link.page ? theCard.link.page.url : null"
+                  :href="theCardExternalLink"
+                  :to="theCard.link.page?.url || undefined"
                   external-target-blank
                 >
                   <HomepageTeaserBlockCardImage :data="theCard" />
@@ -87,14 +87,14 @@
                 {{ theCard.description }}
               </p>
 
-              <template v-if="theCard.link && theLink">
+              <template v-if="theCard.link && theCardExternalLink">
                 <BaseLink
                   variant="primary"
                   class="mt-3"
                   link-class="inline-block"
                   caret-wrapper-class="py-2"
-                  :href="theLink"
-                  :to="theCard.link.page ? theCard.link.page.url : null"
+                  :href="theCardExternalLink"
+                  :to="theCard.link.page ? theCard.link.page.url : undefined"
                   external-target-blank
                 >
                   {{ theCard.link.text }}
@@ -135,11 +135,11 @@ export default defineComponent({
     }
   },
   computed: {
-    theCard(): object | null {
-      if (this.data.card && this.data.card.length > 0) {
-        return this.data.card[0]
+    theCard(): object | undefined {
+      if (this.data?.card && this.data.card.length > 0) {
+        return this.data?.card[0]
       }
-      return null
+      return undefined
     },
     hasCoverImage(): boolean {
       if (this.data.coverImage && this.data.coverImage.src) {
@@ -147,8 +147,11 @@ export default defineComponent({
       }
       return false
     },
-    theLink() {
-      return mixinGetExternalLink(this.data.link) || undefined
+    theExternalLink() {
+      return mixinGetExternalLink(this.data?.link) || undefined
+    },
+    theCardExternalLink() {
+      return mixinGetExternalLink(this.theCard?.link) || undefined
     },
     theSrcSet() {
       return this.data.coverImage
diff --git a/packages/vue/src/utils/mixins.ts b/packages/vue/src/utils/mixins.ts
index f8210258..bd323bf9 100644
--- a/packages/vue/src/utils/mixins.ts
+++ b/packages/vue/src/utils/mixins.ts
@@ -217,9 +217,9 @@ export const mixinGetSrcSet = (srcSetObject: Partial<ImageObject>): string => {
 }
 // Use with RelatedLinkBlock to retrieve the external link to use with an href prop
 export const mixinGetExternalLink = (link: RelatedLinkObject): string | undefined => {
-  if (link.externalLink) {
+  if (link?.externalLink) {
     return link.externalLink
-  } else if (link.document) {
+  } else if (link?.document) {
     return link.document.url
   }
   return undefined

From 5e3c448175349e5e3fa040ad5d36d51df19a21fb Mon Sep 17 00:00:00 2001
From: Stephanie Smith <stephanie.c.smith@jpl.nasa.gov>
Date: Tue, 19 Nov 2024 10:37:54 -0800
Subject: [PATCH 2/2] fixing a missed operator

---
 .../components/HomepageTeaserBlock/HomepageTeaserBlock.vue    | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/packages/vue/src/components/HomepageTeaserBlock/HomepageTeaserBlock.vue b/packages/vue/src/components/HomepageTeaserBlock/HomepageTeaserBlock.vue
index 0f01906a..44126fa7 100644
--- a/packages/vue/src/components/HomepageTeaserBlock/HomepageTeaserBlock.vue
+++ b/packages/vue/src/components/HomepageTeaserBlock/HomepageTeaserBlock.vue
@@ -60,7 +60,7 @@
               v-if="theCard.image"
               class="relative h-auto"
             >
-              <template v-if="theCardExternalLink || theCard.link">
+              <template v-if="theCard.link || theCardExternalLink">
                 <BaseLink
                   variant="none"
                   link-class="block"
@@ -87,7 +87,7 @@
                 {{ theCard.description }}
               </p>
 
-              <template v-if="theCard.link && theCardExternalLink">
+              <template v-if="theCard.link || theCardExternalLink">
                 <BaseLink
                   variant="primary"
                   class="mt-3"