Merge pull request #165 from denis1011101/feature/training-blocks-account-section

Feature/training blocks account section

Author: denis1011101

app/services/telegram/helpers/game_formatting.rb

@@ -95,9 +95,17 @@ def self.training_program(game, limit: 600, locale: Telegram::I18n::DEFAULT_LOCA
 
         text = kept.join(", ")
         dropped = titles.size - kept.size
-        return text if dropped.zero?
+        text = "#{text} #{Telegram::I18n.t(:program_more, locale: locale, count: dropped)}" if dropped.positive?
+        text = "#{text} #{Telegram::I18n.t(:program_details_hint, locale: locale)}" if details?(game)
 
-        "#{text} #{Telegram::I18n.t(:program_more, locale: locale, count: dropped)}"
+        text
+      end
+
+      # Подробное описание блока в сообщение не тянем — оно бывает на 500
+      # символов. Но человек должен знать, что оно есть и где искать: ссылка на
+      # игру идёт следом.
+      def self.details?(game)
+        game.game_training_blocks.any? { |entry| entry.training_block&.description.to_s.strip.present? }
       end
 
       def self.resolve_date(game)

app/views/training_blocks/index.html.erb

@@ -3,7 +3,7 @@
 <div class="mx-auto max-w-3xl space-y-4 px-4 py-6">
   <div class="flex items-center justify-between">
     <h1 class="text-2xl font-bold"><%= t("training_blocks.title") %></h1>
-    <%= link_to t("training_blocks.back_to_games"), games_account_path, class: "text-sm text-indigo-600 hover:underline dark:text-indigo-400" %>
+    <%= link_to t("common.back_to_account"), edit_account_path, class: "text-sm text-indigo-600 hover:underline dark:text-indigo-400" %>
   </div>
 
   <p class="text-sm text-gray-500 dark:text-slate-400"><%= t("training_blocks.hint") %></p>

app/views/users/edit.html.erb

@@ -36,6 +36,17 @@
       <%= t("users.edit.security") %>
     <% end %>
 
+    <%# Библиотека блоков — отдельный раздел кабинета: тренеру она нужна всегда, организатору — если блоки уже заведены. %>
+    <% if current_user.coach? || current_user.training_blocks.exists? %>
+      <%= link_to training_blocks_path,
+            class: "inline-flex items-center gap-2 rounded-md bg-indigo-600 px-4 py-2 text-sm font-medium text-white hover:bg-indigo-700 transition" do %>
+        <svg class="h-4 w-4" fill="none" stroke="currentColor" stroke-width="2" viewBox="0 0 24 24">
+          <path stroke-linecap="round" stroke-linejoin="round" d="M9 6.75h9M9 12h9M9 17.25h9M4.5 6.75h.008v.008H4.5V6.75zm0 5.25h.008v.008H4.5V12zm0 5.25h.008v.008H4.5v-.008z"/>
+        </svg>
+        <%= t("training_blocks.title") %>
+      <% end %>
+    <% end %>
+
     <%= link_to courts_account_path,
           class: "inline-flex items-center gap-2 rounded-md bg-indigo-600 px-4 py-2 text-sm font-medium text-white hover:bg-indigo-700 transition" do %>
       <svg class="h-4 w-4" fill="none" stroke="currentColor" stroke-width="2" viewBox="0 0 24 24">

app/views/users/games.html.erb

@@ -35,19 +35,6 @@
     </section>
   <% end %>
 
-  <%# Конструктор тренировок нужен тренеру рядом с расписанием, но библиотека есть и у организатора. %>
-  <% if current_user.coach? || current_user.training_blocks.exists? %>
-    <section class="rounded-lg border bg-white p-4 dark:border-white/10 dark:bg-slate-800/80">
-      <div class="flex items-center justify-between gap-3">
-        <div>
-          <h2 class="text-sm font-semibold text-gray-700 dark:text-slate-300"><%= t("training_blocks.title") %></h2>
-          <p class="mt-1 text-sm text-gray-500 dark:text-slate-400"><%= t("training_blocks.hint") %></p>
-        </div>
-        <%= link_to t("training_blocks.manage"), training_blocks_path, class: "shrink-0 text-sm text-indigo-600 hover:underline dark:text-indigo-400" %>
-      </div>
-    </section>
-  <% end %>
-
   <% if @games.empty? %>
     <p class="text-gray-500"><%= t("users.games.empty") %></p>
   <% else %>

config/locales/telegram.yml

@@ -133,6 +133,7 @@ ru:
       на корте %{court}'
     program_label: 'Программа: %{items}'
     program_more: '+ ещё %{count}'
+    program_details_hint: '(подробное описание — на странице игры)'
     participants_label: 'Участники:'
     prebooking_request: 'Запрос на предбронирование игры #%{game_id} от %{name}'
     dates_label: 'Даты: %{dates}'
@@ -706,6 +707,7 @@ en:
       on %{court}'
     program_label: 'Programme: %{items}'
     program_more: '+ %{count} more'
+    program_details_hint: '(full description on the game page)'
     participants_label: 'Participants:'
     prebooking_request: 'Prebooking request for Game #%{game_id} from %{name}'
     dates_label: 'Dates: %{dates}'
@@ -1237,6 +1239,7 @@ es:
       a las %{time} en %{court}'
     program_label: 'Programa: %{items}'
     program_more: '+ %{count} más'
+    program_details_hint: '(descripción detallada en la página del partido)'
     participants_label: 'Participantes:'
     prebooking_request: 'Solicitud de reserva previa para el partido #%{game_id} de
       %{name}'

config/locales/web.en.yml

@@ -762,8 +762,6 @@ en:
   training_blocks:
     title: "Training constructor"
     hint: "Session blocks live in your library: fill one in once and just tick it in the next training."
-    manage: "Open"
-    back_to_games: "Back to my games"
     new_title: "New block"
     create: "Add"
     save: "Save"

config/locales/web.ru.yml

@@ -771,8 +771,6 @@ ru:
   training_blocks:
     title: "Конструктор тренировок"
     hint: "Блоки занятия сохраняются в вашей библиотеке: заполнили один раз — дальше просто отмечаете их в тренировке."
-    manage: "Открыть"
-    back_to_games: "К моим играм"
     new_title: "Новый блок"
     create: "Добавить"
     save: "Сохранить"

config/routes.rb

@@ -49,7 +49,7 @@
     post :dismiss_onboarding
   end
 
-  # Библиотека блоков тренировок живёт в личном кабинете рядом с расписанием.
+  # Библиотека блоков тренировок — отдельный раздел личного кабинета.
   resources :training_blocks, only: %i[index create update destroy], path: "account/training_blocks"
 
   resources :users, only: [ :index, :show ] do

docs/notification-examples.md

@@ -15,6 +15,7 @@
 - Корт назван отдельной строкой «Корт: …»: «куда ехать» — первый вопрос, и раньше ответ на него был только по ссылке. В телеграме имя корта — ссылка на его страницу; так и в приглашении, и в напоминании, поэтому оба уходят с `parse_mode: HTML`, а всё, что ввели люди, экранируется (`Telegram::Helpers::Markup`). В письме то же имя идёт без разметки: `@body` вставляется и в текстовый шаблон тоже.
 - Свободные места считаются только для игры. На тренировку на один корт выходит хоть десять человек, и «0 мест свободно» читалось бы как «не приходи».
 - План занятия идёт строкой «Программа: …» — названия блоков через запятую. Длинный список режется по целым блокам, хвост считает пропущенные: «+ ещё 7».
+- Если у блоков заполнено подробное описание, в конце строки появляется «(подробное описание — на странице игры)». Само описание в сообщение не тянем: оно бывает на 500 символов, а ссылка на игру идёт следом.
 - Имя человека зависит от канала: в телеграме `@ник` (живое упоминание), в письме — имя и почта.
 - Превью ссылки в телеграме выключено: оно рисовалось из кэша телеграма и показывало чужой корт и чужую дату.
 

test/controllers/users_controller_test.rb

@@ -407,6 +407,32 @@ class UsersControllerTest < ActionDispatch::IntegrationTest
     user&.destroy
     court&.destroy
   end
+  test "account page links to the training blocks library for a coach" do
+    coach = User.create!(email: "coach-library-#{SecureRandom.hex(4)}@example.com", coach: true)
+    post session_url, params: { email: coach.email }
+
+    get edit_account_url
+
+    assert_response :success
+    assert_includes response.body, training_blocks_path
+    assert_includes response.body, "Training constructor"
+  ensure
+    coach&.destroy
+  end
+
+  test "account page hides the training blocks library from a player without blocks" do
+    user_email = "player_library_#{SecureRandom.hex(4)}@example.com"
+    post session_url, params: { email: user_email }
+    user = User.find_by!(email: user_email)
+
+    get edit_account_url
+
+    assert_response :success
+    assert_not_includes response.body, training_blocks_path
+  ensure
+    user&.destroy
+  end
+
   test "coach sees confirmed future dates in account games" do
     coach = User.create!(email: "coach-schedule@example.com", coach: true)
     game = Game.create!(