Skip to content

Room API実装#20

Merged
hikaru-0602 merged 15 commits intomainfrom
feature/room-api
Mar 27, 2026
Merged

Room API実装#20
hikaru-0602 merged 15 commits intomainfrom
feature/room-api

Conversation

@hikaru-0602
Copy link
Copy Markdown
Contributor

やったこと

  • RoomのAPI実装してみた.とりあえず実装しただけなのでいらなかったら削除してもいいよ

  • Update OpenAPI schema #19 にPR出してる

  • Close

  • [ ]

確認したこと

  • ビルドは通ってる

メモ

  • このブランチから新しくブランチ切ってtimetable実装する.これもいらんかったら削除していいよ

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Room リソースの CRUD API を既存の handler/service/repository/domain/database レイヤ構成に沿って追加し、サーバ起動時の DI と AutoMigrate に組み込む PR です。

Changes:

  • Room の domain/database モデルと GORM AutoMigrate 対象を追加
  • RoomRepository / RoomService を新設し、CRUD を実装
  • Rooms v1 の handler 実装と DI(NewHandler/main)配線を追加

Reviewed changes

Copilot reviewed 13 out of 13 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
internal/service/room.go RoomService と repository IF を追加
internal/repository/room.go Room の CRUD を GORM で実装
internal/handler/room_create.go Room 作成 API 実装
internal/handler/room_update.go Room 更新 API 実装(404 マッピング含む)
internal/handler/room_delete.go Room 削除 API 実装(404 マッピング含む)
internal/handler/room_detail.go Room 詳細 API 実装(404 マッピング含む)
internal/handler/room_list.go Room 一覧 API 実装(※フィルタの扱い要確認)
internal/handler/handler.go Handler に roomSvc を追加し DI を拡張
internal/handler/converter.go domain↔api の Room 変換を追加
internal/domain/room.go Room/Floor の domain 型を追加
internal/database/room.go GORM モデルと domain 変換を追加
internal/database/migrate.go AutoMigrate に Room を追加
cmd/server/main.go Room の repository/service を生成し Handler に注入

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@hikaru-0602 hikaru-0602 requested a review from kantacky March 27, 2026 04:47
@masaya-osuga
Copy link
Copy Markdown
Member

Adminで動作確認したいかも

Copy link
Copy Markdown
Member

@kantacky kantacky left a comment

Choose a reason for hiding this comment

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

dev環境でCRUD確認しました

Base automatically changed from update-openapi-schema to main March 27, 2026 08:36
@hikaru-0602 hikaru-0602 requested a review from a team March 27, 2026 08:36
@hikaru-0602 hikaru-0602 merged commit 94c965e into main Mar 27, 2026
1 check passed
@hikaru-0602 hikaru-0602 deleted the feature/room-api branch March 27, 2026 08:38
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.

4 participants