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: StandardAnswer 업데이트시 기존 StandardAnswer 삭제 안 되던 버그 수정 #173

Merged
merged 2 commits into from
Jan 14, 2024

Conversation

ekzm8523
Copy link
Member

Issue Number

close: #

작업 내역

구현 내용 및 작업 했던 내역

  • �관리자페이지에서 서술형 문제 수정시 모범 답안이 정상적으로 업데이트되지 않던 버그 수정

변경사항

  • 의존성 목록

작업 유형

  • 신규 기능 추가
  • 버그 수정
  • 리펙토링
  • 문서 업데이트

PR 특이 사항

기존에 로직에서 StandardAnswer를 삭제시 LongProblem과 연관관계가 있는 StandardAnswer가 삭제되지 않는 버그가 존재했는데 원인은 정확하게 파악하지 못 했어요.
유추하기론 Relationship에 영향때문에 delete 명령어가 제대로 들어가지 않은 것 같은데 기존에 왜 삭제가 안 되었을지 아시면 알려주세요 🙏

@ekzm8523 ekzm8523 added the bug Something isn't working label Jan 14, 2024
@ekzm8523 ekzm8523 requested a review from kshired January 14, 2024 07:22
Copy link

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@ekzm8523
Copy link
Member Author

@kshired jpa박사 선생님 저거 원인좀 알려주세요..

@kshired
Copy link
Member

kshired commented Jan 14, 2024

@ekzm8523

++ 원래로직에선 deleteAll -> save 했지만 longProblem의 standardAnswers list가 변경되지 않아서 그럴 것 같긴하네용

@kshired kshired merged commit 8724a4a into dev Jan 14, 2024
5 checks passed
@kshired kshired deleted the feat/long-problem-update-bus-fix branch January 14, 2024 10:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants