Closed setwith closed 1 year ago
Хороша робота, але маю декілька мінорних правок 🙂
[x] файл конфігурації бази даних має називатись config/database.yml.example, через невідповідність назви не вдається створити бд https://www.bigbinary.com/books/learn-rubyonrails-book/configuring-and-connecting-to-a-database також думаю, що варто видалити стрічки з development і test середовища
username: default password: default
(оскільки при створення бд свариться на невідповідність доступів)
[x] варто додати валідацію для ціни продукта, оскільки є можливість створювати продукт з відємною ціною
[x] у view якшо паршал, який відображаєємо знаходиться в тій ж папці, з якої ми посилаємось на нього, не потрібно вказувати повний шлях в даному випадку можна написати <%= render "products_index" %>
[x] думаю, також варто прибрати кнопку назви продукту в на сторінці show, оскільки вона перенаправляє на сторінку, на якій ми вже знаходимось
[x] також можливо візуально буде краще сприйматися сайт, якщо при авторизації не відображати сайдбар
Дякую.
У view для паршл просить повний шлях, інакше не працює
Хороша робота, але маю декілька мінорних правок 🙂
[x] файл конфігурації бази даних має називатись config/database.yml.example, через невідповідність назви не вдається створити бд https://www.bigbinary.com/books/learn-rubyonrails-book/configuring-and-connecting-to-a-database також думаю, що варто видалити стрічки з development і test середовища
(оскільки при створення бд свариться на невідповідність доступів)
[x] варто додати валідацію для ціни продукта, оскільки є можливість створювати продукт з відємною ціною
[x] у view якшо паршал, який відображаєємо знаходиться в тій ж папці, з якої ми посилаємось на нього, не потрібно вказувати повний шлях в даному випадку можна написати <%= render "products_index" %>
[x] думаю, також варто прибрати кнопку назви продукту в на сторінці show, оскільки вона перенаправляє на сторінку, на якій ми вже знаходимось
[x] також можливо візуально буде краще сприйматися сайт, якщо при авторизації не відображати сайдбар