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

部屋に関するAPIを追加 #150

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

部屋に関するAPIを追加 #150

wants to merge 6 commits into from

Conversation

akimon658
Copy link
Member

@akimon658 akimon658 commented Feb 2, 2025

Closes #130

TODO:

  • 編集できるように(PUTの実装)
  • マイグレーション(mainをマージしてmigrationディレクトリに追加する必要がある。多分AutoMigrationだけでOK)

@kitsne241 kitsne241 self-assigned this Feb 2, 2025
@kitsne241 kitsne241 marked this pull request as ready for review February 2, 2025 09:51
backend/handler/rooms.go Outdated Show resolved Hide resolved
openapi.yaml Show resolved Hide resolved
backend/handler/rooms.go Show resolved Hide resolved
@akimon658
Copy link
Member Author

あと追加で、マイグレーションに関しては自動で行われなくなったので

  • migrationディレクトリにv4.goみたいなファイルを作る
  • v4.goで定義した関数をmigrate.goに追記する

必要があります。v4.goの内容はv3.goと同じで大丈夫です

@kitsne241
Copy link
Collaborator

めっちゃ参考になります、ありがとう!! やります

@kitsne241
Copy link
Collaborator

とりあえず指摘をもらった部分の修正はしました。時間あるとき再確認お願いします 🙏

@akimon658
Copy link
Member Author

LGTMです!動作確認していないのでまだ僕の方でマージはしませんが、フロントの方で必要になったら勝手にマージしても良いです

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