Refactor: 2차 멘토링 코멘트 반영 #52
Merged
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
… 로그 저장 로직 구현, ConnectTimeout 및 ReadTimeout 5초로 설정
Closed
5 tasks
2iedo
approved these changes
Oct 9, 2024
zzoe2346
approved these changes
Oct 9, 2024
…p3/Team8_BE into Refactor/issue-#51
Merged
cheol-95
pushed a commit
that referenced
this pull request
Oct 13, 2024
* Feat: 로그인 로직에서 프론트에 보내는 DTO에 status 추가 및 리프레쉬 토큰 서버 관리 로직 �구현 (멘토링 코멘트 적용) (#37) * enev: .DS_Store files삭제, gitignore 추가 * feat: h2 콘솔을 외부에서 접속 가능하게 설정 * feat: 회원가입 로직에서 회원여부를 직관적으로 판단할 수 있게 dto에 isMember라는 필드 추가 * feat: 리프레쉬 토큰을 레디스에 저장하는 로직 추가 및, 일회성 사용으로 로직 구성 * refactor: 코드 자동 리팩토링 * fix: 레디스 오류멘트 수정 * fix: 오류 메시지 명확한 확인을 위한 try-catch문 제거 * refactor: extractEmail 로직 활용 * Feat : 리뷰 기능과 관련된 로직 구현 #38 (#41) * feat : Review 엔티티 클래스 추가 * feat : ReviewRepository 인터페이스 추가 * feat : ReviewService 클래스 추가 * feat : ReviewService와 api 연결 * feat : Review 엔티티에 대한 테스트 코드 작성 * feat : ReviewService에 대한 테스트 코드 작성 * refactor : intellij로 자동 리팩토링 * feat : 피그마 양식에 맞게 엔티티 및 DTO 수정 * feat : 피그마에 맞게 서비스 레이어 수정 * refactor : 레이어 변경에 따른 테스트 코드 수정 * feat : 기본 생성자 추가 * Feat: 완료 대기 상태 추가 그리고 특정 시니또가 진행 중인 콜백 조회 기능 (#49) * feat: Callback 상태에 `PENDING_COMPLETE` 상태 추가 - Callback 내부 enum에 PENDING_COMPLETE 추가 - 관련 테스트 추가 - `PENDING_COMPLETE` -> `COMPLETE`는 보호자만 가능하다. 그에 대한 로직 구현 * refactor: Bad Request 는 사용 안 하도록 수정, Not Found 로 변경 * feat: 시니또가 진행중인 콜백 조회 기능 추가 - `/sinitto/accepted` - 테스트 구현 * Feat: 안부전화 서비스 api 구현 (보호자, 시니또용) (#50) * feat: HelloCall과 TimeSlot entity 생성 및 관계성 정의, 보호자의 안부전화 서비스 crud 비즈니스 로직 구현, 안부전화 서비스 이용가격 로직 구현 * feat: 시니또용 안부전화 서비스 비즈니스 로직 구현 및 안부전화 타임로그 저장 로직 구현 * feat: 안부전화 status에 완료 대기 추가, 시니또에게 책임이 있는 안부전화 리스트 비즈니스 로직 구현, 보호자가 안부전화를 완료하는 로직 추가 * fix: 시니또의 모든 안부전화 조회 리턴 타입을 List -> Page로 변경 * feat: 안부전화 컨트롤러 구현 * test: 안부전화 단위 테스트 및 repo 테스트 작성, 잘못된 엔티티 연관관계 수정, day 필드 이름 dayName으로 변경 * fix: http 요청 오류 해결 및 LocalTime 직렬화 문제 해결, senior entity 기본생성자 추가 * fix: 시니어의 안부전화를 신청한 보호자만 보고서를 조회할 수 있도록 수정 * fix: 타임로그 기록 시 시작을 누르고 종료를 누르지 않으면 오류, 종료를 누르고 다시 종료를 누르면 오류 표출 기능 구현 및 테스트코드 작성 * style: finAllBySeniorIn 사용 및 테스트코드 추가 * fix: 안부전화 timeSlot 업데이트 시, 통을 삭제하고 새로 추가하는 방식 적용 * fix: 시니또용 안부전화 리스트 페이징처리 출력 페이징 디폴트값 조정 * fix: 오타 수정, 안부전화 신청 시 존재하는 시니어 확인 조건에 상태 변수 추가 --------- * Refactor: 2차 멘토링 코멘트 반영 (#52) * refactor: URI 경로 상수화 * refactor: 카카오 API 반환값 타입 변경 * style: 카멜케이스로 통일화 * refactor: 카카오 Oauth URL 상수화 * feat: 카카오API 유효성 처리 및 에러 처리를 위한 RestTemplateResponseErrorHandler 추가 및 로그 저장 로직 구현, ConnectTimeout 및 ReadTimeout 5초로 설정 * feat: 카카오 메일이 없는 경우 에러처리 * feat: 백엔드 로그아웃 요청시 레디스에 저장된 리프레쉬 토큰을 삭제하는 로직 추가 * fix: isEmpty에서 == null로 변경 * Deploy: 자동 배포 로직(CI) 구현 (#56) Deploy: 깃액션 deploy.yml 파일 추가 * Fix: 레디스 비밀번호 추가 구현 (#58) Fix: 레디스 비밀번호 추가 --------- Co-authored-by: JIHO LEE <161289673+GitJIHO@users.noreply.github.com> Co-authored-by: 2iedo <162278702+2iedo@users.noreply.github.com>
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.
#️⃣ 연관된 이슈
📝 작업 내용
💬 리뷰 요구사항(선택)
혹시 추가할 사항이나 개선점이 있다면 코멘트 부탁드립니다 :)
⏰ 현재 버그
리팩터링 후 코드 전체 테스트코드 구동 결과 이상 없습니다.
✏ Git Close
close #51