Fix GitHub commit (codesandbox#3553)

MichaelDeBoey · web-flow · commit 179d7f995754 · 2020-02-20T11:10:04.000+01:00
diff --git a/packages/app/src/app/pages/Sandbox/Editor/Workspace/items/GitHub/Git/Git.tsx b/packages/app/src/app/pages/Sandbox/Editor/Workspace/items/GitHub/Git/Git.tsx
@@ -12,7 +12,8 @@ import { WorkspaceInputContainer, WorkspaceSubtitle } from '../../../elements';
 import { Container, Buttons, ErrorMessage, NoChanges } from './elements';
 import { TotalChanges } from './TotalChanges';
 
-const hasWriteAccess = (rights: string) => ['admin', 'write'].includes(rights);
+const hasWriteAccess = (rights: string = '') =>
+  ['admin', 'write'].includes(rights);
 
 export const Git: FunctionComponent = () => {
   const {
@@ -115,7 +116,7 @@ export const Git: FunctionComponent = () => {
                 </WorkspaceInputContainer>
 
                 <Buttons>
-                  {hasWriteAccess(gitChanges.rights) && (
+                  {hasWriteAccess(gitChanges?.rights) && (
                     <Button
                       block
                       disabled={!subject || modulesNotSaved}
diff --git a/packages/app/src/app/pages/Sandbox/Editor/Workspace/screens/GitHub/CommitForm.tsx b/packages/app/src/app/pages/Sandbox/Editor/Workspace/screens/GitHub/CommitForm.tsx
@@ -21,11 +21,11 @@ export const CommitForm = () => {
     },
     state: {
       editor: { isAllModulesSynced },
-      git: { description, originalGitChanges: gitChanges, subject },
+      git: { description, originalGitChanges, subject },
     },
   } = useOvermind();
 
-  const hasWriteAccess = (rights: string) =>
+  const hasWriteAccess = (rights: string = '') =>
     ['admin', 'write'].includes(rights);
 
   const modulesNotSaved = !isAllModulesSynced;
@@ -59,7 +59,7 @@ export const CommitForm = () => {
           />
         </FormField>
         <Stack gap={2}>
-          {hasWriteAccess(gitChanges.rights) && (
+          {hasWriteAccess(originalGitChanges?.rights) && (
             <Button
               variant="secondary"
               disabled={!subject || modulesNotSaved}
