search

shakacode / old-react-on-rails-examples

Various example apps for React on Rails, outdated
1 stars 0 forks source link

remove tempTodo #29

Closed Judahmeek closed 7 years ago

Judahmeek commented 7 years ago

After spending a couple hours trying to figure out the best way to address tempTodo's issue, such as worrying about what should happen if a user edited, toggled, or deleted a tempTodo before the server returned a todo creation response or how best to DRY todo & tempTodos actions and reducers, I decided that tempTodo was ultimately a unnecessary distraction and that this project will develop faster and look cleaner with the assumption that the user has continual internet access.

In the morning, I'll also push a follow commit that modifies normalizeTodo into a normalizeObjectToMap function similar to normalizeArrayToMap.

robwise commented 7 years ago

@Judahmeek pls delete your branch after merge