Clean up keys; Fix up GoalTimeline tests

[imnasnainaec]

Resolves #2679

Removes unnecessary keys
Simplifies unnecessarily complicated keys
Replaces avoidable index-keys
Fixes GoalRedux/GoalTimeline testing

---

This change is 

[codecov-commenter]

Codecov Report

Attention: 4 lines in your changes are missing coverage. Please review.

Files	Coverage Δ
src/components/DataEntry/DataEntryTable/index.tsx	62.53% <100.00%> (ø)
...s/ProjectSettings/ProjectSchedule/CalendarView.tsx	72.72% <ø> (ø)
src/components/ProjectSettings/ProjectSelect.tsx	58.82% <100.00%> (ø)
src/components/Statistics/DomainStatistics.tsx	100.00% <ø> (ø)
src/components/Statistics/UserStatistics.tsx	90.90% <ø> (ø)
...viewEntries/ReviewEntriesComponent/CellColumns.tsx	60.83% <ø> (ø)
...iewEntriesComponent/CellComponents/AlignedList.tsx	100.00% <ø> (ø)
...EntriesComponent/CellComponents/DefinitionCell.tsx	66.66% <100.00%> (ø)
...viewEntriesComponent/CellComponents/DomainCell.tsx	25.80% <100.00%> (ø)
...eviewEntriesComponent/CellComponents/GlossCell.tsx	66.66% <100.00%> (ø)
... and 6 more

... and 44 files with indirect coverage changes

📢 Thoughts on this report? Let us know!.

[jasonleenaylor]

Reviewed 14 of 19 files at r1, 4 of 4 files at r2, 1 of 1 files at r3, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @jmgrady)

---

src/goals/ReviewEntries/ReviewEntriesComponent/CellComponents/NoteCell.tsx line 13 at r3 (raw file):

    <TextField
      variant="standard"
      key={`row-${props.rowData.id}-note`}

So we don't need this even if there is more than one note in the row?

Code quote:

      key={`row-${props.rowData.id}-note`}

[imnasnainaec]

Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @jmgrady)

---

src/goals/ReviewEntries/ReviewEntriesComponent/CellComponents/NoteCell.tsx line 13 at r3 (raw file):

Previously, jasonleenaylor (Jason Naylor) wrote…

So we don't need this even if there is more than one note in the row?

The key (if necessary) will be assigned directly on the <NoteCell> component wherever it is called. And in our case, each row only has one instance of each type of cell anyway (1 per column).

[jasonleenaylor]

Reviewed 1 of 1 files at r4, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @imnasnainaec)