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

cascade 제거 #321

Merged
merged 3 commits into from
Sep 23, 2024
Merged

cascade 제거 #321

merged 3 commits into from
Sep 23, 2024

Conversation

leeeryboy
Copy link
Contributor

한/영 연결 테이블에서 cascade 옵션 제거

한줄 요약

api 테스트 중 integrity가 깨지는 버그가 있었는데 연결 테이블에서 cascade 옵션을 사용하는게 원인 제공을 할 수 있을것 같아 제거하였습니다.

@leeeryboy leeeryboy added the Fix label Sep 23, 2024
@leeeryboy leeeryboy self-assigned this Sep 23, 2024
Copy link

github-actions bot commented Sep 23, 2024

Test Results

11 files  ±0  11 suites  ±0   1s ⏱️ ±0s
54 tests ±0  54 ✔️ ±0  0 💤 ±0  0 ±0 
58 runs  ±0  58 ✔️ ±0  0 💤 ±0  0 ±0 

Results for commit 78fc594. ± Comparison against base commit 2ded285.

♻️ This comment has been updated with latest results.

Copy link
Collaborator

@huGgW huGgW left a comment

Choose a reason for hiding this comment

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

LGTM!

@leeeryboy leeeryboy merged commit 20c134e into develop Sep 23, 2024
3 checks passed
leeeryboy added a commit that referenced this pull request Sep 23, 2024
* 교과목 정렬, 쿠키 secure 옵션 추가 (#308)

* feat: 교과목 정렬

* feat: 쿠키 secure 옵션 추가

* referrer policy 헤더 추가 (#311)

* referrer policy 헤더 추가

* 헤더 수정

* feat: scholarship CRUD (#309)

* feat: scholarship CRUD

* 불필요한 라인 삭제

* 리뷰 반영

* Update deploy_dev.yaml

* 강의실 예약 제한 (#312)

* feat: 소개 탭 CRUD (#314)

* feat: 소개 탭 CRUD

* fix: 권한 체크 추가

* fix: 동아리 id 기준으로 로직 변경

* fix: 쿠키 same-site: strict

* 리뷰 반영

* 리뷰 반영

* 로그인 핫픽스 (#316)

* Feat: Research Group & Center 언어포함 CRUD 생성 (#315)

* refactor: deprecated v1 controller

* refactor: extract types in seperate package, remove unused attachment from research

* feat: make lab's research as nullable

* feat: add research language entity & repository

* feat: add language & sealed dtos

* feat: remove migrate methods & seperate update index methods

* feat: add CRUD language considered methods

* feat: add v2 reserach controller

* fix: fix test compiling

* test: fix professerservice test

* refactor: fix ktlint formatting

* review: research -> fetch lazy

* Refactor: 구성원 POST API image => mainImage 리펙토링 #317

* Refactor: PUT API newImage => newMainImage 수정 #318

* Feat: Lab CRUD (#319)

* feat: add request body

* feat: modify dto nullability

* feat: extract lab service && implement CRUD

* feat: add lab language dto

* feat: add lab events

* fix: fix unique constraints for research language entity

* feat: add methods to repositories

* feat: add LAB CRUD APIs

* refactor: seperate event services

* refactor: professors to mutable var

* fix: fix typos

* comment: add comment to professor service method

* test: remove unneed test file

* feat: split search service and event service

* feat: implement professor, research event service

* fix: apply ktlint

* cascade 제거 (#321)

* cascade 제거

* 이미지 제거 추가

---------

Co-authored-by: Junhyeong Kim <[email protected]>
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.

2 participants