-
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
Bug Fix and Feature Added #17
base: master
Are you sure you want to change the base?
Conversation
…alidation app only validates upon create
@@ -19,6 +19,8 @@ dbfile = File.expand_path("../config/database.yml", __FILE__) | |||
# end | |||
#end | |||
|
|||
gem 'pry-nav', group: [:development, :test] |
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.
Is there an existing development, test group that this gem should be added to?
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.
moved to pre-existing group.
you successfully excluded the db/db_test file but unfortunately the db/db_development file is still in here |
@@ -1,84 +0,0 @@ | |||
class LangaugeFile |
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.
Did you intend to delete this file?
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.
no, I accidently added it to git ignore and ran git rm --cached on it. It's now back in the app and being tracked. I added db/db_development to gitignore and it's no longer being tracked by git. I think this resolves all the issues from this PR. Thanks for the help!!
…ntroller, remove test for article count as it is covered in model spec
Fixed bugs and created associated tests: (1) problem clicking Categories link on dashboard, (2) flash errors not displaying upon create.
Added feature allowing admin users to merge articles. Created cucumber and rspec tests before developing feature. Feature appears to work, but there are currently 3 failing cucumber tests and 2 failing rspec tests.