Skip to content

同期メソッドで note_date と note_link も含めて完全同期#1787

Merged
yasulab merged 1 commit into
mainfrom
sync-event-dates-with-note-info
Jan 4, 2026
Merged

同期メソッドで note_date と note_link も含めて完全同期#1787
yasulab merged 1 commit into
mainfrom
sync-event-dates-with-note-info

Conversation

@yasulab
Copy link
Copy Markdown
Member

@yasulab yasulab commented Jan 4, 2026

📝 概要

同じイベントサービスを共有する道場グループ間で、完全な日付情報の同期を実現しました。

🔍 問題の詳細

発見された問題

柏グループの道場で表示日付に不整合がありました:

  • 柏(ID: 23): 2026-01-04 を表示(note_date が優先)
  • 南柏(ID: 112): 2021-05-16 を表示(latest_event_at のみ)
  • 柏の葉(ID: 311): 2021-05-16 を表示(latest_event_at のみ)

原因

sync_event_date メソッドが部分的な同期しか行っていなかった:

# 修正前:2つの情報のみ同期
target_dojo[:latest_event_at]  = source_dojo[:latest_event_at]
target_dojo[:latest_event_url] = source_dojo[:latest_event_url]
# note_date と note_link は同期されない ❌

✅ 解決策

実装内容

sync_event_date メソッドを拡張して、4つの情報すべてを同期:

# 修正後:完全な同期
target_dojo[:latest_event_at]  = source_dojo[:latest_event_at]
target_dojo[:latest_event_url] = source_dojo[:latest_event_url]
target_dojo[:note_date]        = source_dojo[:note_date]  # 新規追加 ✅
target_dojo[:note_link]        = source_dojo[:note_link]  # 新規追加 ✅

📊 効果

柏グループの表示日付

道場名 修正前 修正後
2026-01-04 2026-01-04
南柏 2021-05-16 ❌ 2026-01-04 ✅
柏の葉 2021-05-16 ❌ 2026-01-04 ✅

奈良グループ(既存の同期も改善)

同様に、奈良・生駒・平群の道場も完全な同期により一貫性が向上します。

🎯 対象道場

現在の同期設定:

  • 奈良グループ
    • 奈良(ID: 35)→ 生駒(ID: 36)、平群(ID: 294)
  • 柏グループ
    • 柏(ID: 23)→ 南柏(ID: 112)、柏の葉(ID: 311)

🧪 テスト

  • Rails runner で柏グループの同期を検証
  • すべての道場で同じ日付(2026-01-04)が表示されることを確認
  • ビューでの表示ロジックと整合性があることを確認

💡 レビューポイント

  • 完全な情報同期が適切か
  • パフォーマンスへの影響は問題ないか
  • 他のグループも同様の同期が必要か

📚 関連PR

📚 補足

柏道場のnoteには「柏・南柏・柏の葉が、合同でイベント管理 (2026-01-04)」と記載があり、実際に3つの道場が協力して活動していることが確認できます。この修正により、その実態が正しく反映されるようになります。

同じイベントサービスを共有する道場間で、より完全な情報同期を実現しました。

問題:
- 柏道場は note_date(2026-01-04)で表示されるが、
- 南柏・柏の葉は latest_event_at(2021-05-16)で表示される不整合

原因:
- sync_event_date が latest_event_at/url のみ同期
- note_date/link は同期されていなかった

解決:
- note_date と note_link も同期するように修正
- これにより、同じグループの道場で同じ日付が表示される

適用道場:
- 奈良グループ:奈良、生駒、平群
- 柏グループ:柏、南柏、柏の葉
@yasulab yasulab merged commit ca071b4 into main Jan 4, 2026
5 checks passed
@yasulab yasulab deleted the sync-event-dates-with-note-info branch January 4, 2026 08:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant