From 4663317ff7a73cc603b25ccb70d33d0f4639429c Mon Sep 17 00:00:00 2001
From: krilklem <rafi.panyotova@gmail.com>
Date: Tue, 16 Dec 2025 14:53:51 +0000
Subject: [PATCH 1/2] fix: Add key to accordion items, add message if no
 feedback has been saved, and fix feedback link name in header

---
 src/App.tsx                        |  2 +-
 src/components/FeedbackGallery.tsx | 27 +++------------------------
 2 files changed, 4 insertions(+), 25 deletions(-)

diff --git a/src/App.tsx b/src/App.tsx
index fe84b86..2308d8d 100644
--- a/src/App.tsx
+++ b/src/App.tsx
@@ -75,7 +75,7 @@ function App() {
         <div className="app-nav__center"> We highlight what matters</div>
         <div className="app-nav__actions">
           {isSignedIn ? (
-            <Link to="/feedback">Past feedback</Link>
+            <Link to="/feedback">Feedback</Link>
           ) : (
             <Link to="/create-user">Join Us</Link>
           )}
diff --git a/src/components/FeedbackGallery.tsx b/src/components/FeedbackGallery.tsx
index 3799057..86f7330 100644
--- a/src/components/FeedbackGallery.tsx
+++ b/src/components/FeedbackGallery.tsx
@@ -54,9 +54,10 @@ export default function Feedback() {
   return (
     <>
       {message && <p>{message}</p>}
+      {allFeedback.length > 0 ? 
       <Accordion type="single" className="w-full" collapsible>
         {allFeedback.map((item, index) => (
-          <AccordionItem value={`feedback-${index}`}>
+          <AccordionItem key={`feedback-${index}`} value={`feedback-${index}`}>
             <AccordionTrigger>{item.job_name}</AccordionTrigger>
             <AccordionContent>
               <ResultCard
@@ -66,29 +67,7 @@ export default function Feedback() {
             </AccordionContent>
           </AccordionItem>
         ))}
-      </Accordion>
-
-      {/* {allFeedback.length > 0 && (
-        <ul>
-          {allFeedback.map((item, index) => (
-            <li key={index}>{item.job_name}
-              <ResultCard
-                jobDescription={item.job_description}
-                feedback={item.gen_feedback}
-              />
-            </li>
-          ))}
-
-          <Accordion type="single" collapsible>
-            <AccordionItem value="item-1">
-              <AccordionTrigger>Is it accessible?</AccordionTrigger>
-              <AccordionContent>
-                Yes. It adheres to the WAI-ARIA design pattern.
-              </AccordionContent>
-            </AccordionItem>
-          </Accordion>
-        </ul>
-      )} */}
+      </Accordion> : <p>You don't have any saved feedback yet...</p>}
     </>
   );
 }

From 3bb5176639f93ad80cf238dd3e9ee4a476ae8184 Mon Sep 17 00:00:00 2001
From: krilklem <rafi.panyotova@gmail.com>
Date: Tue, 16 Dec 2025 14:54:53 +0000
Subject: [PATCH 2/2] chore: Lint

---
 src/components/FeedbackGallery.tsx | 34 ++++++++++++++++++------------
 1 file changed, 20 insertions(+), 14 deletions(-)

diff --git a/src/components/FeedbackGallery.tsx b/src/components/FeedbackGallery.tsx
index 86f7330..8fd694c 100644
--- a/src/components/FeedbackGallery.tsx
+++ b/src/components/FeedbackGallery.tsx
@@ -54,20 +54,26 @@ export default function Feedback() {
   return (
     <>
       {message && <p>{message}</p>}
-      {allFeedback.length > 0 ? 
-      <Accordion type="single" className="w-full" collapsible>
-        {allFeedback.map((item, index) => (
-          <AccordionItem key={`feedback-${index}`} value={`feedback-${index}`}>
-            <AccordionTrigger>{item.job_name}</AccordionTrigger>
-            <AccordionContent>
-              <ResultCard
-                jobDescription={item.job_description}
-                feedback={item.gen_feedback}
-              />
-            </AccordionContent>
-          </AccordionItem>
-        ))}
-      </Accordion> : <p>You don't have any saved feedback yet...</p>}
+      {allFeedback.length > 0 ? (
+        <Accordion type="single" className="w-full" collapsible>
+          {allFeedback.map((item, index) => (
+            <AccordionItem
+              key={`feedback-${index}`}
+              value={`feedback-${index}`}
+            >
+              <AccordionTrigger>{item.job_name}</AccordionTrigger>
+              <AccordionContent>
+                <ResultCard
+                  jobDescription={item.job_description}
+                  feedback={item.gen_feedback}
+                />
+              </AccordionContent>
+            </AccordionItem>
+          ))}
+        </Accordion>
+      ) : (
+        <p>You don't have any saved feedback yet...</p>
+      )}
     </>
   );
 }