Skip to content

Conversation

@YosukeIida
Copy link
Collaborator

@YosukeIida YosukeIida commented Jul 5, 2025

対応Issue

resolve

概要

  • Groupモデルから不要になったcooking_process_order関連のメソッドを削除します。

実装詳細

  • 以下のクラスメソッドをapp/models/group.rbから削除しました。
    • self.with_cooking_process_orders
    • self.with_cooking_process_order
    • self.have_no_cooking_process_order
    • self.with_cooking_process_order_narrow_down_by_fes_year
    • self.with_cooking_process_order_narrow_down_by_search_word

画面スクリーンショット等

テスト項目

  • 関連機能の動作に影響がないこと
  • CIが正常にパスすること

備考

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Groupモデルからcooking_process_order関連の不要メソッドをまとめて削除するPRだよ✨

  • cooking_process_orderに関するクラスメソッド5つを削除
  • 他機能に影響がないかテスト確認をお願い🙌
Comments suppressed due to low confidence (1)

api/app/models/group.rb:856

  • 削除したクラスメソッドを参照するテストが残っていないか確認して、必要に応じてテストケースを更新または削除してね😉
        }

@hikahana
Copy link
Contributor

@YosukeIida
これって別でAPIあるからgroupから消したっていう認識であってる?

@YosukeIida
Copy link
Collaborator Author

YosukeIida commented Jul 11, 2025

@hikahana あってる,削除の部分だけ別にPR作成したというかんじです

調理工程申請を販売品申請に紐づけて表示する仕様変更

調理工程申請を販売品申請ベースで表示するようになった

  • Group -> CookingProcessOrder から FoodProduct -> CookingProcessOrder へ

データ取得方法の変更

  • Group経由じゃなくて、FoodProduct経由で取得するように
    より自然な関係性になった(販売品 -> 調理工程)

コードの責任の明確化

  • Groupモデルから調理工程関連の責任を削除
  • CookingProcessOrderモデルが自分の責任を持つように

@hikahana
Copy link
Contributor

それならあぷろぶで良さそう

@izuizu0424 izuizu0424 merged commit 9248976 into gm3/develop Jul 12, 2025
2 checks passed
@izuizu0424 izuizu0424 deleted the chore/i2/cleanup_group_model branch July 12, 2025 05:04
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.

4 participants