Skip to content

Kevin Riehl Homework #4 - #11

Open
kevriehl wants to merge 1 commit into
FEWD87:masterfrom
kevriehl:master
Open

Kevin Riehl Homework #4#11
kevriehl wants to merge 1 commit into
FEWD87:masterfrom
kevriehl:master

Conversation

@kevriehl

Copy link
Copy Markdown

No description provided.

@bmwertman

Copy link
Copy Markdown

Nice use of .toUpperCase() @kevriehl! Not a requirement for the assignment but in the interest of keeping your code DRY(remember Don't Repeat Yourself?) you could have chained .toUpperCase() on the end of .val() where you set your city variable. Great job though, approved 👍 !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants