-
Notifications
You must be signed in to change notification settings - Fork 21
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
Final PR #48
base: riley/master
Are you sure you want to change the base?
Final PR #48
Conversation
@@ -0,0 +1,57 @@ | |||
class AlbumsController < ApplicationController | |||
before_action :set_album, only: [:edit, :show, :update, :destroy, :upvote] |
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.
Nice use of a before_action. You could also do an except instead of only like so -> before_action :set_album , except: [:index, :create, :new]
Neat job. I'd add /coverage to your .gitignore so you're not uploading the huge files that get generated every time you run rspec and push. I like your use of shared examples, everything looked really consistent between the different medias. 👍 |
I made a thing that looks like the other thing. It has some partials. That's cool.