Skip to content

Comments

Redux Toolkit Eval#21

Open
meccacodes wants to merge 13 commits intoprojectshft:mainfrom
meccacodes:main
Open

Redux Toolkit Eval#21
meccacodes wants to merge 13 commits intoprojectshft:mainfrom
meccacodes:main

Conversation

@meccacodes
Copy link

This was a frustrating one for me. I got it working the way I wanted and then made a mistake merging one branch into another and lost a bunch of work. And this is after spending tons of extra time and days on a feature that wasn't even a requirement. Anywayyyyy sorry for the spaghetti code in advance dear evaluator.

zip: geoZip,
};

return { weatherData: weatherResponse.data, city: `${city}, ${country}` };

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

irrelevant line of code, this is unaccessible.

<br></br>
<h3>Temperature</h3>
<Sparklines
data={data.list ? data.list.map((item) => item.main.temp) : []}

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what is this for?
its creating the empty space

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.

2 participants