search

arkjoseph / ReactApp

0 stars 0 forks source link

reducer issue #1

Open coryhouse opened 7 years ago

coryhouse commented 7 years ago

Hi Joe - You've declared rate as a root piece of data. So you should likely create a rate reducer that works with that data. If rate is tied to the course, then it should be a property underneath the course object so you can properly handle it in the courseReducer.

arkjoseph commented 7 years ago

Hey Cory, Ive made a commit and replaced Rates with FyRates globally. I am now receiving map undefined with FyRateList which i assume is because I have not configured FyRateReducer. Do you agree or am i missing something more.

arkjoseph commented 7 years ago

in an attempt to fix the map error, i have also added ManageFyRatesPage. Error is still present unless i remove the map function. I will continue to debug

coryhouse commented 7 years ago

Hi Joe - I'd suggest rewatching the Redux flow clip that shows all the moving parts together. You've gotta really understand that flow before you're ready to go creating your own reducers and actions.

arkjoseph commented 7 years ago

Hey Cory- I am def going to re-watch the series. I was able to solve the map error which was stemming from the outer most component page ManageFyRatePage.

Do you mind if we touch on the nested array within the mock api. Just want to get past this initial hurdle of actually pulling in the data.

coryhouse commented 7 years ago

Sorry, I'm unclear - What's your exact question/issue?