bug (#30): fixes double rendering

Co-authored-by: Prince Wilson <prince.wilson@netlify.com>
netlify-templates · May 24, 2022 · c5650af · c5650af
2 parents 35b3a16 + 1d54452
commit c5650af
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/src/components/AddNotification/AddNotification.js b/src/components/AddNotification/AddNotification.js
@@ -46,8 +46,10 @@ const AddNotification = (props) => {
       </div>
 
       <div className={styles.actionContainer}>
-        <Button level={'secondary'}>view my bag (1)</Button>
-        <Button onClick={() => navigate('/cart')} level={'primary'}>
+        <Button onClick={props.openCart} level={'secondary'}>
+          view my bag (1)
+        </Button>
+        <Button level="primary" href="/cart">
           checkout
         </Button>
         <div className={styles.linkContainer}>

diff --git a/src/components/Header/Header.js b/src/components/Header/Header.js
@@ -154,10 +154,10 @@ const Header = (prop) => {
               <div className={styles.bagNotification}>
                 <span>1</span>
               </div>
-              <div className={styles.notificationContainer}>
-                <AddNotification />
-              </div>
             </button>
+            <div className={styles.notificationContainer}>
+              <AddNotification openCart={() => setShowMiniCart(true)} />
+            </div>
           </div>
         </div>