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

Yordanos & Jess' FarMar Rails #3

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

Conversation

jweeber
Copy link

@jweeber jweeber commented Apr 26, 2016

http://farmar-rails-jy.herokuapp.com/

Pair project - Jess & Yordanos

It's pretty. We think all functionality is there! When you add a sale, it's added in pennies. 💰

farmer

https://trello.com/b/w9HJM483/farmar

ERD:

img_1057

img_1058



Please feel free to use a different markup language if you do not plan to run
<tt>rake doc:app</tt>.
Copy link

Choose a reason for hiding this comment

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

Best practice would be to copy and paste the Readme from the class assignment into this one :)

@hougland
Copy link

hougland commented May 3, 2016

Yayyyyyy 🎉 you did an awesome job! Your routes and layouts are much cleaner than mine were when I did this project.

Tammy and I both ❤️ your gifs and photos.

For the Trello board - I'd advise having another column for "bugs" that you can come back to later if you want.

You're rocking Rails! 🎉 🎉 🎆

:shipit:

@jweeber
Copy link
Author

jweeber commented May 4, 2016

Thanks to both of you for taking the time to review! 💜 Appreciate the feedback.

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.

4 participants