@kalisio/kdk 2.1.9 → 2.2.1

package/.travis.test.sh

@@ -3,23 +3,56 @@
 check_code()
 {
    if [ $1 -eq 1 ]; then
-	  echo "$2 has failed [error: $1]"
-	  exit 1
+      echo "$2 has failed [error: $1]"
+      exit 1
   fi
 }
 
+AGE_VERSION=1.1.1
+SOPS_VERSION=3.8.1
+TMP_PATH="$(mktemp -d -p "${XDG_RUNTIME_DIR:-}" kalisio.XXXXXX)"
+
+install_age() {
+    local DL_PATH="$TMP_PATH/age"
+    mkdir "$DL_PATH" && pushd "$DL_PATH" || exit
+    wget -q https://github.com/FiloSottile/age/releases/download/v${AGE_VERSION}/age-v${AGE_VERSION}-linux-amd64.tar.gz
+    # no checksum ...
+    tar xf age-v${AGE_VERSION}-linux-amd64.tar.gz
+    cp age/age "$HOME/.local/bin"
+    popd || exit
+}
+
+install_sops() {
+    local DL_PATH="$TMP_PATH/sops"
+    mkdir "$DL_PATH" && pushd "$DL_PATH" || exit
+    wget -q https://github.com/getsops/sops/releases/download/v${SOPS_VERSION}/sops-v${SOPS_VERSION}.linux.amd64
+    wget -q https://github.com/getsops/sops/releases/download/v${SOPS_VERSION}/sops-v${SOPS_VERSION}.checksums.txt
+    sha256sum --ignore-missing --quiet -c sops-v${SOPS_VERSION}.checksums.txt
+    cp sops-v${SOPS_VERSION}.linux.amd64 "$HOME/.local/bin/sops"
+    chmod a+x "$HOME/.local/bin/sops"
+    popd || exit
+}
+
+# Make sure that folder exists, with Travis CI it's already in the PATH
+mkdir -p "$HOME/.local/bin"
+install_age
+install_sops
+
 echo -e "machine github.com\n  login $GITHUB_TOKEN" > ~/.netrc
 
 # Setup KDK
 yarn install
 
-# Clone the workspace 
-git clone https://github.com/kalisio/kdk-workspaces workspace
-
 # Clone others direct dependencies we'd like to use for testing
 git clone https://github.com/kalisio/feathers-distributed && cd feathers-distributed && yarn install && yarn link && cd ..
 yarn link @kalisio/feathers-distributed
 
+git clone https://github.com/kalisio/feathers-s3 && cd feathers-s3 && yarn install && yarn link && cd ..
+yarn link @kalisio/feathers-s3
+
+git clone https://github.com/kalisio/feathers-import-export && cd feathers-import-export && yarn install && yarn link && yarn link @kalisio/feathers-s3 && cd ..
+yarn link @kalisio/feathers-import-export
+
 git clone https://github.com/kalisio/feathers-webpush && cd feathers-webpush && yarn install && yarn link && cd ..
 yarn link @kalisio/feathers-webpush
 
@@ -30,11 +63,10 @@ yarn link @weacast/core
 yarn link @weacast/gfs
 yarn link @weacast/probe
 
-# Read extra environment variables (merges common and flavor env)
-cp workspace/.env .env
-set -a
-. .env
-set +a
+# Clone the development project and configure the env
+git clone https://oauth2:$GITHUB_TOKEN@github.com/kalisio/development.git "development"
+source development/workspaces/libs/libs.sh kdk
 
+# Run the test
 yarn test
 check_code $? "Running tests"

package/README.md

@@ -1,10 +1,10 @@
-[![KDK](https://s3.eu-central-1.amazonaws.com/kalisioscope/kdk/kdk-logo-black-512x168.png)](https://kalisio.github.io/kdk/)
+[![KDK](https://s3.eu-central-1.amazonaws.com/kalisioscope/kdk/kdk-logo-color-512x192.png)](https://kalisio.github.io/kdk/)
 
 [![Latest Release](https://img.shields.io/github/v/tag/kalisio/kdk?sort=semver&label=latest)](https://github.com/kalisio/kdk/releases)
 [![Build Status](https://app.travis-ci.com/kalisio/kdk.svg?branch=master)](https://app.travis-ci.com/kalisio/kdk)
 [![Code Climate](https://codeclimate.com/github/kalisio/kdk/badges/gpa.svg)](https://codeclimate.com/github/kalisio/kdk)
 [![Test Coverage](https://codeclimate.com/github/kalisio/kdk/badges/coverage.svg)](https://codeclimate.com/github/kalisio/kdk/coverage)
-[![Dependency Status](https://img.shields.io/david/kalisio/kdk.svg?style=flat-square)](https://david-dm.org/kalisio/kdk)
+[![Dependency Status](https://img.shields.io/librariesio/github/kalisio/kdk?style=flat-square)](https://david-dm.org/kalisio/kdk)
 [![Documentation](https://img.shields.io/badge/documentation-available-brightgreen.svg)](https://kalisio.github.io/kdk/)
 [![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](https://opensource.org/licenses/MIT)
 

package/core/api/application.js

@@ -174,7 +174,7 @@ async function createService (name, app, options = {}) {
     paginate,
     multi: true,
     whitelist: [
-      '$exists', '$and', '$or', '$eq', '$elemMatch', '$distinct', '$groupBy', '$group', '$regex',
+      '$exists', '$and', '$or', '$not', '$eq', '$elemMatch', '$distinct', '$groupBy', '$group', '$regex',
       '$text', '$search', '$caseSensitive', '$language', '$diacriticSensitive',
       '$aggregate', '$near', '$nearSphere', '$geoIntersects', '$geoWithin',
       '$maxDistance', '$minDistance', '$geometry', '$box', '$polygon', '$center', '$centerSphere'
@@ -182,7 +182,7 @@ async function createService (name, app, options = {}) {
   }, options)
   if (serviceOptions.disabled) return undefined
   // For DB services a model has to be provided
-  const fileName = serviceOptions.fileName || name
+  const fileName = serviceOptions.modelName || name
 
   let dbService = false
   try {
@@ -523,6 +523,11 @@ export function kdk (config = {}) {
     service = await configureService(name, service, servicesPath)
     return service
   }
+  // This is used to declare existing service
+  app.declareService = function (name, service, serviceOptions) {
+    service = declareService(name, app, service, serviceOptions)
+    return service
+  }
   // This is used to create standard services
   app.createService = async function (name, options) {
     const service = await createService(name, app, options)

package/core/api/authentication.js

@@ -15,6 +15,22 @@ const { ObjectID } = mongodb
 const { oauth, OAuthStrategy } = OAuth
 const { NotAuthenticated } = errors
 
+export class Authentication extends AuthenticationService {
+  // Feathers does not seem to take input payload into account when renewing the token,
+  // see https://github.com/feathersjs/feathers/issues/3419
+  async getPayload (authResult, params) {
+    // Params can override defaults
+    if (params.payload) {
+      return params.payload
+    } else if (authResult.authentication && authResult.authentication.payload) {
+      // Avoid conflicting with default token options added when generating a new token
+      return _.omit(authResult.authentication.payload, ['aud', 'iss', 'exp', 'sub', 'iat', 'jti', 'nbf'])
+    } else {
+      return {}
+    }
+  }
+}
+
 export class AuthenticationProviderStrategy extends OAuthStrategy {
   async getEntityData (profile, entity) {
     const createEntity = _.isNil(entity)
@@ -151,7 +167,7 @@ export default function auth (app) {
   if (config.oauth) config.oauth = _.omitBy(config.oauth, _.isNil)
   app.set('authentication', config)
 
-  const authentication = new AuthenticationService(app)
+  const authentication = new Authentication(app)
   const strategies = config.authStrategies || []
   if (strategies.includes('jwt')) authentication.register('jwt', new JWTAuthenticationStrategy())
   if (strategies.includes('local')) authentication.register('local', new LocalStrategy())

package/core/api/db.js

@@ -72,8 +72,13 @@ export function objectifyIDs (object) {
 export function toObjectIDs (object, properties) {
   properties.forEach(property => {
     const value = _.get(object, property)
-    const id = createObjectID(value)
-    if (id) _.set(object, property, id)
+    if (Array.isArray(value)) {
+      const ids = value.map(id => createObjectID(id)).filter(id => id)
+      if (ids.length > 0) _.set(object, property, ids)
+    } else {
+      const id = createObjectID(value)
+      if (id) _.set(object, property, id)
+    }
   })
 }
 

package/core/api/hooks/hooks.authentication.js

@@ -14,6 +14,7 @@ export async function verifyGuest (hook) {
   }
   const app = hook.app
   const user = hook.result.user
+  if (!user) return hook
   debug('verifyGuest hook called on ', user._id)
 
   // Check whether the user has been inivted. If not, nothing to do
@@ -42,6 +43,7 @@ export async function consentGuest (hook) {
   }
   const app = hook.app
   const user = hook.result.user
+  if (!user) return hook
   debug('consentGuest hook called on ', user._id)
 
   // Check whether the user has been invited. If not, nothing to do
@@ -66,9 +68,9 @@ export async function consentGuest (hook) {
 
 export function discardAuthenticationProviders (hook) {
   const providers = hook.app.authenticationProviders || []
-  
+
   // Iterate through known providers
   for (const provider of providers) {
     discard(provider)(hook)
   }
-}
+}

package/core/api/hooks/hooks.authorisations.js

@@ -186,9 +186,19 @@ export async function authorise (hook) {
   if (checkAuthorisation) {
     // Build ability for user
     const authorisationService = hook.app.getService('authorisations')
-    const abilities = await authorisationService.getAbilities(hook.params.user)
+    // If no user we allow for a stateless token with permissions inside
+    let subject = hook.params.user
+    if (!subject) {
+      const payload = _.get(hook.params, 'authentication.payload')
+      // Token targeting API gateway (sub = keyId) or app used through iframe (appId = keyId)
+      if (payload && (payload.sub || payload.appId)) {
+        subject = Object.assign({ _id: (payload.sub || payload.appId) }, payload)
+      }
+    }
+    const abilities = await authorisationService.getAbilities(subject)
     hook.params.abilities = abilities
-    debug('User abilities are', abilities.rules)
+    if (hook.params.user) debug('User abilities are', abilities.rules)
+    else debug('Stateless abilities are', abilities.rules)
 
     // Check for access to service fisrt
     if (!hasServiceAbilities(abilities, hook.service)) {

package/core/api/hooks/hooks.model.js

@@ -8,7 +8,7 @@ import common from 'feathers-hooks-common'
 import makeDebug from 'debug'
 
 const { Conflict, BadRequest } = errors
-const { discard, disallow, getItems, replaceItems } = common
+const { getItems, replaceItems } = common
 const sift = siftModule.default
 const debug = makeDebug('kdk:core:model:hooks')
 
@@ -181,7 +181,7 @@ export async function populatePreviousObject (hook) {
     throw new Error('The \'populatePreviousObject\' hook should only be used as a \'before\' hook.')
   }
   const item = getItems(hook)
-  const id = (hook.id ? hook.id : item._id)
+  const id = (hook.id ? hook.id : _.get(item, '_id'))
   // Retrieve previous version of the item and make it available to next hooks
   if (id) {
     hook.params.previousItem = await hook.service.get(id.toString())
@@ -264,20 +264,20 @@ export function checkUnique (options = {}) {
 // Prevent patch service calls from changing certain fields.
 // Based on https://hooks-common.feathersjs.com/hooks.html#preventchanges
 // but updated to handle dot notation
-export function preventChanges(ifThrow, fieldNames) {
+export function preventChanges (ifThrow, fieldNames) {
   return (hook) => {
     if ((hook.type !== 'before') || (hook.method !== 'patch')) {
       throw new Error('The \'preventChanges\' hook should only be used as a \'before\' patch hook.')
     }
 
-    let data = { ...hook.data }
+    const data = { ...hook.data }
     // Check all data fields
     _.forOwn(hook.data, (value, key) => {
       fieldNames.forEach(name => {
         // If a prevented field is found or dot notation with a prevented field is used
         if ((key === name) || key.startsWith(`${name}.`)) {
           if (ifThrow) {
-            throw new BadRequest(`Field ${name} may not be patched. (preventChanges)`);
+            throw new BadRequest(`Field ${name} may not be patched. (preventChanges)`)
           }
           delete data[key]
         }

package/core/api/hooks/hooks.organisations.js

@@ -1,10 +1,7 @@
 import _ from 'lodash'
 import makeDebug from 'debug'
-import errors from '@feathersjs/errors'
 const debug = makeDebug('kdk:core:organisations:hooks')
 
-const { Forbidden } = errors
-
 export async function createOrganisationServices (hook) {
   if (hook.type !== 'after') {
     throw new Error('The \'createOrganisationServices\' hook should only be used as a \'after\' hook.')
@@ -153,4 +150,3 @@ export function removeOrganisationResources (resourceScope) {
     return hook
   }
 }
-

package/core/api/services/account/account.hooks.js

@@ -8,11 +8,15 @@ export default {
     all: [],
     find: [],
     get: [],
-    create: [when(hook => hook.data.action === 'resetPwdShort' || hook.data.action === 'passwordChange',
-      enforcePasswordPolicy({ userAsItem: false, passwordField: 'value.password' }))],
+    create: [
+      when(
+        (hook) => ['passwordChange', 'resetPwdShort', 'verifySignupSetPasswordLong', 'verifySignupSetPasswordShort'].includes(hook.data.action),
+        enforcePasswordPolicy({ userAsItem: false, passwordField: 'value.password' })
+      ),
+    ],
     update: [],
     patch: [],
-    remove: []
+    remove: [],
   },
 
   after: {
@@ -22,7 +26,7 @@ export default {
     create: [],
     update: [],
     patch: [],
-    remove: []
+    remove: [],
   },
 
   error: {
@@ -32,6 +36,6 @@ export default {
     create: [],
     update: [],
     patch: [],
-    remove: []
-  }
+    remove: [],
+  },
 }

package/core/api/services/account/account.service.js

@@ -103,7 +103,7 @@ export default function (name, app, options) {
 
   const servicePath = app.get('apiPath') + '/account'
   const userService = app.getService('users')
-  
+
   return new AccountService(app, {
     // By default it is impossible to reset password if email is not verified
     // The problem is that if you loose your password before validating your email you are blocked,

package/{map/api/services/geocoder/geocoder.hooks.js → core/api/services/import-export/import-export.hooks.js}

@@ -1,12 +1,14 @@
+import commonHooks from 'feathers-hooks-common'
+
 export default {
   before: {
     all: [],
-    find: [],
-    get: [],
+    find: [commonHooks.disallow()],
+    get: [commonHooks.disallow()],
     create: [],
-    update: [],
-    patch: [],
-    remove: []
+    update: [commonHooks.disallow()],
+    patch: [commonHooks.disallow()],
+    remove: [commonHooks.disallow()]
   },
 
   after: {

package/core/api/services/import-export/import-export.service.js

@@ -0,0 +1,11 @@
+import { Service } from '@kalisio/feathers-import-export'
+import makeDebug from 'debug'
+
+const debug = makeDebug('kdk:import-export:service')
+
+export default function (name, app, options) {
+  const config = app.get('import-export')
+  debug('Creating import-export service with config ', config)
+  const s3ServicePath = app.get('apiPath') + '/' + (config.s3Service)
+  return new Service(Object.assign({ app }, config, { s3ServicePath }))
+}

package/core/api/services/index.js

@@ -94,7 +94,10 @@ export default async function () {
   if (authConfig) {
     await app.createService('users', { modelsPath, servicesPath })
     debug('\'users\' service created')
-    await app.createService('account', { servicesPath, methods: ['create', 'verifyEmail'] })
+    await app.createService('account', {
+      servicesPath,
+      methods: ['create', 'verifyEmail']
+    })
     debug('\'account\' service created')
     await app.createService('authorisations', { servicesPath })
     debug('\'authorisations\' service created')
@@ -106,6 +109,15 @@ export default async function () {
     debug('\'storage\' service created')
   }
 
+  const importExportConfig = app.get('import-export')
+  if (importExportConfig) {
+    await app.createService('import-export', {
+      servicesPath,
+      events: ['import-created', 'import-completed', 'export-created', 'export-completed']
+    }, app)
+    debug('\'import-export\' service created')
+  }
+
   const orgConfig = app.get('organisations')
   if (orgConfig) {
     await createOrganisationService.call(app)

package/core/api/services/users/users.hooks.js

@@ -1,4 +1,3 @@
-import _ from 'lodash'
 import {
   serialize, updateAbilities, populatePreviousObject, hashPassword, disallowRegistration, allowLocalAuthentication,
   discardAuthenticationProviders, enforcePasswordPolicy, storePreviousPassword, sendNewSubscriptionEmail
@@ -49,11 +48,11 @@ export default {
       commonHooks.when(hook => hook.params.provider,
         commonHooks.discard('password'),
         commonHooks.discard('previousPasswords'),
-        discardAuthenticationProviders),
+        discardAuthenticationProviders)
       // Hide profile for external user as it may contain personal information
       // However, this causes an issue: https://github.com/feathersjs-ecosystem/feathers-reactive/issues/214
       // So let the application decide what to do
-      //commonHooks.when(isNotMe, commonHooks.discard('profile'))
+      // commonHooks.when(isNotMe, commonHooks.discard('profile'))
     ],
     find: [],
     get: [],

package/core/client/api.js

@@ -7,10 +7,10 @@ import feathers from '@feathersjs/client'
 import { io } from 'socket.io-client'
 import reactive from 'feathers-reactive/dist/feathers-reactive.js'
 import configuration from 'config'
-import { Platform } from 'quasar'
 import { permissions } from '../common/index.js'
 import { Store } from './store.js'
 import { Events } from './events.js'
+import { LocalStorage } from './local-storage.js'
 
 // Setup log level
 if (_.get(configuration, 'logs.level')) {
@@ -22,10 +22,7 @@ if (_.get(configuration, 'logs.level')) {
 export function createClient (config) {
   // Initiate the client
   const api = feathers()
-
-  function getBaseUrlStorageKey () {
-    return config.appName.toLowerCase() + '-baseUrl'
-  }
+  const baseUrlStorageKey = 'baseUrl'
 
   // Matchers that can be added to customize route guards
   let matchers = []
@@ -142,7 +139,7 @@ export function createClient (config) {
   }
   // Change the base URL/domain to be used (useful for mobile apps)
   api.setBaseUrl = function (baseUrl) {
-    window.localStorage.setItem(getBaseUrlStorageKey(), baseUrl)
+    LocalStorage.set(baseUrlStorageKey, baseUrl)
     // Updating this setting live does not seem to work well in Feathers
     // For now the caller should simply "reload" the app
     /*
@@ -162,15 +159,19 @@ export function createClient (config) {
     }
     */
   }
-
+  // Get the base URL/domain to be used (useful for mobile apps)
   api.getBaseUrl = function () {
-    // In case we are not in a mobile platform we can override the default app origin anyway
-    let origin = (Platform.is.cordova ? config.domain : config.origin || window.location.origin)
+    // We can override the default app origin anyway
+    const origin = config.origin || window.location.origin
     // Check for registered custom base Url if any
-    if (window.localStorage.getItem(getBaseUrlStorageKey())) {
-      origin = window.localStorage.getItem(getBaseUrlStorageKey())
-    }
-    return origin
+    return LocalStorage.get(baseUrlStorageKey, origin)
+  }
+  // Helper fonctions to access/alter config used at creation time
+  api.getConfig = function (path) {
+    return (path ? _.get(config, path) : config)
+  }
+  api.setConfig = function (path, value) {
+    _.set(config, path, value)
   }
 
   api.can = function () {
@@ -287,7 +288,8 @@ export function createClient (config) {
 // eg it might be imported before another one updating the config.
 // It is up to the application to instanciate the client when required.
 export let api
-export function initializeApi () {
+export function initializeApi (fn) {
   api = createClient(configuration)
+  if (fn) fn.call(api, configuration)
   return api
 }

package/core/client/capabilities.js

@@ -11,12 +11,16 @@ export const Capabilities = {
     const content = await capabilities.json()
     logger.debug('[KDK] fetched capabilities:', JSON.stringify(content, null, 4))
     this.content = content
+    // Backend might override some defaults in client config
+    _.forOwn(_.pick(content, ['gateway']), (value, key) => {
+      api.setConfig(key, value)
+    })
     // Used to ensure backward compatibility
     Store.set('capabilities.api', content)
     Store.set('capabilities.client', _.pick(config, ['version', 'buildNumber']))
   },
-  get (key) {
+  get (key, defaultValue) {
     if (!this.content) logger.error(new Error('Capabilities must be initialized first'))
-    else return _.get(this.content, key)
+    else return _.get(this.content, key, defaultValue)
   }
 }

package/core/client/components/KContent.vue

@@ -1,7 +1,17 @@
 <template>
   <div>
     <template v-for="component in avaiableComponents" :key="component.uid">
-      <component
+      <Suspense v-if="component.suspense">
+        <component
+          v-if="component.isVisible && !component.isHidden"
+          :is="component.instance"
+          :context="context"
+          :renderer="component.renderer ? component.renderer: actionRenderer"
+          v-bind="component"
+          v-on="component.on ? { [component.on.event]: component.on.listener } : {}"
+          @triggered="onTriggered" />
+      </Suspense>
+      <component v-else
         v-if="component.isVisible && !component.isHidden"
         :is="component.instance"
         :context="context"

package/core/client/components/KDialog.vue

@@ -7,21 +7,23 @@
     :toolbar="toolbar"
     :width-policy="widthPolicy"
   >
-    <!-- component with v-model -->
-    <component
-      v-if="_.has(attrs, 'v-model')"
-      ref="componentRef"
-      :is="computedComponent"
-      v-model="computedModel"
-      v-bind="computedProps"
-    />
-    <!-- component without v-model -->
-    <component
-      v-else
-      ref="componentRef"
-      :is="computedComponent"
-      v-bind="computedProps"
-    />
+    <Suspense>
+      <!-- component with v-model -->
+      <component
+        v-if="_.has(attrs, 'v-model')"
+        ref="componentRef"
+        :is="computedComponent"
+        v-model="computedModel"
+        v-bind="computedProps"
+      />
+      <!-- component without v-model -->
+      <component
+        v-else
+        ref="componentRef"
+        :is="computedComponent"
+        v-bind="computedProps"
+      />
+    </Suspense>
   </KModal>
 </template>
 

package/core/client/components/KModal.vue

@@ -112,7 +112,7 @@ const xlMinWidths = { wide: 90, medium: 55, narrow: 25 }
 const computedStyle = computed(() => {
   // compute the modal max height
   const screenHeight = $q.screen.height
-  const modalMaxHeight = props.maximized ? screenHeight : 0.8 * screenHeight
+  const modalMaxHeight = props.maximized ? screenHeight : 0.9 * screenHeight
   // compute the scroll area max height
   // take into account the header and footer height
   let contentMaxHeight = modalMaxHeight - headerHeight.value

package/core/client/components/KSponsor.vue

@@ -2,7 +2,7 @@
   <div class="column items-center">
     <!-- KDK attribution -->
     <div class="row justify-center items-center">
-      <div>
+      <div class="text-caption">
         {{ $t('KSponsor.KDK_POWERED') }}
       </div>
       <KAction

package/core/client/components/KTextArea.vue

@@ -63,6 +63,10 @@ const props = defineProps({
   dense: {
     type: Boolean,
     default: false
+  },
+  isExpanded: {
+    type: Boolean,
+    default: false
   }
 })
 
@@ -70,7 +74,7 @@ const props = defineProps({
 const scrollArea = ref(null)
 const scrollAreaKey = ref(0)
 const isExpandable = ref(false)
-const isExpanded = ref(false)
+const isExpanded = ref(props.isExpanded)
 const isScrollable = ref(false)
 
 // computed

package/core/client/components/app/KAbout.vue

@@ -74,8 +74,7 @@ const defaultActions = [{
     okAction: 'CLOSE',
     widthPolicy: 'narrow'
   }
-}
-]
+}]
 if (!_.isNil(bugReport.address)) {
   defaultActions.unshift({
     id: 'report-bug',

package/core/client/components/app/KWelcome.vue

@@ -78,6 +78,7 @@ import { ref, computed, onBeforeUnmount } from 'vue'
 import { openURL, useQuasar } from 'quasar'
 import { Store, api } from '../..'
 import { loadComponent } from '../../utils'
+import { LocalStorage } from '../../local-storage.js'
 import KAction from '../KAction.vue'
 
 // Data
@@ -94,11 +95,8 @@ const dense = computed(() => {
 })
 
 // functions
-function getWelcomeKey () {
-  return _.get(config, 'appName').toLowerCase() + '-welcome'
-}
 function show () {
-  const canShow = window.localStorage.getItem(getWelcomeKey())
+  const canShow = LocalStorage.get('welcome')
   // Introduction is only for logged users
   showWelcome.value = (_.isNil(canShow) ? _.get(config, 'layout.welcome', true) : JSON.parse(canShow))
 }
@@ -106,7 +104,7 @@ function hide () {
   showWelcome.value = false
 }
 function onToggleIntroduction (toggle) {
-  window.localStorage.setItem(getWelcomeKey(), JSON.stringify(!toggle))
+  LocalStorage.set('welcome', !toggle)
 }
 function onOnlineHelp () {
   const onlineHelp = _.get(config, 'appOnlineHelp')