Feature/add only necessary default boards on login

src/api/communicators.json

@@ -6,6 +6,7 @@
     "author": "Cboard Team",
     "email": "support@cboard.io",
     "rootBoard": "root",
-    "boards": ["root"]
+    "boards": ["root"],
+    "defaultBoardBlackList": []
   }
 ]

src/components/Account/Login/Login.actions.js

@@ -15,6 +15,7 @@ import {
 } from '../../App/App.actions';
 import { getVoiceURI } from '../../../i18n';
 import { isCordova, isElectron } from '../../../cordova-util';
+import { isRemoteIdChecker } from '../../../helpers';
 
 export function loginSuccess(payload) {
   return dispatch => {
@@ -169,13 +170,12 @@ export function login({ email, password, activatedData }, type = 'local') {
           .map(async id => {
             let board = null;
             try {
-              board = await API.getBoard(id);
+              if (isRemoteIdChecker(id)) board = await API.getBoard(id);
             } catch (e) {}
             return board;
           })
           .filter(b => b !== null)
       );
-
       dispatch(addBoards(apiBoards));
       if (type === 'local') {
         dispatch(

src/components/Board/Board.actions.js

@@ -38,7 +38,8 @@ import {
   DOWNLOAD_IMAGES_FAILURE,
   DOWNLOAD_IMAGES_STARTED,
   DOWNLOAD_IMAGE_SUCCESS,
-  DOWNLOAD_IMAGE_FAILURE
+  DOWNLOAD_IMAGE_FAILURE,
+  REMOVE_BOARDS_FROM_LIST
 } from './Board.constants';
 
 import API from '../../api';
@@ -51,7 +52,8 @@ import {
   upsertApiCommunicator,
   updateDefaultBoardsIncluded,
   addDefaultBoardIncluded,
-  verifyAndUpsertCommunicator
+  verifyAndUpsertCommunicator,
+  concatDefaultBoardIdToBlacklist
 } from '../Communicator/Communicator.actions';
 import { isAndroid, writeCvaFile } from '../../cordova-util';
 import { DEFAULT_BOARDS } from '../../helpers';
@@ -151,10 +153,14 @@ export function changeDefaultBoard(selectedBoardNameOnJson) {
 
     const switchActiveBoard = homeBoardId => {
       if (homeBoardId) {
+        const storeBoards = getState().board.boards;
+        const board = storeBoards.find(board => board.id === homeBoardId);
+        if (!board) return null;
         const goTo = `/board/${homeBoardId}`;
 
         dispatch(switchBoard(homeBoardId));
         history.replace(goTo);
+        return true;
       }
     };
 
@@ -198,9 +204,7 @@ export function changeDefaultBoard(selectedBoardNameOnJson) {
       homeBoardId
     });
 
-    switchActiveBoard(homeBoardId);
-
-    replaceHomeBoard(homeBoardId);
+    if (switchActiveBoard(homeBoardId)) replaceHomeBoard(homeBoardId);
   };
 }
 
@@ -267,8 +271,19 @@ export function previousBoard() {
 }
 
 export function toRootBoard() {
-  return {
-    type: TO_ROOT_BOARD
+  return (dispatch, getState) => {
+    const navHistory = getState().board.navHistory;
+    const firstBoardOnHistory = navHistory[0];
+    const allBoardsIds = getState().board.boards.map(board => board.id);
+
+    if (!firstBoardOnHistory || !allBoardsIds.includes(firstBoardOnHistory)) {
+      return null;
+    }
+    history.replace(firstBoardOnHistory);
+    dispatch({
+      type: TO_ROOT_BOARD
+    });
+    return firstBoardOnHistory;
   };
 }
 
@@ -515,7 +530,8 @@ export function createApiBoard(boardData, boardId) {
       isPublic: false
     };
     return API.createBoard(boardData)
-      .then(res => {
+      .then(async res => {
+        await dispatch(concatDefaultBoardIdToBlacklist(boardId));
         dispatch(createApiBoardSuccess(res, boardId));
         return res;
       })
@@ -821,3 +837,19 @@ export function updateApiObjects(
       });
   };
 }
+
+export function removeBoardsFromList(blacklist = [], rootBoard) {
+  return (dispatch, getState) => {
+    const actualBoardId = getState().board.activeBoardId;
+    if (blacklist.includes(actualBoardId)) {
+      history.replace(rootBoard);
+      dispatch(switchBoard(rootBoard));
+      const rootBoardFinded = dispatch(toRootBoard());
+      if (!rootBoardFinded || blacklist.includes(rootBoard)) return;
+    }
+    dispatch({
+      type: REMOVE_BOARDS_FROM_LIST,
+      blacklist
+    });
+  };
+}

src/components/Board/Board.constants.js

@@ -3,6 +3,7 @@ export const ADD_BOARDS = 'cboard/Board/ADD_BOARDS';
 export const CREATE_BOARD = 'cboard/Board/CREATE_BOARD';
 export const UPDATE_BOARD = 'cboard/Board/UPDATE_BOARD';
 export const DELETE_BOARD = 'cboard/Board/DELETE_BOARD';
+export const REMOVE_BOARDS_FROM_LIST = 'cboard/Board/REMOVE_BOARDS_FROM_LIST';
 export const CHANGE_BOARD = 'cboard/Board/CHANGE_BOARD';
 export const REPLACE_BOARD = 'cboard/Board/REPLACE_BOARD';
 export const SWITCH_BOARD = 'cboard/Board/SWITCH_BOARD';

src/components/Board/Board.container.js

@@ -70,6 +70,7 @@ import {
   IS_BROWSING_FROM_APPLE_TOUCH,
   IS_BROWSING_FROM_SAFARI
 } from '../../constants';
+import { ALL_DEFAULT_BOARDS, isRemoteIdChecker } from '../../helpers';
 //import { isAndroid } from '../../cordova-util';
 
 const ogv = require('ogv');
@@ -273,9 +274,14 @@ export class BoardContainer extends Component {
 
     if (!boardExists) {
       // try the root board
-      boardExists = boards.find(b => b.id === communicator.rootBoard);
+      const homeBoard = communicator.rootBoard;
+      boardExists = boards.find(b => b.id === homeBoard);
       if (!boardExists) {
-        boardExists = boards.find(b => b.id !== '');
+        if (isRemoteIdChecker(homeBoard))
+          boardExists = this.tryRemoteBoard(homeBoard);
+        if (!boardExists)
+          boardExists = this.addDefaultBoardIfnecessary(homeBoard);
+        if (!boardExists) boardExists = boards.find(b => b.id !== '');
       }
     }
     const boardId = boardExists.id;
@@ -868,8 +874,21 @@ export class BoardContainer extends Component {
     };
 
     if (tile.loadBoard) {
+      const loadBoardFinder = loadBoardSearched => {
+        const findBoardOnStore = boardId =>
+          this.props.boards.find(b => b.id === boardId);
+
+        const nextBoard = findBoardOnStore(loadBoardSearched);
+        if (nextBoard) return nextBoard;
+        if (
+          ALL_DEFAULT_BOARDS.map(({ id }) => id).includes(loadBoardSearched)
+        ) {
+          const nextBoard = this.addDefaultBoardIfnecessary(loadBoardSearched);
+          if (nextBoard) return nextBoard;
+        }
+      };
       const nextBoard =
-        boards.find(b => b.id === tile.loadBoard) ||
+        loadBoardFinder(tile.loadBoard) ||
         // If the board id is invalid, try falling back to a board
         // with the right name.
         boards.find(b => b.name === tile.label);
@@ -1511,6 +1530,18 @@ export class BoardContainer extends Component {
       : [];
   };
 
+  addDefaultBoardIfnecessary = boardId => {
+    const { boards, addBoards } = this.props;
+    if (!boards.find(b => b.id === boardId)) {
+      const board = ALL_DEFAULT_BOARDS.find(({ id }) => id === boardId);
+      if (board) {
+        addBoards([board]);
+        return board;
+      }
+      return;
+    }
+  };
+
   render() {
     const {
       navHistory,

src/components/Board/Board.reducer.js

@@ -36,7 +36,8 @@ import {
   GET_API_MY_BOARDS_STARTED,
   DOWNLOAD_IMAGES_STARTED,
   DOWNLOAD_IMAGE_SUCCESS,
-  DOWNLOAD_IMAGE_FAILURE
+  DOWNLOAD_IMAGE_FAILURE,
+  REMOVE_BOARDS_FROM_LIST
 } from './Board.constants';
 import { LOGOUT, LOGIN_SUCCESS } from '../Account/Login/Login.constants';
 
@@ -249,7 +250,13 @@ function boardReducer(state = initialState, action) {
           board => action.boardId.indexOf(board.id) === -1
         )
       };
-
+    case REMOVE_BOARDS_FROM_LIST:
+      return {
+        ...state,
+        boards: state.boards.filter(
+          board => !action.blacklist?.includes(board.id)
+        )
+      };
     case CREATE_TILE:
       return {
         ...state,

src/components/Board/__tests__/Board.actions.test.js

@@ -132,11 +132,21 @@ describe('actions', () => {
     expect(actions.previousBoard()).toEqual(expectedAction);
   });
 
-  it('should create an action to REPLACE_ME', () => {
+  it('should create an action to REPLACE_ME', async () => {
     const expectedAction = {
       type: types.TO_ROOT_BOARD
     };
-    expect(actions.toRootBoard()).toEqual(expectedAction);
+    const store = mockStore({
+      ...initialState,
+      board: {
+        ...initialState.board,
+        navHistory: ['12345678901234567'],
+        boards: [{ ...mockBoard, id: '12345678901234567' }]
+      }
+    });
+    await store.dispatch(actions.toRootBoard());
+    const toRootBoardAction = store.getActions()[0];
+    expect(toRootBoardAction).toEqual(expectedAction);
   });
 
   it('should create an action to REPLACE_ME', () => {
@@ -318,7 +328,10 @@ describe('actions', () => {
           boardId: '12345678901234567',
           type: 'cboard/Board/CREATE_API_BOARD_SUCCESS'
         };
-        expect(actions[1]).toEqual(dataResp);
+        const successAction = actions.find(
+          action => action.type === types.CREATE_API_BOARD_SUCCESS
+        );
+        expect(successAction).toEqual(dataResp);
         expect(data).toEqual(mockBoard);
       })
       .catch(e => {

src/components/Communicator/Communicator.actions.js

@@ -23,8 +23,9 @@ import {
 import { defaultCommunicatorID } from './Communicator.reducer';
 import API from '../../api';
 import shortid from 'shortid';
+import { removeBoardsFromList, switchBoard } from '../Board/Board.actions';
+import { ALL_DEFAULT_BOARDS } from '../../helpers';
 import moment from 'moment';
-import { switchBoard } from '../Board/Board.actions';
 import history from './../../history';
 
 export function importCommunicator(communicator) {
@@ -234,7 +235,7 @@ export function verifyAndUpsertCommunicator(
  */
 
 export function getApiMyCommunicators() {
-  return async dispatch => {
+  return async (dispatch, getState) => {
     dispatch(getApiMyCommunicatorsStarted());
     try {
       const res = await API.getCommunicators();
@@ -245,6 +246,18 @@ export function getApiMyCommunicators() {
         } catch (e) {
           console.error(e);
         }
+        const activeCommunicator =
+          res.data.find(
+            communicator =>
+              communicator.id === getState().communicator.activeCommunicator
+          ) ?? res.data[res.data.length - 1];
+        const defaultBoardBlackList = activeCommunicator?.defaultBoardBlackList;
+        dispatch(
+          removeBoardsFromList(
+            defaultBoardBlackList,
+            activeCommunicator.rootBoard
+          )
+        );
       }
 
       return res;
@@ -384,3 +397,27 @@ export function syncCommunicators(remoteCommunicators) {
     }
   };
 }
+
+export function concatDefaultBoardIdToBlacklist(boardId) {
+  const getActiveCommunicator = getState => {
+    return getState().communicator.communicators.find(
+      c => c.id === getState().communicator.activeCommunicatorId
+    );
+  };
+  return (dispatch, getState) => {
+    if (!ALL_DEFAULT_BOARDS.map(({ id }) => id).includes(boardId)) return;
+    const updatedCommunicatorData = { ...getActiveCommunicator(getState) };
+
+    const concatBoardIdIfNecessary = () => {
+      if (!updatedCommunicatorData?.defaultBoardBlackList.includes(boardId))
+        return updatedCommunicatorData?.defaultBoardBlackList.concat(boardId);
+      return updatedCommunicatorData?.defaultBoardBlackList;
+    };
+
+    updatedCommunicatorData.defaultBoardBlackList = updatedCommunicatorData?.defaultBoardBlackList
+      ? concatBoardIdIfNecessary()
+      : [boardId];
+
+    return dispatch(verifyAndUpsertCommunicator(updatedCommunicatorData));
+  };
+}

src/helpers.js

@@ -5,6 +5,16 @@ export const DEFAULT_BOARDS = {
   advanced: boards.advanced,
   picSeePal: picSeePal
 };
+let advancedCopy = JSON.parse(JSON.stringify(DEFAULT_BOARDS.advanced));
+let picSeePalCopy = JSON.parse(JSON.stringify(DEFAULT_BOARDS.picSeePal));
+
+export const ALL_DEFAULT_BOARDS = [...advancedCopy, ...picSeePalCopy];
+
+export const SHORT_ID_MAX_LENGTH = 14;
+
+export const isRemoteIdChecker = id => {
+  return !(id.length < SHORT_ID_MAX_LENGTH);
+};
 
 export const dataURLtoFile = (dataurl, filename, checkExtension = false) => {
   // https://stackoverflow.com/a/38936042