Skip to content

Comments

RTK-Weather #30

Open
mstwssts wants to merge 12 commits intoprojectshft:mainfrom
mstwssts:main
Open

RTK-Weather #30
mstwssts wants to merge 12 commits intoprojectshft:mainfrom
mstwssts:main

Conversation

@mstwssts
Copy link

Had to take a long break unfortunately. Hope this doesn't affect me finishing the course.

I made my folders and files for what I will be needing. Plus added my .env.local files for my api key which I will try to hide when it comes to submitting.
I made the store and the provider, no reducer yet
main skeleton/ design
we added the logic for the API call using axios and createAsyncThunk as well as adding extraReducers to update slice
Forgot to commit my last section as I am running into some problems but will explain in next commit
Well I fixed some error but now i see that when I run my server I see nothing. Will have to make sure everything is linked up correctly, could be that there is also no css.
in the file SearchBar I had put cityName insteatd of state which I problably should reverse but thats fine for now
fixed some name errors. I wanted to change some names and i got mixed up with where I put them for a bit But I am now all organized and will continue with the app.
After a very long time I looked at all the naming mistakes in the entire app and was able to get it working. Now using Redux/RTK we can type a city name and get 3 charts back using React SparkLines.
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