Fix issues with privacy settings

components/ads/CookieConsent.vue

@@ -13,8 +13,8 @@
           button below:
         </span>
         <div class="actions">
-          <button class="btn button" @click="hide">Accept all</button>
+          <button class="btn button" @click="review">Review</button>
-          <button class="btn brand-button" @click="review">Review</button>
+          <button class="btn brand-button" @click="hide">Accept all</button>
         </div>
       </div>
     </div>
@@ -24,12 +24,7 @@
 <script>
 export default {
   name: 'CookieConsent',
-  data() {
+  fetch() {
-    return {
-      shown: false,
-    }
-  },
-  mounted() {
     // Get informations in the store
     this.$store.dispatch('consent/loadFromCookies', this.$cookies)
     if (
@@ -39,12 +34,20 @@ export default {
       this.shown = true
     }
   },
+  data() {
+    return {
+      shown: false,
+    }
+  },
   methods: {
     hide() {
-      this.shown = false
+      this.$store.commit('consent/set_consent', true)
+      this.$store.commit('consent/add_scope', true)
+      this.$store.commit('consent/remove_scope', true)
+      this.$store.dispatch('consent/save', this.$cookies)
     },
     review() {
-      this.hide()
+      this.shown = false
       this.$router.push('/dashboard/privacy')
     },
   },

modules/analytics/lib/templates/plugin.js

@@ -9,6 +9,9 @@ function isAnalyticsOn(ctx) {
     // Rely on the client
     cookies = document.cookie;
   }
+
+  if(!cookies) return true;
+
   let processed = {}
   cookies.split(';').forEach((e) => {
     let val = e.trim().split('=');
@@ -34,7 +37,6 @@ export default async function (ctx, inject) {
 
 
   if (!enabled) {
-    console.log("Analytics are not enabled.")
     return;
   }
 

modules/gpt-ads/lib/templates/plugin.js

@@ -1,7 +1,7 @@
 import Vue from 'vue';
 
 function isPersonalizedAdsOn(ctx) {
-  let cookies = null
+  let cookies = []
   if (ctx.req != null) {
     //Server side rendering
     cookies = ctx.req.headers.cookie;
@@ -9,6 +9,9 @@ function isPersonalizedAdsOn(ctx) {
     // Rely on the client
     cookies = document.cookie;
   }
+
+  if(!cookies) return true;
+
   let processed = {}
   cookies.split(';').forEach((e) => {
     let val = e.trim().split('=')

pages/dashboard.vue

@@ -29,6 +29,10 @@
             <SettingsIcon />
             Settings
           </nuxt-link>
+          <nuxt-link :to="'/dashboard/privacy'" class="tab last">
+            <SettingsIcon />
+            Privacy Settings
+          </nuxt-link>
         </div>
         <div v-else class="card page-nav">
           <a :href="authUrl" class="tab last">

pages/dashboard/privacy.vue

@@ -47,20 +47,25 @@
 import scopes from '@/privacy-toggles'
 export default {
   name: 'Privacy',
-  data: () => {
+  fetch() {
-    const settings = scopes.settings
+    Object.keys(scopes.settings).forEach((key) => {
-    return {
+      scopes.settings[key].value = scopes.settings[key].default
-      scopes: settings,
+    })
-    }
+
-  },
-  mounted() {
     this.$store.dispatch('consent/loadFromCookies', this.$cookies)
+
     // Load the allowed scopes from the store
     this.$store.state.consent.scopes_allowed.forEach((scope) => {
       if (this.scopes[scope] != null)
         this.$set(this.scopes[scope], 'value', true)
     })
   },
+  data: () => {
+    const settings = scopes.settings
+    return {
+      scopes: settings,
+    }
+  },
   options: {
     auth: false,
   },

privacy-toggles.js

@@ -5,7 +5,7 @@ export default {
       description: `Marketing/target cookies are usually used to show you advertisements that meet your interests.
        When you visit another website, your browser's cookie is recognized and selected ads are displayed to you
        based on the information stored in this cookie.`,
-      default: false,
+      default: true,
     },
     analytics: {
       title: 'Analytics',

store/consent.js

@@ -36,7 +36,7 @@ export const actions = {
     }
     state.commit('set_consent', $cookies.get('modrinth-consent') === true)
     const scopes = $cookies.get('modrinth-scopes')
-    if (scopes == null) return
+    if (!scopes) return
     scopes.split(',').forEach((elem) => {
       state.commit('add_scope', elem)
     })