RychardHunt / Fyt

Personal Workout Application
2 stars 5 forks source link

[Bug] Fix workout's redux state and reducer #118

Open athammer opened 5 years ago

athammer commented 5 years ago

name: 🐛 Bug Report about: The whole redux state, reducers, and actions for the workout page are a mess. Names that should be the same are not, bad practices like mutating state, and reducers just plain not doing the expected behavior need to be fixed.


Bug Report

Current Behavior Often there is double ghosting for who knows why when saving a new workout changes/deletions are made on both the new workout and the old one even though the reducer is only called once? This is just one issue.

Input Code /reducers/WorkoutReducer.js /actions/WorkoutActions.js

Expected behavior/code stuff should just work

Possible Solution Convert the nested objects to arrays and reamp the code to fix this.