From 24189918f3f875a96704d033dbfb566843bae150 Mon Sep 17 00:00:00 2001
From: LEGO Technix <109212476+lego-technix@users.noreply.github.com>
Date: Thu, 29 Dec 2022 18:49:00 +0100
Subject: [PATCH 1/6] sr: simplify rootRedirect condition

---
 nuxt.config.js | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/nuxt.config.js b/nuxt.config.js
index c1d6867ae..83ea76198 100755
--- a/nuxt.config.js
+++ b/nuxt.config.js
@@ -18,8 +18,7 @@ const i18nConfigurationForInternationalDomain = {
   vueI18n: {
     fallbackLocale: 'fr',
   },
-  rootRedirect:
-    config.isPixSite && !config.isFrenchDomain ? 'locale-choice' : undefined,
+  rootRedirect: config.isPixSite && !config.isFrenchDomain && 'locale-choice',
 }
 
 const nuxtConfig = {

From afbc6b49ac45c5e3d244d1ec93ed3f5b39b1ed35 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?R=C3=A9becca?= <rebecca.kaci@gmail.com>
Date: Tue, 27 Dec 2022 14:51:29 +0100
Subject: [PATCH 2/6] feat: add PixLink component

---
 components/PixLink.vue           | 42 ++++++++++++++++++++++++++++
 tests/components/PixLink.test.js | 47 ++++++++++++++++++++++++++++++++
 2 files changed, 89 insertions(+)
 create mode 100644 components/PixLink.vue
 create mode 100644 tests/components/PixLink.test.js

diff --git a/components/PixLink.vue b/components/PixLink.vue
new file mode 100644
index 000000000..9090f0894
--- /dev/null
+++ b/components/PixLink.vue
@@ -0,0 +1,42 @@
+<template>
+  <component :is="linkComponent">
+    <slot />
+  </component>
+</template>
+
+<script>
+export default {
+  name: 'PixLink',
+  props: {
+    href: {
+      required: true,
+      type: String,
+      default: '',
+    },
+  },
+  computed: {
+    linkComponent() {
+      let template = ''
+      if (this.isOnCurrentDomain(this.href)) {
+        template = `
+          <nuxt-link to="${this.href}">
+            <slot />
+          </nuxt-link>
+         `
+      } else {
+        template = `
+          <a href="${this.href}">
+            <slot />
+          </a>
+        `
+      }
+      return { template }
+    },
+  },
+  methods: {
+    isOnCurrentDomain(targetDomain) {
+      return targetDomain.startsWith('/') && !targetDomain.startsWith('//')
+    },
+  },
+}
+</script>
diff --git a/tests/components/PixLink.test.js b/tests/components/PixLink.test.js
new file mode 100644
index 000000000..1d502fdc1
--- /dev/null
+++ b/tests/components/PixLink.test.js
@@ -0,0 +1,47 @@
+import { mount } from '@vue/test-utils'
+import PixLink from '~/components/PixLink'
+
+describe('Component: PixLink', () => {
+  let component
+
+  beforeEach(() => {
+    component = mount(PixLink, {
+      shallow: true,
+      propsData: { href: '' },
+      slots: { default: '<h1>Dummy</h1>' },
+      stubs: ['nuxt-link'],
+    })
+  })
+
+  describe('href is in domain', () => {
+    it('displays the component', async () => {
+      // given
+      await component.setProps({ href: '/fr-be' })
+
+      // when
+      const result = component.html()
+
+      // then
+      expect(result).toEqual(
+        `<nuxt-link-stub to="/fr-be">
+  <h1>Dummy</h1>
+</nuxt-link-stub>`
+      )
+    })
+  })
+
+  describe('href is not in domain', () => {
+    it('displays the component', async () => {
+      // given
+      await component.setProps({ href: 'https://example.net/' })
+
+      // when
+      const result = component.html()
+
+      // then
+      expect(result).toEqual(`<a href="https://example.net/">
+  <h1>Dummy</h1>
+</a>`)
+    })
+  })
+})

From 46c82eb08d546458f60349e20740f65e6afb07ae Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?R=C3=A9becca?= <rebecca.kaci@gmail.com>
Date: Tue, 27 Dec 2022 14:52:03 +0100
Subject: [PATCH 3/6] feat: use pix-link in LocaleLink component

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

diff --git a/components/LocaleLink.vue b/components/LocaleLink.vue
index 213b093c0..c2e44725e 100644
--- a/components/LocaleLink.vue
+++ b/components/LocaleLink.vue
@@ -1,8 +1,8 @@
 <template>
-  <a class="locale-link" :href="indexForGivenLocale()">
+  <pix-link class="locale-link" :href="indexForGivenLocale()">
     <img :src="'/images/' + locale.icon" alt="" />
     <span class="locale-link__text">{{ locale.name }}</span>
-  </a>
+  </pix-link>
 </template>
 
 <script>

From 42d9cc3229eec0830b52eb8bda04c6e4e2617d6d Mon Sep 17 00:00:00 2001
From: Vincent Hardouin <vincent.hardouin@pix.fr>
Date: Tue, 27 Dec 2022 16:57:20 +0100
Subject: [PATCH 4/6] feat: use pix-link in LanguageSwitcher component

---
 components/LanguageSwitcher/LanguageSwitcher.vue     | 12 ++++++------
 .../LanguageSwitcher/LanguageSwitcherSubMenu.vue     |  4 ++--
 tests/components/LanguageSwitcher.test.js            |  7 +++++--
 3 files changed, 13 insertions(+), 10 deletions(-)

diff --git a/components/LanguageSwitcher/LanguageSwitcher.vue b/components/LanguageSwitcher/LanguageSwitcher.vue
index 52d91fcd3..252975424 100644
--- a/components/LanguageSwitcher/LanguageSwitcher.vue
+++ b/components/LanguageSwitcher/LanguageSwitcher.vue
@@ -53,7 +53,7 @@
               :selected-menu="selectedMenu"
             />
           </template>
-          <a v-else-if="!option.children" :href="getIndexUrl(option)">
+          <pix-link v-else-if="!option.children" :href="getIndexUrl(option)">
             <img
               class="language-switcher__img"
               :src="'/images/' + option.icon"
@@ -65,7 +65,7 @@
                 {{ $t(option.subtitle) }}
               </div>
             </div>
-          </a>
+          </pix-link>
         </div>
       </li>
     </ul>
@@ -84,9 +84,9 @@
                 child.lang === currentLocaleCode,
             }"
           >
-            <a :href="getIndexUrl(child)">
+            <pix-link :href="getIndexUrl(child)">
               {{ $t(option.name) }} - {{ $t(child.name) }}
-            </a>
+            </pix-link>
             <br />
           </span>
         </template>
@@ -98,9 +98,9 @@
                 option.lang === currentLocaleCode,
             }"
           >
-            <a :href="getIndexUrl(option)">
+            <pix-link :href="getIndexUrl(option)">
               {{ $t(option.name) }}
-            </a>
+            </pix-link>
           </span>
         </template>
       </li>
diff --git a/components/LanguageSwitcher/LanguageSwitcherSubMenu.vue b/components/LanguageSwitcher/LanguageSwitcherSubMenu.vue
index f7b63c359..ec32a18e0 100644
--- a/components/LanguageSwitcher/LanguageSwitcherSubMenu.vue
+++ b/components/LanguageSwitcher/LanguageSwitcherSubMenu.vue
@@ -9,9 +9,9 @@
       }"
     >
       <div class="language-switcher__lang">
-        <a :href="getIndexUrl(availableLanguage)">
+        <pix-link :href="getIndexUrl(availableLanguage)">
           {{ $t(availableLanguage.name) }}
-        </a>
+        </pix-link>
       </div>
     </li>
   </ul>
diff --git a/tests/components/LanguageSwitcher.test.js b/tests/components/LanguageSwitcher.test.js
index 9786816f0..a68fa5dda 100644
--- a/tests/components/LanguageSwitcher.test.js
+++ b/tests/components/LanguageSwitcher.test.js
@@ -1,5 +1,6 @@
 import { shallowMount, mount } from '@vue/test-utils'
 import { config } from '~/config/environment'
+import PixLink from '~/components/PixLink'
 import LanguageSwitcher from '~/components/LanguageSwitcher/LanguageSwitcher'
 import LanguageSwitcherSubMenu from '~/components/LanguageSwitcher/LanguageSwitcherSubMenu.vue'
 
@@ -14,6 +15,8 @@ jest.mock('~/config/environment', () => ({
   },
 }))
 
+LanguageSwitcherSubMenu.components = { 'pix-link': PixLink }
+
 describe('Component: LanguageSwitcher', () => {
   describe('when site is pix-pro', () => {
     beforeEach(() => {
@@ -51,7 +54,7 @@ describe('Component: LanguageSwitcher', () => {
 
       // when
       const wrapper = mount(LanguageSwitcher, {
-        components: { LanguageSwitcherSubMenu },
+        components: { LanguageSwitcherSubMenu, PixLink },
         mocks: { $t, $i18n },
         stubs: { fa: true },
       })
@@ -76,7 +79,7 @@ describe('Component: LanguageSwitcher', () => {
 
       // when
       const wrapper = mount(LanguageSwitcher, {
-        components: { LanguageSwitcherSubMenu },
+        components: { LanguageSwitcherSubMenu, PixLink },
         mocks: { $t, $i18n },
         stubs: { fa: true },
       })

From c6957ce5efd0d120b6100341d019406f5e5bb67e Mon Sep 17 00:00:00 2001
From: LEGO Technix <109212476+lego-technix@users.noreply.github.com>
Date: Thu, 29 Dec 2022 11:08:16 +0100
Subject: [PATCH 5/6] fix: watch i18n change to fetch documents again in the
 new locale

---
 components/FooterSliceZone.vue     | 3 +++
 components/HotNewsBanner.vue       | 3 +++
 components/NavigationSliceZone.vue | 3 +++
 3 files changed, 9 insertions(+)

diff --git a/components/FooterSliceZone.vue b/components/FooterSliceZone.vue
index 92f5839b7..3cdc5ff0c 100644
--- a/components/FooterSliceZone.vue
+++ b/components/FooterSliceZone.vue
@@ -75,6 +75,9 @@ export default {
       )
     },
   },
+  watch: {
+    '$i18n.locale': '$fetch',
+  },
 }
 </script>
 
diff --git a/components/HotNewsBanner.vue b/components/HotNewsBanner.vue
index a95511e20..266b6ffbe 100644
--- a/components/HotNewsBanner.vue
+++ b/components/HotNewsBanner.vue
@@ -29,6 +29,9 @@ export default {
 
     this.hotNews = hotNews?.data?.description
   },
+  watch: {
+    '$i18n.locale': '$fetch',
+  },
   methods: {
     closeBanner() {
       this.isOpen = false
diff --git a/components/NavigationSliceZone.vue b/components/NavigationSliceZone.vue
index c45f7d25d..e6085309c 100644
--- a/components/NavigationSliceZone.vue
+++ b/components/NavigationSliceZone.vue
@@ -77,6 +77,9 @@ export default {
       }
     },
   },
+  watch: {
+    '$i18n.locale': '$fetch',
+  },
 }
 </script>
 

From 568740e450b83ab5a10cb3b8761251bcecfe26c4 Mon Sep 17 00:00:00 2001
From: Vincent Hardouin <vincent.hardouin@pix.fr>
Date: Tue, 27 Dec 2022 18:07:13 +0100
Subject: [PATCH 6/6] sr: add empty values for navigation

---
 components/FooterSliceZone.vue       | 2 +-
 components/NavigationSliceZone.vue   | 6 +++---
 components/slices/NavigationZone.vue | 6 +++++-
 3 files changed, 9 insertions(+), 5 deletions(-)

diff --git a/components/FooterSliceZone.vue b/components/FooterSliceZone.vue
index 3cdc5ff0c..8a726fdd0 100644
--- a/components/FooterSliceZone.vue
+++ b/components/FooterSliceZone.vue
@@ -57,7 +57,7 @@ export default {
   data() {
     return {
       socialMediasHoverMap: {},
-      usedMainFooter: null,
+      usedMainFooter: [],
     }
   },
   async fetch() {
diff --git a/components/NavigationSliceZone.vue b/components/NavigationSliceZone.vue
index e6085309c..7e2960383 100644
--- a/components/NavigationSliceZone.vue
+++ b/components/NavigationSliceZone.vue
@@ -35,7 +35,7 @@ export default {
   name: 'NavigationSliceZone',
   data() {
     return {
-      usedMainNavigation: null,
+      usedMainNavigation: [],
     }
   },
   async fetch() {
@@ -67,10 +67,10 @@ export default {
     burgerMenuLinks() {
       const navigationZone = this.usedMainNavigation.find(
         (slice) => slice.slice_type === 'navigation_zone'
-      )
+      ) || { items: [] }
       const actionsZone = this.usedMainNavigation.find(
         (slice) => slice.slice_type === 'actions_zone'
-      )
+      ) || { items: [] }
       return {
         navigationZone: navigationZone.items,
         actionsZone: actionsZone.items,
diff --git a/components/slices/NavigationZone.vue b/components/slices/NavigationZone.vue
index 2db7ca298..2752dce61 100644
--- a/components/slices/NavigationZone.vue
+++ b/components/slices/NavigationZone.vue
@@ -51,7 +51,7 @@ export default {
   props: {
     navigationZoneItems: {
       type: Array,
-      default: null,
+      default: () => [],
     },
   },
   data() {
@@ -62,6 +62,10 @@ export default {
   },
   computed: {
     navigationLinks() {
+      if (this.navigationZoneItems.length === 0) {
+        return null
+      }
+
       return this.navigationZoneItems
         .map(({ items: navItems }) =>
           navItems.reduce((navGroup, navItem) => {