Merge pull request #1789 from coderdojo-japan/change-created-at-to-days-passed

feat: 掲載日を経過日に変更 (/dojos/activity)

Author: yasulab

app/views/dojos/activity.html.erb

@@ -18,9 +18,9 @@
     <br>
     <div class='form__terms list'>
       <ul style='list-style-type: "\2713\0020"; font-size: smaller;'>
-	<li>「掲載日」は<%= link_to '統計情報 (道場別) ', dojos_path %> と同じロジックで表示しています。</li>
 	<li>「記録日」は<%= link_to '統計情報 (開催数)',  stats_path %> や活動の確認/記録日を表示しています。</li>
 	<li>「記録日」は <a href='https://doorkeeper.jp/'>Doorkeeper</a> と <a href='http://connpass.com/'>connpass</a> の場合、自動で更新されます。</li>
+	<li>「経過日」は記録日からの経過日数（<%= link_to '閾値', signup_path(anchor: 'terms') %>の確認用）を表示しています。</li>
       </ul>
     </div>
   </p>
@@ -38,16 +38,16 @@
         </th>
         <th>
           <small>
-            🗾
+            🗓
             <br class='ignore-pc'>
-            <%= link_to '掲載日', signup_path %>
+            <a href='<%= events_path %>'>記録日</a>
           </small>
         </th>
         <th>
           <small>
-            🗓
+            ⏱️
             <br class='ignore-pc'>
-            <a href='<%= events_path %>'>記録日</a>
+            <%= link_to '経過日', signup_path(anchor: 'terms') %>
           </small>
         </th>
         <th>
@@ -82,9 +82,6 @@
               <% end %>
             </small>
           </td>
-          <td class="<%= 'inactive-item' unless dojo[:is_active] %>">
-            <small><%= dojo[:created_at].strftime("%Y-%m-%d") %></small>
-          </td>
           <td class="<%= 'inactive-item' unless dojo[:is_active] %>">
             <small>
               <% if dojo[:note_date] && dojo[:latest_event_at] %>
@@ -128,6 +125,33 @@
               <% end %>
             </small>
           </td>
+          <td class="<%= 'inactive-item' unless dojo[:is_active] %>">
+            <small>
+              <% 
+                # 記録日を決定（note_date と latest_event_at のより新しい方）
+                record_date = nil
+                if dojo[:note_date] && dojo[:latest_event_at]
+                  record_date = dojo[:note_date] > dojo[:latest_event_at] ? dojo[:note_date] : dojo[:latest_event_at]
+                elsif dojo[:note_date]
+                  record_date = dojo[:note_date]
+                elsif dojo[:latest_event_at]
+                  record_date = dojo[:latest_event_at]
+                end
+                
+                if record_date
+                  # 経過日数を計算
+                  days_passed = (Date.current - record_date.to_date).to_i
+                  expired = days_passed >= 365 && !dojo[:note]&.include?('Active')
+              %>
+                <span class="<%= 'expired' if expired %>">
+                  <%= days_passed %> 日
+                </span>
+              <% else %>
+                <!-- 記録日がない場合 -->
+                <span style="color: #999;">-</span>
+              <% end %>
+            </small>
+          </td>
           <td class="<%= 'inactive-item' unless dojo[:is_active] %>">
             <small><small>
               <% search_query = URI.encode_www_form(q: "CoderDojo #{dojo[:name].split('@').first.strip}") %>

spec/requests/dojos_spec.rb

@@ -431,15 +431,15 @@
 
     it "displays proper column headers" do
       get activity_dojos_path
-      expect(response.body).to include("掲載日")
+      expect(response.body).to include("経過日")
       expect(response.body).to include("記録日")
       expect(response.body).to include("ノート")
     end
 
-    it "displays created_at date for active dojos" do
+    it "displays days passed for active dojos" do
       get activity_dojos_path
-      # 掲載日は YYYY-MM-DD 形式で表示される
-      expect(response.body).to match(@active_dojo.created_at.strftime("%Y-%m-%d"))
+      # 経過日は日数で表示される
+      expect(response.body).to include("日")
     end
 
     it "displays dojo ID same as /dojos page" do
@@ -644,9 +644,9 @@
       # Extract just the 記録日 column to check the date display
       td_matches = dojo_row.scan(/<td[^>]*>(.*?)<\/td>/m)
 
-      # Based on debug output: td_matches[1] is the 記録日 column
+      # Based on debug output: td_matches[0] is the 記録日 column (列順変更後)
       # (道場名 column seems to be skipped in regex due to complex link structure)
-      event_date_column = td_matches[1]&.first # 記録日 column
+      event_date_column = td_matches[0]&.first # 記録日 column
 
       expect(event_date_column).not_to be_nil, "Could not find 記録日 column"