Task 7126 #4

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
2 participants
Member

bonyiii commented Mar 22, 2013

No description provided.

@trekdemo trekdemo commented on the diff Mar 22, 2013

app/views/companies/_settings.html.erb
@@ -13,6 +13,13 @@
</div>
<div class="control-group">
+ <label for="company_show_wiki"><%=_ 'Allow Resources' %></label>
@trekdemo

trekdemo Mar 22, 2013

for=company_allow_resources

@trekdemo trekdemo commented on the diff Mar 22, 2013

spec/models/user_spec.rb
@@ -80,6 +80,31 @@
t.reload.creator.should == @user.reload
end
end
+
+ describe "Use resources" do
+ subject{ FactoryGirl.create(:company_with_admin) }
@trekdemo

trekdemo Mar 22, 2013

Please use the subject only for user instances. And create a variable for the company.

let(:company) { FactoryGirl.create(:company_with_admin) }
subject { company.admin }

@trekdemo trekdemo commented on the diff Mar 22, 2013

db/schema.rb
@@ -32,6 +32,7 @@
t.string "logo_content_type"
t.integer "logo_file_size"
t.datetime "logo_updated_at"
+ t.boolean "allow_resources", :default => true
@trekdemo

trekdemo Mar 22, 2013

Please rename it to use_resources for clarity.

trekdemo closed this Mar 22, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment