Merge pull request #257 from cmu15122/jtromero/fixPersonalStatsPagina…

…tion Fix pagination with unique keys
cmu15122 · Jan 13, 2024 · 7cff516 · 7cff516
2 parents 364acca + 3f03d73
commit 7cff516
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/client/src/components/metrics/AdminMetrics.tsx b/client/src/components/metrics/AdminMetrics.tsx
@@ -103,7 +103,7 @@ export default function AdminMetrics() {
                       .slice(studentPage * rowsPerStudentPage, studentPage * rowsPerStudentPage + rowsPerStudentPage)
                       .map((row) => {
                         return (
-                          <TableRow hover role="checkbox" tabIndex={-1} key={row.andrewId}>
+                          <TableRow hover role="checkbox" tabIndex={-1} key={row.student_andrew}>
                             {studentCols.map((column) => {
                               const value = row[column.id];
                               return (
@@ -149,7 +149,7 @@ export default function AdminMetrics() {
                       .slice(taPage * rowsPerTAPage, taPage * rowsPerTAPage + rowsPerTAPage)
                       .map((row) => {
                         return (
-                          <TableRow hover role="checkbox" tabIndex={-1} key={row.andrewId}>
+                          <TableRow hover role="checkbox" tabIndex={-1} key={row.ta_andrew}>
                             {taCols.map((column) => {
                               const value = row[column.id];
                               return (

diff --git a/client/src/components/metrics/PersonalStats.tsx b/client/src/components/metrics/PersonalStats.tsx
@@ -107,9 +107,9 @@ export default function PersonalStats() {
                 <TableBody>
                   {helpedStudents
                       .slice(page * rowsPerPage, page * rowsPerPage + rowsPerPage)
-                      .map((row) => {
+                      .map((row, i) => {
                         return (
-                          <TableRow hover role="checkbox" tabIndex={-1} key={row.andrewId}>
+                          <TableRow hover role="checkbox" tabIndex={-1} key={row.andrewId + i}>
                             {columns.map((column) => {
                               const value = row[column.id];
                               return (