SOS #15
SOS #15
Conversation
There was a problem hiding this comment.
Next time you can google for a CDN for the flag library instead of downloading and using node to install. That will eliminate having 500+ files you have to add to the repo: https://cdnjs.com/libraries/flag-icon-css
| console.log("Error: Please input a valid city name."); | ||
| }; | ||
|
|
||
| function clearContents() { |
There was a problem hiding this comment.
You can clear contents just like you set them but instead of giving it text to set, you can set it with an empty string. $(something).text('')
| var temp = (data.main.temp); | ||
| var fahren = (9/5) * (temp - 273) + 32; | ||
| var humid = (data.main.humidity); | ||
| $('.temperature').text(Math.round(fahren) + " degrees Fahrenheit"); |
There was a problem hiding this comment.
Instead of doing all the math, you could have also used the API which requires another parameter to be sent as data in your ajax call 'units=imperial'. Since you are serializing your form to get the data automatically, you could have added another hidden input for this <input type="hidden" name="units" value="imperial">
|
You commit messages make me think you struggled but looking at your code, I'm not sure where you had issues? If you have a specific area that you don't understand, please let us know! But otherwise this hw looks great! |
incomplete and not working.