-
Notifications
You must be signed in to change notification settings - Fork 20
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
Hw/master #20
base: master
Are you sure you want to change the base?
Hw/master #20
Conversation
…l in new_edit categories instead of moving view
@@ -1,13 +1,16 @@ | |||
class Admin::CategoriesController < Admin::BaseController | |||
cache_sweeper :blog_sweeper | |||
|
|||
def index; redirect_to :action => 'new' ; end | |||
def index | |||
redirect_to :action => 'new' ; |
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.
If you refactor this code to be more than one line, then you should no longer use semi-colons
When you create a PR you should include a more meaningful title rather than the name of the branch |
Given the blog is set up | ||
And I am logged into the admin panel | ||
|
||
Scenario: Successfully Add Categories |
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 scenario looks good!
Not finished debugging yet, but starting on cucumber tests for editing categories. Would like feedback on if I fixed previous PR issues correctly : )