-
Notifications
You must be signed in to change notification settings - Fork 0
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
sutdy domain 관련 API 제작 #27
Open
cmj7271
wants to merge
20
commits into
develop
Choose a base branch
from
feature/study-endpoint
base: develop
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
create method 는 builder 로 대신할 수 있다
스터디를 만드는 사람을 leader 로 설정 TODO: AUTH 구현 후 API 를 요청한 멤버를 조회해서 추가하는 로직
명세서대로 간략화된 response 로 수정 feat: simple response 추가 명세서에 따른 간단한 응답 DTO 추가 feat: simple response 추가 및 simple res 에 불필요한 import 제거 feat: 전체 스터디 조회 API 를 명세서대로 수정 simple response 로 수정
스터디가 없으면 404 Not Found 반환 refact: getstudyDetail 의 exception 을 메소드가 던지도록 변경 controller 에서 처리할 예정 refact: 스터디 정보 조회 api 익셉션 처리 스터디가 없을 경우, 404 Not Found 반환
스터디가 없는 경우 404 Not Found 반환 refact: updateStudy가 id 를 반환하도록 수정 및 익셉션 반환 처리 수정된 스터디의 ID 를 반환하며 스터디가 없을 경우 커스텀 익셉션을 반환 refact: updateStudy 에서 스터디가 없는 경우 추가 스터디가 없다면 404 Not Found 반환
없을 경우 404 Not Found 반환
질문이 존재하는 스터디에 지원하는 method 구현 Answer.java: 생성자, 커스텀 생성자 및 getter 추가 ErrorCode.java: 질문 존재 없는 경우, 멤버가 없는 경우에 대응하는 에러코드 추가(404 Not Found) StudyController.java: 스터디 지원 API 구현 StudentStudyService.java: 스터디 지원 메소드 추가 Study.java: Answer 를 추가하는 addAnswer 추가 TODO: cotroller 에 auth 를 통한 member 조회 추가
globalExceptionHandler 로 처리 refact: import 제거
ErrorCode.java: answer 관련 error code 추가 StudyRepository.java: answer 를 찾는 메소드 추가 StudyController.java: 스터디 지원 API 구현 TODO: auth 를 통해 member Id 구하기
지원 질문이 없지만, 답변이 있는 경우, exception을 반환하게 변경
Study.java: answer 를 초기화하는 메소드 StudentStudyService.java: answer 를 제거하는 메소드 추가 StudyController.java: API 구현
기존의 study_member table 에 answer col 추가로 리팩토링
StudyRepository.java: 특정 status 의 StudyMember 를 조회하는 퀴리 작성 LeaderStudyService.java: 지원자 전체를 조회하는 서비스 작성 StudyController.java: api 작성 TODO: auth 를 통해 leader 확인
StudentStudyService.java: 참여자 조회하는 메소드 작성 studentController.java: 조회 api 작성
study.java 에서 사용한 메소드를 제거하고, 기존의 studyRepository 의 쿼리로 처리
cancelApply -> cancel
studyMember.java: method 명 변경 LeaderStudyService.java: 탈퇴 처리 메소드 작성 StudyController.java: 탈퇴 api 작성
MemberRepository.java: member 정보를 바탕으로 member 객체를 찾는 쿼리 추가 StudyMember.java: 참여자로 변경하는 메소드 추가 StudentStudyService.java: 로직 리팩토링(커밋과 무관) LeaderStudyService.java: 지원 상태를 변경하는 메소드 추가 및 일부 리팩토링 StudyController.java: API 작성
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
#18 해당하는 API 모두 제작 완료했습니다
TODO: auth 가 필요한 부분은 임시 구현해둔 상태라 auth 구현 후 리팩토링 필요