Skip to content

[Improve]: cores/core に配置している model や exception を cores/model を作成して配置する #329

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
1 task done
blendthink opened this issue Sep 25, 2024 · 1 comment
Assignees

Comments

@blendthink
Copy link
Member

これに関する既存の Issue はありますか?

  • 既存の Issue はありませんでした

背景

  • cores/core の役割が分かりづらい
  • 便利ツール的な役割のものは utils/~ に配置することに決定した
  • model などの各機能で共通して利用することが多いものについては一箇所にまとめておいたほうが実装スピード向上しそう
  • exceptions.dart で定義しているものは model である

提案の詳細

cores/core に配置している model や exception を cores/model を作成して配置する

代替案

domains/model や commons/model としてもいいかもしれない。
ただ、いったんは cores 配下に配置しておいて後から整理したい。

環境

- Commit Hash: 6b68c4fdd5f631faab0d7b8d220987fe802cb5eb
@tatsutakein
Copy link
Member

📝 #359 (comment)

📝 Slack のハドルでお話した結果

  • cores/model パッケージからは Riverpod の依存は削除
  • cores/core パッケージにある Provider は cores/data パッケージに移動

@tatsutakein tatsutakein moved this from Todo to In Progress in Flutter Mobile Project Template 開発 Jan 7, 2025
@blendthink blendthink assigned blendthink and unassigned tatsutakein Feb 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In Progress
Development

No branches or pull requests

2 participants