Merge pull request #102 from concord-consortium/188300239-fix-new-tab…

…le-bug 188300239 Fix joining with a new table
concord-consortium · Sep 19, 2024 · 042a2d4 · 042a2d4
2 parents b09d7df + 2a0f015
commit 042a2d4
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/App.tsx b/src/App.tsx
@@ -87,12 +87,12 @@ export default class App extends Component {
     const handleDataContextChange = (event: ChangeEvent<HTMLSelectElement>) => this.handleDataContextChange(event);
     const initiateShare = (selectedContext?: string) => {
       if (selectedContext) this.setState({ selectedDataContext: selectedContext });
-      this.initiateShare()
+      this.initiateShare();
     };
     const joinShare = (selectedContext?: string) => {
       if (selectedContext) this.setState({ selectedDataContext: selectedContext });
-      this.joinShare()
-    }
+      this.joinShare();
+    };
 
     const availableContextOptions = availableDataContexts.map((dc: DataContext) =>
       <option key={dc.name} value={dc.name}>{dc.title ?? dc.name}</option>

diff --git a/src/ui-pages/join-without-merging.tsx b/src/ui-pages/join-without-merging.tsx
@@ -34,7 +34,7 @@ export const JoinWithoutMerging = (props: JoinWithoutMergingProps) => {
         </button>
         <button
           disabled={!joinShareId || (!personalDataLabel && !lastPersonalDataLabel)}
-          onClick={joinShare}>
+          onClick={() => joinShare()}>
             {BEGIN_COLLABORATION}
         </button>
       </div>