Skip to content

James Tang weather app#8

Open
TangJames wants to merge 1 commit intowdi-atx-12:masterfrom
TangJames:master
Open

James Tang weather app#8
TangJames wants to merge 1 commit intowdi-atx-12:masterfrom
TangJames:master

Conversation

@TangJames
Copy link

flags RIP :(

Copy link

@BritneyJo BritneyJo left a comment

Choose a reason for hiding this comment

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

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 if you had downloaded it: https://cdnjs.com/libraries/flag-icon-css

console.log('I am ready for this stuff');
// eventually this should be triggered by search button
$("form").submit(function(e) {
e.preventDefault();

Choose a reason for hiding this comment

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

watch your indents!

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