-
Notifications
You must be signed in to change notification settings - Fork 1
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
Random recipes #45
Merged
Merged
Random recipes #45
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* deleted rando greeting from home page and renamed logo * logoupdated along with file names * working on nav items * navbar looks good working well * removed navbar scss not needed * fixed layout view added favicon
* loginpage prettified * in progress * added oauth buttons to login page * removed seed file * added original seed
Loginroutes
Signup front end
* first ingredients seeded * fixed db errors for ingredients and recipes
* made changes suggested by glebec * Fixed async
* deleted rando greeting from home page and renamed logo * logoupdated along with file names * working on nav items * navbar looks good working well * removed navbar scss not needed * fixed home layout dimensions and added favicon * updated package.json details * updated logo size * set fixed main page vh * updated favicon * navbar view complete * prev commit footer view complete and navbar view now * fixed signup view form * signup login and oauth button views done * about page view set * developmentjs file keys * changed verbiage for signup on login page * updated logo size * corrected user model to allow oauth (#19)
* added allownull falses to recipe database and changed seed file to allow these errors * working on seed file * worked on the seed file without improving it * resolved unique validation errors
* extracted food aisles * user prefs model complete * user prefs model updated * filter recipes by user prefs * associations added for user prefs
* added allownull falses to recipe database and changed seed file to allow these errors * working on seed file * worked on the seed file without improving it * started im * Started... * Minor changes... nbd * added preliminary tests for recipe model * working on tests * added tests for getRelatedMeals
* revert favicon * updated favicon for heroku * submit button on sign up takes you to prefs page * user prefs test running no tests passing * user prefs tests debugging * user prefs tests passing
* extracted food aisles * user prefs model complete * user prefs model updated * filter recipes by user prefs * associations added for user prefs * writing tests for user prefs * user prefs routes * working on user pref routes * third time is the charm?
* updated nav bar to be hamburger menu for small device view * cleaned up tests and instance method
* getMealsWithSimilarIngredients first version * Created tests for getMealsWithSimilarIngredients * Added some comments for geatMealsWithSmiliarIngredients
* get ALL the ingredients (route) * removed extraneous code
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Commented out tests.