Comments 3
а нельзя название класса CreateAddPersonAndGroceryListsAndFoodItemsTables сделать немного короче?
+1
Несколько замечаний по статье:
«Затем в файле app/views/grocery_list/show.rhtml мы создадим соответствующее представление:» — представления нету, исправьте, что ли…
Миграции лучше писать так:
class InitialMigrate < ActiveRecord:: Migration
def self.up
create_table: people do |t|
t.string: name
t.timestamps
end
create_table: grocery_lists do |t|
t.string: name
t.references: person
t.timestamps
end
create_table: food_items do |t|
t.string: name
t.integer: quantity
t.integer: position
t.references: grocery_list
t.timestamps
end
end
def self.down
drop_table: people
drop_table: grocery_lists
drop_table: food_items
end
end
Вместо создания «standard.rhtml» и включения его через layout, лучше создать «application.html.erb», который будет подключатся автоматически по-умолчанию.
Ну и вообще, используйте, всё таки, <code>, пожалуста, а то этот грёбанный чёртов Типограф уродует код.
А в целом, статья отличная, добавил в избранное.
Спасибо.
«Затем в файле app/views/grocery_list/show.rhtml мы создадим соответствующее представление:» — представления нету, исправьте, что ли…
Миграции лучше писать так:
class InitialMigrate < ActiveRecord:: Migration
def self.up
create_table: people do |t|
t.string: name
t.timestamps
end
create_table: grocery_lists do |t|
t.string: name
t.references: person
t.timestamps
end
create_table: food_items do |t|
t.string: name
t.integer: quantity
t.integer: position
t.references: grocery_list
t.timestamps
end
end
def self.down
drop_table: people
drop_table: grocery_lists
drop_table: food_items
end
end
Вместо создания «standard.rhtml» и включения его через layout, лучше создать «application.html.erb», который будет подключатся автоматически по-умолчанию.
Ну и вообще, используйте, всё таки, <code>, пожалуста, а то этот грёбанный чёртов Типограф уродует код.
А в целом, статья отличная, добавил в избранное.
Спасибо.
0
Sign up to leave a comment.
Создание сортируемого списка с перетаскиваемыми элементами