From 9d8e99400f06e655f16894ede644d260bf4d7e2c Mon Sep 17 00:00:00 2001
From: Tom Moor <tom.moor@gmail.com>
Date: Sun, 5 Apr 2020 13:27:11 -0700
Subject: [PATCH] fix: Various React errors in console

---
 app/components/Sidebar/Main.js                   | 7 +++++--
 app/components/Sidebar/components/Collections.js | 2 ++
 app/scenes/Document/components/Contents.js       | 1 +
 3 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/app/components/Sidebar/Main.js b/app/components/Sidebar/Main.js
index 76302a595..c6919ee49 100644
--- a/app/components/Sidebar/Main.js
+++ b/app/components/Sidebar/Main.js
@@ -45,11 +45,13 @@ class MainSidebar extends React.Component<Props> {
     this.props.documents.fetchDrafts();
   }
 
-  handleCreateCollection = () => {
+  handleCreateCollection = (ev: SyntheticEvent<>) => {
+    ev.preventDefault();
     this.props.ui.setActiveModal('collection-new');
   };
 
-  handleInviteModalOpen = () => {
+  handleInviteModalOpen = (ev: SyntheticEvent<>) => {
+    ev.preventDefault();
     this.inviteModalOpen = true;
   };
 
@@ -145,6 +147,7 @@ class MainSidebar extends React.Component<Props> {
               />
               {can.invite && (
                 <SidebarLink
+                  to="/settings/people"
                   onClick={this.handleInviteModalOpen}
                   icon={<PlusIcon />}
                   label="Invite people…"
diff --git a/app/components/Sidebar/components/Collections.js b/app/components/Sidebar/components/Collections.js
index 254e7d628..e6fa8abec 100644
--- a/app/components/Sidebar/components/Collections.js
+++ b/app/components/Sidebar/components/Collections.js
@@ -66,9 +66,11 @@ class Collections extends React.Component<Props> {
           />
         ))}
         <SidebarLink
+          to="/collections"
           onClick={this.props.onCreateCollection}
           icon={<PlusIcon />}
           label="New collection…"
+          exact
         />
       </Flex>
     );
diff --git a/app/scenes/Document/components/Contents.js b/app/scenes/Document/components/Contents.js
index aef858dac..bd0720241 100644
--- a/app/scenes/Document/components/Contents.js
+++ b/app/scenes/Document/components/Contents.js
@@ -51,6 +51,7 @@ export default function Contents({ document }: Props) {
           <List>
             {headings.map(heading => (
               <ListItem
+                key={heading.slug}
                 level={heading.level}
                 active={activeSlug === heading.slug}
               >