Skip to content
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: frontend containerのビルド時にtraQ-S_UIをダウンロードしてくる #2559

Merged
merged 4 commits into from
Jan 17, 2025

Conversation

ras0q
Copy link
Member

@ras0q ras0q commented Dec 21, 2024

make up (docker compose up --build -d)の前にmake update-frontendをし忘れるとフロントエンドが立ち上がらないのでDocker Composeのビルド時に引っ張ってくるようにしました

Copy link

@Pugma Pugma left a comment

Choose a reason for hiding this comment

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

ありがとうございます!
改修してくださった部分が正常に動作することを mac でも確認いたしました

docs/development.md の ll.33-34 も削除していいのかなと思いました
あとは makefile も改修がいりそうな気がしていますがどうでしょうか…?

(l.16 で First Up (or entirely rebuild) ってリビルドについても触れているし、 ll.30-31 も要らない気がしますが、別で書いている方が見落としづらくてやさしいですかね)

@ras0q
Copy link
Member Author

ras0q commented Jan 16, 2025

Dockerを使わずにローカルでフロントエンドをダウンロードしたいこともあるかなと思って一応残してたんですがMakefileとdevelopment.md見る感じDockerを使わない開発は想定されてなさそうなのでMakefileのターゲットごと消しちゃいますね

@ras0q ras0q force-pushed the feat/fetch-frontend-on-docker-build branch from abc9c6f to c4275f6 Compare January 16, 2025 12:20
@ras0q ras0q force-pushed the feat/fetch-frontend-on-docker-build branch from c4275f6 to 60fbf65 Compare January 16, 2025 12:20
@ras0q ras0q requested a review from Pugma January 16, 2025 12:21
Copy link

@Pugma Pugma left a comment

Choose a reason for hiding this comment

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

ありがとうございます!
よさそうです

@ras0q ras0q merged commit e88f5d5 into master Jan 17, 2025
6 checks passed
@ras0q ras0q deleted the feat/fetch-frontend-on-docker-build branch January 17, 2025 04:07
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.

2 participants