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.

Sign up for GitHub

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

Jump to bottom

[BE] fix: 메시지가 남아있는 채로 모임 삭제가 불가능하던 버그 수정 #792

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

[BE] fix: 메시지가 남아있는 채로 모임 삭제가 불가능하던 버그 수정 #792

wants to merge 2 commits into from
+26 −4

Conversation

takoyakimchi
Copy link
Contributor

@takoyakimchi takoyakimchi commented Jan 9, 2025 •
edited
Loading

이슈

개발 사항

  • ChatRoom - ChatMessage 간 연관관계 해제

전달 사항

@takoyakimchi takoyakimchi added 🐞 bug 버그 수정 🖥 backend backend labels Jan 9, 2025
@takoyakimchi takoyakimchi self-assigned this Jan 9, 2025
@github-actions GitHub Actions
Copy link

github-actions bot commented Jan 9, 2025

Test Results

231 tests   231 ✅  32s ⏱️
 47 suites    0 💤
 47 files      0 ❌

Results for commit 412f4c3.

@takoyakimchi
Copy link
Contributor Author

dev 환경에서는 잘 작동하는 것을 확인했습니다.
다만 ddl-auto: update 만으로는 외래 키가 삭제되지는 않기 때문에 직접 삭제하는 작업이 필요합니다.

@ehtjsv2
Copy link
Contributor

ehtjsv2 commented Jan 22, 2025

@takoyakimchi
그러면

  1. PROD에 이미 있는 club 데이터는 직접 삭제하면 됨
  2. 이후에 들어오는 club은 chatMessage와 연관관계가 없으니, 삭제가능

맞나요?

@takoyakimchi
Copy link
Contributor Author

takoyakimchi commented Jan 22, 2025 •
edited
Loading

@ehtjsv2 앗 정확히는 연관관계 삭제하는 명령어를 직접 입력해야 되는데,

validate를 통과 못할 수도 있으니 dev에서 테스트가 필요할 것 같습니다

정상 작동하면 테이블 명세 변경 없이 데이터 그대로 가져갈 수 있슴다

@ehtjsv2
Copy link
Contributor

ehtjsv2 commented Jan 22, 2025

  • dev(ddl-auto: update)

    • 외래키 제약조건 직접 삭제해줘야됨(정상동작 확인완료)

  • prod(ddl-auto: validate)

    • 외래키 제약조건 직접 삭제하고 어플리케이션 실행 시, validate옵션에 방해받지 않는 지?(확인해야됨)

@takoyakimchi
Copy link
Contributor Author

@ehtjsv2
dev에서 validate로 바꾸고 테스트 필요할 것 같은데
오늘 내일 중으로 작업해보고 잘 되면 머지할게요

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ehtjsv2 ehtjsv2 ehtjsv2 approved these changes

@jimi567 jimi567 Awaiting requested review from jimi567 jimi567 is a code owner

@J-I-H-O J-I-H-O Awaiting requested review from J-I-H-O J-I-H-O is a code owner

Assignees

@takoyakimchi takoyakimchi

Labels
🖥 backend backend 🐞 bug 버그 수정
Projects
2024-friendogly
Status: No status
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

모임 채팅방에 메시지가 남아있는 채로 모임을 나가면 예외가 발생하는 버그 수정
2 participants
@takoyakimchi @ehtjsv2