Move goal-general function from CharacterInventory to GoalActions.

[imnasnainaec]

Twin pr to #922.

A follow-up pr is needed to work the bugs out of these goal-general functions and make sure they are generally useful.

This change is 

[codecov-io]

Codecov Report

Merging #923 (e49dc46) into master (c4f43f8) will not change coverage.
The diff coverage is 68.18%.

@@           Coverage Diff           @@
##           master     #923   +/-   ##
=======================================
  Coverage   50.40%   50.40%           
=======================================
  Files         238      238           
  Lines        6660     6660           
  Branches      425      425           
=======================================
  Hits         3357     3357           
  Misses       2985     2985           
  Partials      318      318           

Flag	Coverage Δ
backend	55.47% <ø> (ø)
frontend	45.26% <68.18%> (ø)

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

Impacted Files	Coverage Δ
...harInventoryCreation/CharacterInventoryActions.tsx	25.00% <50.00%> (-9.85%)	⬇️
src/components/GoalTimeline/GoalsActions.tsx	77.77% <90.00%> (+1.52%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c4f43f8...e49dc46. Read the comment docs.

[jmgrady]

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