-
Notifications
You must be signed in to change notification settings - Fork 1
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
Feat/backend/api/seed #15
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ドキュメントとDockerしか見てません
@KinjiKawaguchi レビューとは別にコンテナが起動しなかったです 2024-03-23 12:36
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
学部コメントアウトなんでか知りたい.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ent generateしてるのにスキーマのメールアドレスの部分(とそれに準じる依存したファイル)がこうしんされてないみたいなことかねこれ.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
ボリュームが残ってた。消したら解決した docker compose -f docker-compose.dev.yml stop db
docker compose -f docker-compose.dev.yml rm db
docker volume rm typing-server_db-data
docker compose -f docker-compose.dev.yml up -d |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
学部コメントアウトなんでか知りたい.
コメントアウトなので、削除とは違うということをまず伝えておきます。
開発の簡素化のために一旦コメントアウトしました。
また、履修名簿からのインポート機能についても考えないといけないので、履修者名簿の仕様にもよるかなと思い、一旦最低限の情報だけ残したという感じです。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
修正感謝です~
やったこと