-
Notifications
You must be signed in to change notification settings - Fork 4
Fix: 라우트 순서 충돌로 컬렉션 순서 변경 API 동작 문제 해결 #131
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
Merged
Merged
Changes from all commits
Commits
Show all changes
81 commits
Select commit
Hold shift + click to select a range
dd0a987
feat: Youtube 영상 검색 api 구현
ekdbss 00866f8
Merge branch 'main' of https://github.com/ON-AIR-mate/Node.js
ekdbss 468b6d6
Merge branch 'main' of https://github.com/ON-AIR-mate/Node.js
ekdbss f05c17f
Merge branch 'main' of https://github.com/ON-AIR-mate/Node.js
ekdbss 73c20f8
feat: YouTube 영상 상세 조회 API 구현 및 DB 저장 기능 추가
ekdbss f3e296c
Fix: Add authentication middleware (requireAuth) to search route
ekdbss 247790a
Chore: remove unnecessary folder
ekdbss 14292ac
Chore: format code using Prettier
ekdbss 62715a1
Merge branch 'main' into main
DongilMin 533f93e
Update src/controllers/youtubeDetailController.ts
ekdbss 9ee4d5a
Chore: format code using Prettier
ekdbss 6570822
fix: add .js extensions to import paths in controller and route for d…
ekdbss 75c89cc
Fix: merge conflict resolved from upstream main
ekdbss 469a2aa
Fix: resolved merge conflict after pulling from upstream main
ekdbss 4bcfb35
Merge remote-tracking branch 'upstream/main'
ekdbss 09d063b
Merge branch 'main' of https://github.com/ON-AIR-mate/Node.js
ekdbss 7f14ccc
Merge remote-tracking branch 'upstream/main'
ekdbss fd10f1f
Merge branch 'main' of https://github.com/ON-AIR-mate/Node.js
ekdbss 7aae32a
Merge branch 'main' of https://github.com/ON-AIR-mate/Node.js
ekdbss c611ef9
Merge branch 'main' of https://github.com/ON-AIR-mate/Node.js
ekdbss eba55a4
feat: Youtube 영상 검색 api 구현
ekdbss 007165a
feat: YouTube 영상 상세 조회 API 구현 및 DB 저장 기능 추가
ekdbss 36a8c72
Fix: Add authentication middleware (requireAuth) to search route
ekdbss 47a2c52
Chore: remove unnecessary folder
ekdbss e6b86c2
Chore: format code using Prettier
ekdbss e8ef12f
Update src/controllers/youtubeDetailController.ts
ekdbss d270b19
Chore: format code using Prettier
ekdbss dba47d0
fix: add .js extensions to import paths in controller and route for d…
ekdbss 3c38a3b
Fix: resolved merge conflict after pulling from upstream main
ekdbss 24fe5e0
Feat: bookmark API 개발 및 schema 수정
ekdbss cfcff45
Feat: Bookmark message, timeline parsing을 위한 util 함수 추가
ekdbss 7692c40
Fix: 북마크 생성 시 parsing 함수 적용 및 timeline 분리 처리
ekdbss d09a695
Update src/dtos/bookmarkDto.ts
ekdbss 8ee9f11
Update src/controllers/bookmarkController.ts
ekdbss 92833dc
Refactor: 북마크 기반 방 생성 로직 정리 및 유효성 검사 강화
ekdbss 2d39a0a
feat: Add validation for maxParticipants in createRoomFromBookmark
ekdbss 487a5ff
Add /api prefix to bookmark-related endpoints
ekdbss c34ab21
Refactor: Move Prisma client import to lib/prisma.js
ekdbss 1d3353e
Feat: 사용자 추천 관련 api 구현
ekdbss 7f82c06
Feat: add search user status interface (#65)
DongilMin cbe2453
DOCS: README.md에 Swagger 접속 경로 추가 (#60)
DongilMin 1f69f8e
Feat/search user interface (#69)
DongilMin 66df72b
feat: 특정 방의 상세정보 조회 api 구현 (#68)
gaaahee ad78593
Chore: Remove unintended sharedCollection-related files
ekdbss 814179b
Feat: 공유받은 컬렉션 목록 조회 API 구현 및 upstream main 병합
ekdbss 70ffc5d
feat: 특정 방 상세정보 조회 api - duration 형식 변환 적용 (#70)
gaaahee 268151f
fix: npm run build 에러 해결 (#73)
gaaahee 0e08568
fix: npm run build 에러 해결 (#74)
gaaahee 0d1e5f9
feat: 방 설정 수정 api 구현 (#75)
gaaahee f8abc5e
feat: 활성화된 방 목록 조회 api 구현 (#77)
gaaahee 1b90d87
feat: 방 기존 설정 값 가져오는 api 구현 (#78)
gaaahee f70ce3b
feat: 활성화된 방 목록 조회 api 일부 수정 (#79)
gaaahee 2c22164
Fix: Remove unrelated code for collection acceptance/rejection, keep …
ekdbss 7c3d125
Feat/s3 upload (#80)
DongilMin 16b3c2e
Feat/ai summary (#81)
DongilMin 94b3173
Feat: Add Bedrock Claude Sonnet v1
DongilMin b6177f6
Feat/ai summary (#82)
DongilMin d7387cf
Feature: bookmark 관련 api 개발 및 prisma 수정 (#66)
ekdbss d6ab9fc
Feature: 사용자 recommendation 관련 api 구현 (#67)
ekdbss e7c8824
Feat: 공유 컬렉션 수락/거절 API 구현
ekdbss 66270d4
chore: resolve merge conflict with upstream main
ekdbss 00f80fb
Feature: sharedCollection 목록 조회 api 구현 (#72)
ekdbss a712700
chore: resolve merge conflict with upstream main
ekdbss f538714
chore: fix conflicts and finalize rebase
ekdbss 1b8d955
Merge branch 'main' of https://github.com/ON-AIR-mate/Node.js
ekdbss ab4b022
chore: resolve merge conflicts after pulling upstream main
ekdbss 0db5445
Merge branch 'main' of https://github.com/ON-AIR-mate/Node.js
ekdbss 3313440
Merge branch 'main' of https://github.com/ON-AIR-mate/Node.js into fe…
ekdbss e973e57
Merge branch 'main' of https://github.com/ON-AIR-mate/Node.js into fe…
ekdbss 0908b65
fix: bookmarkRoute.ts 충돌 해결
ekdbss 7f078ab
Merge branch 'main' of https://github.com/ON-AIR-mate/Node.js into fe…
ekdbss 7240c38
Merge branch 'main' of https://github.com/ON-AIR-mate/Node.js into fe…
ekdbss 710bed6
Merge branch 'main' of https://github.com/ON-AIR-mate/Node.js into fe…
ekdbss 13ecaa3
fix: 라우트 순서 문제로 인한 북마크 조회 API 응답 오류 수정
ekdbss 4765ac5
fix: 라우트 순서 충돌로 컬렉션 순서 변경 API 동작 문제 해결
ekdbss 5090aee
Merge branch 'main' of https://github.com/ON-AIR-mate/Node.js into fe…
ekdbss 72ed297
Merge branch 'feature/collection-api' of https://github.com/ekdbss/ON…
ekdbss fb5fa85
chore: format code using project formatter
ekdbss 3ca87fa
chore: sync Prisma schema and regenerate client
ekdbss 5e65019
Merge branch 'main' of https://github.com/ON-AIR-mate/Node.js into fe…
ekdbss 7d88be8
fix: Prisma relation 필드명 변경(video → youtube_videos)으로 인한 타입 에러 수정
ekdbss File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
💡 Verification agent
🧩 Analysis chain
관계 필드 네이밍 불일치(youtube_videos → youtubeVideo) 및 onDelete 명시 권장
제안 diff:
검증 스크립트(레거시 참조 정리: Room.video, youtube_videos, VideoOnRoom 등):
🏁 Script executed:
Length of output: 11187
스키마 파일에서 실제
youtube_videos필드 존재 여부를 확인하기 위해 아래 스크립트를 실행해주세요:🏁 Script executed:
Length of output: 442
Prisma 스키마: 관계 필드 youtube_videos → youtubeVideo로 변경 및 onDelete 정책 명시
Prisma 스키마에서 관계 필드명이 복수형·스네이크케이스(
youtube_videos)로 정의되어 있어, 프로젝트의 CamelCase 규칙과 불일치합니다. 또한onDelete행동이 명시되어 있지 않아 암묵적 DB 기본값(NO ACTION/RESTRICT)이 적용되는데, 추후 혼동을 방지하려면 명시하는 것이 좋습니다.Room모델의 관계 필드명을youtube_videos→youtubeVideo(singular CamelCase)로 변경@relation에onDelete: Restrict추가제안 diff:
model Room { participants RoomParticipant[] host User @relation("HostUser", fields: [hostId], references: [userId], onDelete: Cascade) - youtube_videos YoutubeVideo @relation(fields: [videoId], references: [videoId]) + youtubeVideo YoutubeVideo @relation(fields: [videoId], references: [videoId], onDelete: Restrict) @@index([hostId], map: "rooms_host_id_fkey") }위 스키마 변경에 따라 Prisma 클라이언트 타입이 바뀌므로, 다음 패턴을 전역 검색·교체해야 합니다.
•
{ video: true }→{ youtubeVideo: true }•
room.video→room.youtubeVideoBookmarkWithRelations,CollectionWithRelations, 메시지 페이로드 등)도 동일하게 업데이트이 변경은 Prisma 클라이언트 API가 깨지는 브레이킹 체인지이므로, 코드 전반에 걸쳐 반드시 리팩터링을 적용해 주세요.
📝 Committable suggestion
🤖 Prompt for AI Agents