各棟ごとの合計募金額を取得するAPIの作成 by hikahana · Pull Request #986 · NUTFes/FinanSu

hikahana · 2025-05-01T19:29:52Z

対応Issue

resolve #0
なし

概要

棟を一覧で表示し、各棟の募金額を取得するAPI
- 棟に紐づいている教師の募金を合算する。
- 棟ごとに合算した募金額を返す。
  
  このデザインに沿って作成しました。
  おそらく年度ごとの取得になるため、年度ごとで取得できるようにしてます。
  sqlクエリ自体は以下の通りです。

SELECT
  buildings.id AS building_id,
  buildings.name AS building_name,
  fund_informations.price AS price,
  years.year AS year
FROM fund_informations
JOIN teachers ON fund_informations.teacher_id = teachers.id
JOIN users ON fund_informations.user_id = users.id
JOIN departments ON teachers.department_id = departments.id
JOIN room_teachers ON room_teachers.teacher_id = teachers.id
JOIN rooms ON room_teachers.room_id = rooms.id
JOIN floors ON rooms.floor_id = floors.id
JOIN building_units ON floors.building_unit_id = building_units.id
JOIN buildings ON building_units.building_id = buildings.id
JOIN year_periods ON fund_informations.created_at > year_periods.started_at AND fund_informations.created_at < year_periods.ended_at
JOIN years ON year_periods.year_id = years.id
WHERE years.year = 2025
ORDER BY fund_informations.updated_at DESC;

画面スクリーンショット等

URL
スクリーンショット

テスト項目

動作は多分ok。見ても見なくても
要望通りになっているか
コードレビュー一応

備考

APIどこに生やせばいいか迷ってとりあえずでfund_informationsに生やしました。
ここら辺話してから作成に取り掛かればよかったです。

…nsに紐づくデータを取得する機能を実装

…関連する型とフックを更新

Kubosaka

フロントで収支管理がfund_informationsになってたからどうしよ

…ahana/add-get-department-by-donation

cloudflare-workers-and-pages · 2025-05-02T08:11:59Z

Deploying finansu with Cloudflare Pages

Latest commit:	`a0cccf1`
Status:	✅ Deploy successful!
Preview URL:	https://d032a9db.finansu.pages.dev
Branch Preview URL:	https://feat-hikahana-add-get-depart.finansu.pages.dev

View logs

hikahana · 2025-05-05T10:55:11Z

@Kubosaka
募金登録も収支管理に含めるなら、fund_informationsでもいい感ありますけどこれもMTで確認したいですね。

hikahana · 2025-05-09T07:22:37Z

APIのパスをcampus_donationに変更するためいったんdraftにします。
feat/hikahana/get-rooms-by-buildingのブランチ取り込んで作業します。

…inanSu into feat/hikahana/add-get-department-by-donation

…ントを削除

…hub:NUTFes/FinanSu into feat/hikahana/add-get-department-by-donation

Kubosaka

とりあえずコメ

Kubosaka · 2025-05-19T14:38:21Z

openapi/openapi.yaml

+        - name: building_id
+          in: path
+          required: true
+          schema:
+            type: integer
+          description: ID of the building
+        - name: floor_id
+          in: path
+          required: true
+          schema:
+            type: integer
+          description: ID of the floor


こっちも年度かな！

これ違うprの方だと思う！
からこれは一旦考慮しない

Kubosaka · 2025-05-19T14:42:44Z

api/externals/repository/campus_donation_repository.go

+
+func (fir *campusDonationRepository) AllBuildingsByPeriod(c context.Context, year string) (*sql.Rows, error) {
+
+	db := goqu.Dialect("mysql")


dialectはどっかで定義してると思う

Kubosaka · 2025-05-19T14:43:09Z

api/externals/controller/cmapus_donation_controller.go

file名タイポってる
cmapus_donation_controller.go

api/internals/usecase/campus_donation_usecase.go

Kubosaka · 2025-06-04T12:58:03Z

api/internals/usecase/campus_donation_usecase.go

+	for rows.Next() {
+		var buildingTotal domain.CampusDonationBuilding
+		if err := rows.Scan(&buildingTotal.Id, &buildingTotal.Name, &buildingTotal.Price); err != nil {
+			return nil, err
+		}
+		if b, exists := aggregated[buildingTotal.Id]; exists {
+			if b.TotalPrice == nil {
+				b.TotalPrice = new(int)
+			}
+			*b.TotalPrice += buildingTotal.Price
+		} else {
+			id := buildingTotal.Id
+			name := buildingTotal.Name
+			price := buildingTotal.Price
+			aggregated[buildingTotal.Id] = &BuildingTotal{
+				Id:         &id,
+				Name:       &name,
+				TotalPrice: &price,
+			}
+		}
+	}


[imo] nest深いからcontinueとか使ってあさくできたりする？

296d728
一次対応

Kubosaka · 2025-06-04T12:59:23Z

api/externals/repository/campus_donation_repository.go

+			goqu.I("buildings.id"),
+			goqu.I("buildings.name"),
+			goqu.I("campus_donations.price"),


sumとか集計関数つかったらusecaseの処理楽になるのかな、まあどっちでも

棟ごとに分けるの実装するのめんどみがあるからusecaseで処理させます

hikahana added 7 commits May 1, 2025 03:34

gen: orval

5414852

/fund_informations/building/{year}エンドポイントを追加し、年度で指定されたfund_informatio…

75405b6

…nsに紐づくデータを取得する機能を実装

教師モデルを追加し、建物と部屋番号のオプションパラメータを含むfund_informationsエンドポイントを更新

034c108

fund_informationに年度別の建物情報を取得する機能を追加

090464d

fund_informationsエンドポイントのパスを修正: building/{year}をbuildings/{year}に変更

c9bd752

fund_informationsエンドポイントのパスを修正: building/{year}をbuildings/{year}に変更し、…

e15837d

…関連する型とフックを更新

fund_informationsエンドポイントに年度別の建物情報を取得する機能を追加し、関連するコントローラー、リポジトリ、ユースケースを更新

6566275

hikahana requested review from Chikuwa0141, KazumaSun, Kubosaka and TkymHrt May 1, 2025 19:29

github-actions bot assigned hikahana May 1, 2025

Kubosaka reviewed May 2, 2025

View reviewed changes

hikahana changed the base branch from develop to fix/hikahana/er May 2, 2025 06:24

Merge branch 'fix/hikahana/er' of github:NUTFes/FinanSu into feat/hik…

ecc2d2e

…ahana/add-get-department-by-donation

hikahana added 3 commits May 2, 2025 20:22

[fix]]: 配列形式に変更した

5a5fd28

[generated]: buildingTotal

e2384e4

[fix]: totalPriceの合算を返すように修正

cfa968a

hikahana requested a review from Kubosaka May 4, 2025 23:06

hikahana marked this pull request as draft May 9, 2025 07:22

hikahana added 4 commits May 9, 2025 16:23

Merge branch 'feat/hikahana/get-rooms-by-building' of github:NUTFes/F…

318ba7c

…inanSu into feat/hikahana/add-get-department-by-donation

feat: 年度で指定されたfund_informationsに紐づくデータを取得するAPIエンドポイントを追加

63262c4

feat: 年度で指定されたキャンパス寄付データを取得するAPIエンドポイントを追加し、fund_informations関連のエンドポイ…

3a658a9

…ントを削除

feat: 年度別のキャンパス寄付データを取得するAPIエンドポイントを追加し、fund_information関連のエンドポイントを削除

942562f

Base automatically changed from fix/hikahana/er to develop May 11, 2025 21:05

hikahana added 2 commits May 15, 2025 19:26

Merge branch 'fix/hikahana/change-table-name-campus-donations' of git…

bf18310

…hub:NUTFes/FinanSu into feat/hikahana/add-get-department-by-donation

campus_donation_repositoryのfund_informationsをcampus_donationsに置き換え

1ffe7f0

hikahana marked this pull request as ready for review May 15, 2025 10:27

キャンパス寄付ビルディングの構造体を追加し、関連するコードを更新

dc21071

hikahana changed the base branch from develop to fix/hikahana/change-table-name-campus-donations May 16, 2025 07:51

hikahana changed the base branch from fix/hikahana/change-table-name-campus-donations to feat/hikahana/get-rooms-by-building May 16, 2025 07:56

fix: fund_information削除の時に消えたrouter修正

a0cccf1

Kubosaka reviewed May 19, 2025

View reviewed changes

hikahana added 2 commits May 25, 2025 19:24

merge

b5c4ed3

refactor: update database dialect usage in campus_donation_repository

65f1478

hikahana requested a review from Kubosaka May 25, 2025 10:34

Kubosaka reviewed Jun 4, 2025

View reviewed changes

fix: GetCampusDonationBuildingByPeriodでcontinueを使ってネストを低減

296d728


		func (fir campusDonationRepository) AllBuildingsByPeriod(c context.Context, year string) (sql.Rows, error) {

		db := goqu.Dialect("mysql")

Conversation

hikahana commented May 1, 2025

対応Issue

概要

画面スクリーンショット等

テスト項目

備考

Uh oh!

Kubosaka left a comment

Choose a reason for hiding this comment

Uh oh!

cloudflare-workers-and-pages bot commented May 2, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Deploying finansu with Cloudflare Pages

Uh oh!

hikahana commented May 5, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

hikahana commented May 9, 2025

Uh oh!

Kubosaka left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

cloudflare-workers-and-pages bot commented May 2, 2025 •

edited

Loading

hikahana commented May 5, 2025 •

edited

Loading