[ReviewEntries] Persist column order/visibility state in project

[imnasnainaec]

Resolves #3305

This change is 

[codecov]

Codecov Report

Attention: Patch coverage is 56.25000% with 14 lines in your changes missing coverage. Please review.

Project coverage is 74.59%. Comparing base (f08ded3) to head (2a12153).
Report is 15 commits behind head on master.

Files with missing lines	Patch %	Lines
src/components/Project/ProjectReducer.ts	9.09%	10 Missing ⚠️
src/components/Project/ProjectActions.ts	50.00%	2 Missing ⚠️
...c/goals/ReviewEntries/ReviewEntriesTable/index.tsx	88.23%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #3309      +/-   ##
==========================================
- Coverage   74.69%   74.59%   -0.11%     
==========================================
  Files         280      280              
  Lines       10745    10773      +28     
  Branches     1297     1299       +2     
==========================================
+ Hits         8026     8036      +10     
- Misses       2353     2370      +17     
- Partials      366      367       +1     

Flag	Coverage Δ
backend	83.81% <ø> (-0.09%)	⬇️
frontend	66.61% <56.25%> (-0.09%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[jasonleenaylor]

src/components/Project/ProjectReducer.ts line 13 at r2 (raw file):

    setColumnOrderAction: (state, action) => {
      const columns = state.reviewEntriesColumns;
      if (typeof action.payload === "function") {

I must have missed it, how could this be something that isn't a function?

[jasonleenaylor]

src/components/Project/ProjectReducer.ts line 13 at r2 (raw file):

Previously, jasonleenaylor (Jason Naylor) wrote…

I must have missed it, how could this be something that isn't a function?

Oh, its because of definition and part-of-speech isn't it. A comment would be good.

[jasonleenaylor]

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

[jasonleenaylor]

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