Skip to content

finished project#7

Open
phyllispp wants to merge 1 commit into
rocketacademy:mainfrom
phyllispp:branchweather
Open

finished project#7
phyllispp wants to merge 1 commit into
rocketacademy:mainfrom
phyllispp:branchweather

Conversation

@phyllispp
Copy link
Copy Markdown

No description provided.

Comment thread src/App.jsx
import "./App.css";
import axios from "axios";
import { useState } from "react";

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Come back and try use a .env to store this credential!

Comment thread src/App.jsx
)
.then((response) => {
const cityGeoData = response.data[0];
return cityGeoData;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

You dont need to make a variable and then return it

.then((response) => response.data[0])
.then((cityGeoData) => ....
)

Comment thread src/App.jsx
{weatherInfo}
{/* Follow the weather app instructions on the gitbook to implement this exercise */}
</div>
</>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

You only have one commit, you should commit and push more so that you have code to fall back on if something goes wrong with your machine.

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