Skip to content

Redux Weather Eval#68

Open
nilslandsberg wants to merge 9 commits intoprojectshft:masterfrom
nilslandsberg:master
Open

Redux Weather Eval#68
nilslandsberg wants to merge 9 commits intoprojectshft:masterfrom
nilslandsberg:master

Conversation

@nilslandsberg
Copy link

This sucker was TOUGH! Took much longer than I anticipated.

This is not complete - I did not get to work on throwing errors from the API fetch (i.e. "City not found," etc.). However, the app is functional. It certainly needs refinement.

…API Key stored in .env. Folder structure created.
…get the state to add, not replace data when a new search is executed.
…therTableData. Need to implement Sparklines on WeatherTableData.
… create an array to store state on TableWeatherData.
…eated in WeatherTableData. Working on updating cityData state array when the store changes.
…from being added to the cityData array. Table is rendering correctly with Sparklines and average function implemented to find/display average temp, pressure, and humidity.
…ented on citySearch component. Need to find a way to throw errors from the API fetch ('city not found', etc).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant