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.

Sign up for GitHub

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

Jump to bottom

GormRepo の返り値で domain パッケージ定義にされているものを db パッケージ定義に統一する #574

Open

iChemy opened this issue Nov 18, 2024 · 1 comment

Assignees

Labels

discussion refactoring

Contributor

iChemy commented Nov 18, 2024

部屋周りの GormRepo のインターフェースが大体 domain パッケージの定義を返しているのに対してイベントやユーザーはdb定義のものを返し，呼び出し先で型を Conv 系の関数を使って変換しているのでこれを統一したい．

iChemy added refactoring discussion labels

iChemy self-assigned this

Contributor Author

iChemy commented Nov 18, 2024

https://github.com/traPtitech/traPortfolio/blob/main/internal/infrastructure/repository/project_impl.go

trapPortfolio とかは db から取得したらその関数内で domain に変換していそうなので domain で統一する方が良いかもしれないと思っている

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment