-
Notifications
You must be signed in to change notification settings - Fork 44
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
tor s - octos - api-muncher #41
base: master
Are you sure you want to change the base?
Conversation
…rapper - hoping this will resolve heroku issue also
If it's not too much to ask, could mine please, please be looked at later? I really didn't get to finish/polish up things and I wasn't the most comfortable submitting what I had. Please and thank you! |
|
||
post favorites_path, params: recipe_data | ||
|
||
must_respond_with :success |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be a redirect as it redirects back.
<%= csrf_meta_tags %> | ||
</head> | ||
|
||
<body class='grid-y'> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Foundation xy-grid? neat
end | ||
# ======================= | ||
|
||
it 'must return ok for no recipes' do |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe also test the flash notices?
end | ||
|
||
|
||
it 'redirects to root when an error is received' do |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This can be removed
auth_hash = request.env['omniauth.auth'] | ||
|
||
if auth_hash['uid'] | ||
user = User.build_from_google(auth_hash) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🥇
API MuncherWhat We're Looking For
|
API Muncher
Congratulations! You're submitting your assignment!
Comprehension Questions