Skeleton for Google Drive Save All button

Author: sumomomomomo

src/commons/controlBar/ControlBarGoogleDriveButtons.tsx

@@ -21,6 +21,7 @@ type Props = {
   isDirty?: boolean;
   onClickOpen?: () => any;
   onClickSave?: () => any;
+  onClickSaveAll?: () => any;
   onClickSaveAs?: () => any;
   onClickLogOut?: () => any;
   onClickLogIn?: () => any;
@@ -71,7 +72,7 @@ export const ControlBarGoogleDriveButtons: React.FC<Props> = props => {
     <ControlButton
       label="Save All"
       icon={IconNames.DOUBLE_CHEVRON_UP}
-      onClick={props.onClickSaveAs}
+      onClick={props.onClickSaveAll}
       isDisabled={props.accessToken ? false : true}
     />
   );

src/commons/sagas/PersistenceSaga.tsx

@@ -7,6 +7,7 @@ import { call, put, select } from 'redux-saga/effects';
 import {
   PERSISTENCE_INITIALISE,
   PERSISTENCE_OPEN_PICKER,
+  PERSISTENCE_SAVE_ALL,
   PERSISTENCE_SAVE_FILE,
   PERSISTENCE_SAVE_FILE_AS,
   PersistenceFile
@@ -100,10 +101,30 @@ export function* persistenceSaga(): SagaIterator {
         return;
       }
 
+
+      const confirmOpen: boolean = yield call(showSimpleConfirmDialog, {
+        title: 'Opening from Google Drive',
+        contents: (
+          <p>
+            Opening <strong>{name}</strong> will overwrite the current contents of your workspace.
+            Are you sure?
+          </p>
+        ),
+        positiveLabel: 'Open',
+        negativeLabel: 'Cancel'
+      });
+      if (!confirmOpen) {
+        return;
+      }
+
       // Note: for mimeType, text/plain -> file, application/vnd.google-apps.folder -> folder
 
       if (mimeType === "application/vnd.google-apps.folder") { // handle folders
-        yield call(console.log, "is folder");
+        toastKey = yield call(showMessage, {
+          message: 'Opening folder...',
+          timeout: 0,
+          intent: Intent.PRIMARY
+        });
 
         const fileList = yield call(getFilesOfFolder, id, name); // this needed the extra scope mimetypes to have every file
         // TODO: add type for each resp?
@@ -126,43 +147,30 @@ export function* persistenceSaga(): SagaIterator {
 
 
         for (const currFile of fileList) {
-          // TODO add code to actually load contents here
           const contents = yield call([gapi.client.drive.files, 'get'], { fileId: currFile.id, alt: 'media' });
           yield call(writeFileRecursively, fileSystem, "/playground" + currFile.path, contents.body);
         }
 
-
-
-
-
-
-
-
-
-
-
-
+        // set source to chapter 4 TODO is there a better way of handling this
+        yield put(
+          actions.chapterSelect(
+            parseInt('4', 10) as Chapter, 
+            Variant.DEFAULT,
+            'playground'
+          )
+        );
+        // open folder mode
+        yield call(store.dispatch, actions.setFolderMode("playground", true));
 
         // refresh needed
-        yield put(store.dispatch(actions.removeEditorTabsForDirectory("playground", "/"))); // deletes all active tabs
-        yield call(refreshFileView); // refreshes folder view TODO super jank
-
-        return;
-      }
+        yield call(store.dispatch, actions.removeEditorTabsForDirectory("playground", "/")); // deletes all active tabs
+        // TODO find a file to open instead of deleting all active tabs?
+        // TODO without modifying WorkspaceReducer in one function this would cause errors - called by onChange of Playground.tsx?
+        // TODO change behaviour of WorkspaceReducer to not create program.js every time folder mode changes with 0 tabs existing?
+        yield call(refreshFileView); // refreshes folder view TODO super jank?
 
+        yield call(showSuccessMessage, `Loaded folder ${name}.`, 1000);
 
-      const confirmOpen: boolean = yield call(showSimpleConfirmDialog, {
-        title: 'Opening from Google Drive',
-        contents: (
-          <p>
-            Opening <strong>{name}</strong> will overwrite the current contents of your workspace.
-            Are you sure?
-          </p>
-        ),
-        positiveLabel: 'Open',
-        negativeLabel: 'Cancel'
-      });
-      if (!confirmOpen) {
         return;
       }
 
@@ -335,6 +343,13 @@ export function* persistenceSaga(): SagaIterator {
     }
   });
 
+  yield takeEvery(
+    PERSISTENCE_SAVE_ALL,
+    function* () {
+      yield console.log("pers save all!");
+    }
+  );
+
   yield takeEvery(
     PERSISTENCE_SAVE_FILE,
     function* ({ payload: { id, name } }: ReturnType<typeof actions.persistenceSaveFile>) {

src/commons/workspace/WorkspaceReducer.ts

@@ -746,6 +746,11 @@ const oldWorkspaceReducer: Reducer<WorkspaceManagerState, SourceActionType> = (
         throw new Error('Editor tab index must be non-negative!');
       }
       if (editorTabIndex >= state[workspaceLocation].editorTabs.length) {
+        // TODO WHY DOES THIS GET CALLED????????????????????? its from Playground.tsx onChange?
+        if (editorTabIndex === 0) {
+          console.log("Warning: editorTabIndex = 0");
+          return {...state};
+        }
         throw new Error('Editor tab index must have a corresponding editor tab!');
       }
 

src/features/persistence/PersistenceActions.ts

@@ -3,13 +3,16 @@ import { createAction } from '@reduxjs/toolkit';
 import {
   PERSISTENCE_INITIALISE,
   PERSISTENCE_OPEN_PICKER,
+  PERSISTENCE_SAVE_ALL,
   PERSISTENCE_SAVE_FILE,
   PERSISTENCE_SAVE_FILE_AS,
   PersistenceFile
 } from './PersistenceTypes';
 
 export const persistenceOpenPicker = createAction(PERSISTENCE_OPEN_PICKER, () => ({ payload: {} }));
 
+export const persistenceSaveAll = createAction(PERSISTENCE_SAVE_ALL, () => ({ payload: {} }));
+
 export const persistenceSaveFile = createAction(PERSISTENCE_SAVE_FILE, (file: PersistenceFile) => ({
   payload: file
 }));

src/features/persistence/PersistenceTypes.ts

@@ -1,4 +1,5 @@
 export const PERSISTENCE_OPEN_PICKER = 'PERSISTENCE_OPEN_PICKER';
+export const PERSISTENCE_SAVE_ALL = 'PERSISTENCE_SAVE_ALL';
 export const PERSISTENCE_SAVE_FILE_AS = 'PERSISTENCE_SAVE_FILE_AS';
 export const PERSISTENCE_SAVE_FILE = 'PERSISTENCE_SAVE_FILE';
 export const PERSISTENCE_INITIALISE = 'PERSISTENCE_INITIALISE';

src/pages/playground/Playground.tsx

@@ -76,6 +76,7 @@ import {
   persistenceInitialise,
   persistenceOpenPicker,
   persistenceSaveFile,
+  persistenceSaveAll,
   persistenceSaveFileAs
 } from 'src/features/persistence/PersistenceActions';
 import {
@@ -582,7 +583,7 @@ const Playground: React.FC<PlaygroundProps> = props => {
     [handleReplEval, isRunning, selectedTab]
   );
 
-  // Compute this here to avoid re-rendering the button every keystroke
+  // Compute this here to avoid re-rendering the button every keystroke 
   const persistenceIsDirty =
     persistenceFile && (!persistenceFile.lastSaved || persistenceFile.lastSaved < lastEdit);
   const persistenceButtons = useMemo(() => {
@@ -595,6 +596,7 @@ const Playground: React.FC<PlaygroundProps> = props => {
         accessToken={googleAccessToken}
         key="googledrive"
         onClickSaveAs={() => dispatch(persistenceSaveFileAs())}
+        onClickSaveAll={() => dispatch(persistenceSaveAll())}
         onClickOpen={() => dispatch(persistenceOpenPicker())}
         onClickSave={
           persistenceFile ? () => dispatch(persistenceSaveFile(persistenceFile)) : undefined