refactor: Reports code clean 10-29

[lyndsiWilliams]

SUMMARY

Elizabeth and I paired to come up with this solution for cleaning up the state racing issue. This should work, the actions/modal just need to be lined up with the new state structure:

reports: {
  dashboards: {
    [dashboardId]: {...reportObject}
  },
  charts: {
    [chartId]: {...reportObject}
  },
}

ADDITIONAL INFORMATION

• Has associated issue:
• Required feature flags:
• Changes UI
• Includes DB Migration (follow approval process in SIP-59)
  • Migration is atomic, supports rollback & is backwards-compatible
  • Confirm DB migration upgrade and downgrade tested
  • Runtime estimates and downtime expectations provided
• Introduces new feature or API
• Removes existing feature or API

[geido]

LGTM!

[geido]

Suggested change

	// state.users.filter(item => item.id !== action.payload)
	// return {
	// ...state.filter(report => report.id !== action.reportId),
	// };