Goal loading cleanup; Fix goal history reversal bug. (#925)

Cleanup and refactor: 
* Move currentUser check to within userEditFetch. 
* Remove unused GoalSelectorScroll. 
* Remove useless goal update in CharacterInventoryActions. 
* Replace custom functions getIndexInHistory and findGoalByName with use of .find(). 
* Make consistent usage of async/await with goal actions. 
Bug-fix: Remove .reverse() on the goal history in the state, and only reverse what is given to the history display. Resolves issues #243, #698, #795.

src/components/App/DefaultState.tsx

@@ -4,7 +4,6 @@ import { defaultState as reviewEntriesState } from "../../goals/ReviewEntries/Re
 import { defaultProject } from "../../types/project";
 import { defaultState as loginState } from "../Login/LoginReducer";
 import { defaultState as goalTimelineState } from "../GoalTimeline/DefaultState";
-import { defaultState as goalSelectorState } from "../GoalTimeline/GoalSelectorScroll/GoalSelectorReducer";
 import { defaultState as passwordResetState } from "../PasswordReset/reducer";
 import { defaultState as exportProjectState } from "../ProjectExport/ExportProjectReducer";
 import { defaultState as createProjectState } from "../ProjectScreen/CreateProject/CreateProjectReducer";
@@ -46,9 +45,6 @@ export const defaultState = {
   },
 
   //general cleanup tools
-  goalSelectorState: {
-    ...goalSelectorState,
-  },
   goalsState: {
     ...goalTimelineState,
   },