Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Food Tunes #4

Open
wants to merge 46 commits into
base: master
Choose a base branch
from
Open

Food Tunes #4

wants to merge 46 commits into from

Conversation

jweeber
Copy link

@jweeber jweeber commented May 23, 2016

All requirements completed. I feel like my code is fairly clean and readable and I'm really proud of that. I'm also really proud of pushing through and finishing despite the really rough previous week with this.

🎶 http://food-tunes.herokuapp.com/ 🍕

https://trello.com/b/fquzfaPD/food-tunes

jweeber added 30 commits May 16, 2016 15:26
…ed wrapper to a class because it sort of makes more sense
@@ -26,7 +26,7 @@
/tmp/

# Used by dotenv library to load environment variables.
# .env
.env

Choose a reason for hiding this comment

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

Not sure If you've had to deal with this, but it's a pain if you accidentally commit something to github that should be secret. So, nice job!

@rmtolmach
Copy link

Hey there,

Nice job overall! Working with APIs can be tricky but it looks like you had a good grasp on this project.

Well done!
-Becca

@jweeber
Copy link
Author

jweeber commented May 27, 2016

Thanks, Becca! This was really tough, but I definitely understand APIs more now.

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