chore: extend expired time

[takaishi]

No description provided.

[claude]

Claude finished @takaishi's task —— View job

---

コードレビューフィードバック

このPRは、スポンサー担当者招待の有効期限を1日から7日に延長する変更です。

変更内容

1. コントローラー (app/controllers/admin/sponsor_contact_invites_controller.rb:15): 有効期限を1.days.from_nowから7.days.from_nowに変更
2. メールテンプレート (app/views/sponsor_contact_invite_mailer/invite.text.erb:6): メール本文の説明を「24時間有効」から「1週間有効」に変更

レビューポイント

✅ 良い点

• 一貫性のある変更: コードとメール文言の両方が適切に更新されている
• 明確なコメント: 日本語コメントで変更内容が明確に説明されている
• 適切な期限設定: 1週間という期限は、週末を挟んでも対応できる妥当な設定

⚠️ 潜在的な問題点

1. バグの可能性 (app/controllers/admin/sponsor_contact_invites_controller.rb:20):

   flash[:alert] = "#{@invite.email} への招待メール送信に失敗しました"

   @inviteは定義されていません。@sponsor_contact_invite.emailであるべきです。

📋 改善提案

1. 定数化の検討:

   # 例: モデルまたは設定ファイルで
   INVITE_EXPIRATION_DAYS = 7
   
   # コントローラーで
   @sponsor_contact_invite.expires_at = INVITE_EXPIRATION_DAYS.days.from_now

   将来的に期限を変更する際の保守性が向上します。

2. テストの確認:

   • 招待の有効期限が正しく設定されるかのテストケースの追加を推奨
   • 既存のテストで期限に関する検証がある場合は更新が必要

🔒 セキュリティ

• 有効期限の延長自体にセキュリティ上の問題はありません
• 7日間は一般的な招待メールの有効期限として適切です

🚀 パフォーマンス

• この変更によるパフォーマンスへの影響はありません

総評

シンプルで明確な変更ですが、@inviteの未定義変数の問題を修正することを強く推奨します。また、今後の保守性を考慮して、有効期限を定数として定義することも検討してください。

タスクリスト

• CLAUDE.mdファイルを読んでリポジトリ固有の指示を確認する
• 変更されたファイルを分析する
• 関連するテストファイルを確認する
• コードレビューフィードバックを作成する

[github-actions]

Review app

• https://dreamkast-dk-2633.dev.cloudnativedays.jp

[github-actions]

Simplecov Report

Covered	Threshold
65%	60%