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

스터디 개설 API 수정, 스터디 삭제 API 개발 #56

Merged
merged 2 commits into from
Jun 23, 2022

Conversation

gandi0330
Copy link
Contributor

@gandi0330 gandi0330 commented Jun 23, 2022

📑 제목

fix : 스터디 개설 API에 스터디 리더를 스터디원으로 추가하는 기능 추가
feat : 스터디 삭제 API 개발, 테스트 완료

📎 관련 이슈

✔️ 셀프 체크리스트

최소 1명 이상의 assign을 받아야만 Merge 가능합니다.

  • Warning Message가 발생하지 않았나요?
  • Coding Convention을 준수했나요?

💬 작업 내용

구현 내용 및 작업 했던 내역

  • 스터디 개설 API에 스터디 리더를 스터디원으로 추가하는 기능 추가

🚧 PR 특이 사항

PR을 볼 때 주의깊게 봐야하거나 말하고 싶은 점

  • application.properties 에 mysql로 다시 바꿨어요 테스트하느라
  • 그 외래키 관련 db에서 설정해서 배포한걸로 테스트 하려면 db 또 수정해야 할듯 제가 수정할 수 있는 방법 알려주시면 개발하면서 수정할 수 있을텐데
  • 스터디만 삭제했을 때 관련 테이블 삭제되는 거 확인했습니다. ( 스터디원 테이블, 사전질문 테이블 )

🕰 실제 소요 시간

작업을 시작하기 부터 PR을 올리기 까지 소요된 시간입니다.

  • (2시간)

@gandi0330 gandi0330 requested a review from lion1735 June 23, 2022 07:32
@gandi0330 gandi0330 added this to the API개발 milestone Jun 23, 2022
@gandi0330 gandi0330 changed the title fix : 스터디 개설 API에 스터디 리더를 스터디원으로 추가하는 기능 추가 스터디 개설 API 수정, 스터디 삭제 API 개발 Jun 23, 2022
@lion1735 lion1735 merged commit a04805a into back Jun 23, 2022
@lion1735 lion1735 deleted the F04_BE_정현명 branch June 23, 2022 10:30
@lion1735
Copy link
Contributor

여기두 앞에 제목 달아주세용

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