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

feat : 문제 풀이 결과 세분화 #36

Merged
merged 4 commits into from
Aug 20, 2024
Merged

feat : 문제 풀이 결과 세분화 #36

merged 4 commits into from
Aug 20, 2024

Conversation

rladmstn
Copy link
Contributor

@rladmstn rladmstn commented Aug 18, 2024

📌 Related Issue

#29

🚀 Description

  • 문제 풀이 필드를 boolean에서 String으로 수정했습니다

📢 Review Point

  • 제가 또 수정해야 하는데 빠뜨린 곳이 있을 것 같습니다.. 수정해야 하는데 안된 곳 보이면 알려주세요!

📚Etc (선택)

  • hibernate ddl-auto가 update인지라 배포할 때 result 컬럼은 자동으로 생성 되지만, 기존의 isCorrect 컬럼이 삭제되진 않습니다! 그래서 merge 될 때 rds에서 alter table 쿼리 날리겠습니다
  • controller 테스트 코드는 solution 엔드포인트 먼저 수정되면 그 때 테스트 코드 추가하겠습니다..

@rladmstn rladmstn added the new-feature 기능 추가 label Aug 18, 2024
@hwangjokim
Copy link
Contributor

뉴비들에게 생각하는 힘을 길러주기 위해 제가 마지막으로 리뷰합니다

Copy link
Contributor

@s-hwan s-hwan left a comment

Choose a reason for hiding this comment

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

잘봤습니당..!
아직 Query 어노테이션이 붙은 부분들은 이해하기가 좀 어렵군요..

Copy link
Contributor

@sh0723 sh0723 left a comment

Choose a reason for hiding this comment

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

잘 봤습니다! ㅎㅎ 보다보니 생각난건데 지금 만들던 API에서 IsCorrect가 여기서 바뀐걸 까먹고 코드를 짰네요 .......... 다시 짜러 가겠읍니다..

@sh0723 sh0723 merged commit 15e45ee into develop Aug 20, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new-feature 기능 추가
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants