Feedback HW 12

[setwith]

Хороша робота! Проте можна застосувати декілька мінорних правок)

• [x] Можна добавити annotate gem, для зручності розуміння взаємозв'язків між моделями

• [x] Варто додати в модель Сart has_one :order, dependent: :destroy , адже ордер не можу існувти без корзини. з поточною логікою не можна видалити корзину

  

  Такий ж звязок в моделі category

• [x] #21

відповідно можна також спростити код тут https://github.com/alinajuly/RubyHW/blob/095322b7a4ed845b63e8761ea9d33ecac3a40537/HW12/shop/app/views/orders/show.html.erb#L12

• [x] Для кращої читабельності, я б радив не додавати класи в application.html.erb, залишити лише:

  <%= render partial: "layouts/template/sidebar" %>
  <%= render partial: "layouts/template/navbar" %>
  <%= render partial: "layouts/template/flash_message" %>
  <%= yield %>

• [x] На ідекс сторінці продуктів айтеми накладаються один на одну

  

  виправити це можна змінивши код в _products_index на такий:

  <div class='container'>
  <div class='row g-5 d-flex justify-content-center align-items-center vh-100'>
    <% @products.each do |product| %>
      <div class='col-auto'>
        <div class='card text-bg-dark' style='width: 18rem'>
          <img src=<%= product.image %> class="card-img-top" alt=<%= product.name %>>
          <div class='card-body'>
            <h5 class='card-title text-center'><%= link_to product.name, product %></h5>
            <p class='card-price text-white'><%= number_to_currency(product.price) %></p>
            <% if current_user %>
              <%= button_to 'Add to cart', line_items_path(product_id: product.id), class: 'btn btn-warning text-end' %>
            <% end %>
          </div>
        </div>
      </div>
    <% end %>
  </div>
  </div>

[alinajuly]

Дякую.

Без класів в application.html.erb некоректно відображаються sidebar та navbar. Прописувала класи в самих файлах, вони не працюють таким чином.