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

fix: 중복된 여러개의 lecture_id 삭제 해결 #1128

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

kwoo28
Copy link
Contributor

@kwoo28 kwoo28 commented Dec 11, 2024

🔥 연관 이슈

🚀 작업 내용

  1. JPA Repository에서 해당 쿼리반환타입을 List로 변경했습니다.

💬 리뷰 중점사항

@kwoo28 kwoo28 self-assigned this Dec 11, 2024
@github-actions github-actions bot added the 버그 정상적으로 동작하지 않는 문제상황입니다. label Dec 11, 2024
Copy link

Unit Test Results

343 tests   342 ✔️  1m 25s ⏱️
  41 suites      1 💤
  41 files        0

Results for commit 65da5bc.

Copy link
Contributor

@seongjae6751 seongjae6751 left a comment

Choose a reason for hiding this comment

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

R
문제점이 강의가 중복으로 들어가는 것이 문제지 잘못 들어간 중복된 강의들을 한번에 못 삭제 하는 것이 문제는 아닌 것 같다는 생각이 드는데 어떻게 생각하실까요
(Unique 제약 조건 걸어주시면 좋을 것 같아용)

@kwoo28
Copy link
Contributor Author

kwoo28 commented Dec 11, 2024

R 문제점이 강의가 중복으로 들어가는 것이 문제인거지 잘못 들어간 중복된 강의들을 한번에 못 삭제 하는 것이 문제는 아닌 것 같다는 생각이 드는데 어떻게 생각하실까요 (Unique 제약 조건 걸어주시면 좋을 것 같아용)

소프트딜리트라서 유니크 제약조건을 어덯게 걸어야할지 모르겠더라고요..

Copy link
Contributor

@Soundbar91 Soundbar91 left a comment

Choose a reason for hiding this comment

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

소프트딜리트라서 유니크 제약조건을 어덯게 걸어야할지 모르겠더라고요..

소프트 딜리트가 걸리네용,,
클라에서 시간표 복구 시간이 끝나면 하드 딜리트하는 api 호출하는 건 어떻게 생각하시나요?? 불필요할까요..?

Copy link
Contributor

@20HyeonsuLee 20HyeonsuLee left a comment

Choose a reason for hiding this comment

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

소프트 딜리트가 걸리네용,,
클라에서 시간표 복구 시간이 끝나면 하드 딜리트하는 api 호출하는 건 어떻게 생각하시나요?? 불필요할까요..?

redis를 이용해보는건 어떨까요👀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
버그 정상적으로 동작하지 않는 문제상황입니다.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

시간표 정규 강의 삭제 버그
4 participants