Merge pull request #1731 from tomivm/fix/avoid-store-boards-from-user…

…Data

Save only necessary keys on local userData

src/components/App/App.constants.js

@@ -63,3 +63,23 @@ export const APP_LANGS = [
   'zh-CN',
   'zu-ZA' // for crowdin contextual translation
 ];
+
+export const USER_DATA_PROPERTIES = [
+  'id',
+  'google',
+  'facebook',
+  'apple',
+  'name',
+  'role',
+  'provider',
+  'locale',
+  'password',
+  'location',
+  'email',
+  'isFirstLogin',
+  'birthdate',
+  'lastlogin',
+  'createdAt',
+  'updatedAt',
+  'authToken'
+];

src/components/App/App.reducer.js

@@ -7,7 +7,8 @@ import {
   UPDATE_USER_DATA,
   DISABLE_TOUR,
   ENABLE_ALL_TOURS,
-  SET_UNLOGGED_USER_LOCATION
+  SET_UNLOGGED_USER_LOCATION,
+  USER_DATA_PROPERTIES
 } from './App.constants';
 import { LOGIN_SUCCESS, LOGOUT } from '../Account/Login/Login.constants';
 import {
@@ -60,6 +61,15 @@ const initialState = {
   userData: {}
 };
 
+const getKeysFromApiUserDataResponse = payload => {
+  const newUser = {};
+  if (!payload) return newUser;
+  USER_DATA_PROPERTIES.forEach(prop => {
+    if (payload[prop] !== undefined) newUser[prop] = payload[prop];
+  });
+  return newUser;
+};
+
 function appReducer(state = initialState, action) {
   let displaySettings = { ...state.displaySettings };
   let navigationSettings = { ...state.navigationSettings };
@@ -147,7 +157,7 @@ function appReducer(state = initialState, action) {
         isFirstVisit: false,
         displaySettings,
         navigationSettings,
-        userData: action.payload || {}
+        userData: getKeysFromApiUserDataResponse(action.payload)
       };
     case LOGOUT:
       return {
@@ -157,7 +167,7 @@ function appReducer(state = initialState, action) {
     case UPDATE_USER_DATA:
       return {
         ...state,
-        userData: action.userData
+        userData: getKeysFromApiUserDataResponse(action.userData)
       };
     case SET_UNLOGGED_USER_LOCATION:
       return {