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

7주차 산출물(Weekly -> Develop) #90

Merged
merged 12 commits into from
Oct 18, 2024
Merged

7주차 산출물(Weekly -> Develop) #90

merged 12 commits into from
Oct 18, 2024

Conversation

zzoe2346
Copy link
Contributor

#️⃣ 연관된 이슈

ex) #이슈번호, #이슈번호

📝 작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요.(이미지 첨부 가능)

  • TODO
  • TODO

스크린샷 (선택)

💬 리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

⏰ 현재 버그

✏ Git Close

close #이슈번호

eunsoni and others added 12 commits October 12, 2024 20:02
* feat: 가이드라인 dto(request) 추가

* refactor: 엔터티 이름 수정

* feat: GuardGuidelineRepository 추가

* refactor: 매개변수에 코드컨벤션 적용해 수정

* refactor: type 필드의 데이터 타입 String에서 enum으로 변경

* refactor: dto에서 type 필드 데이터 타입 수정

* feat: GuardGuidelineRepository 구현

* feat: GuardGuidelineService 추가 및 구현

* refactor: type 필드 종류 수정

* feat: GuardGuidelineController 구현

* feat: GuardGuidelineNotFoundException 구현

* feat: ControllerAdvice 구현

* refactor: Reformat Code

* refactor: 엔터티 테스트를 위한 getID()메서드 추가

* refactor: Reformat Code

* feat: 엔터티 테스트 코드 작성

* feat: Repository 테스트 코드 작성

* refactor: 사용하지 않는 매개변수 삭제

* refactor: 기본생성자 접근 제한자 protected로 수정

* feat: 해당 시니어에 대한 보호자 검증에 실패했을 때 발생시킬 예외클래스 구현

* refactor: 가이드라인 업데이트 및 추가 시 해당 시니어의 보호자인지 회원검증하는 로직 추가

* refactor: 기본 생성자의 접근제한자를 protected로 수정

* refactor: 엔터티만 독립적으로 테스트하도록 수정 및 @DisplayName 추가

* refactor: GuardGuidelineRepositoryTest에 @DisplayName 추가
* feat: 새로운 멤버 생성시 멤버에 포인트 연관 시킨다

* fix: 실수로 이상하게 머지된것 정상적으로 수정

* build: 타임리프 의존성 추가

* feat: 포인트 조회 기능

* feat: 포인트 로그 조회 기능

- 페이징으로 조회

* feat: 포인트 충전 기능 구현

- 관리자 페이지 구현
- 테스트 구현

* feat: 콜백, 안부전화 완료로 인한 시니또 포인트 적립 기능

* feat: 서비스 요청시 포인트 차감 관련 기능과 요청 취소시 차감된거 원복 기능 구현

- 포인트 값 변경을 위한 조회시 비관적 락 걸음

* fix: 머지후 수정사항

* feat: 포인트 적립 완료 상태 변화와 동시에 적립이 되도록하는 기능 추가

* refactor: 포인트 충전 관리자 페이지 path 경로 추가

- css html 분리

* feat: 충전 요청 시 이름+번호4자리를 입금 메시지로 적도록 수정, 충전 실패 상태 추가

* feat: 포인트 출금 요청
- 관리자 페이지 추가

* refactor: 안부전화 삭제 서비스로직 논리적 순서로 개선, 안쓰는 클래스 제거, 예외 처리 추가

* feat: 포인트 로그 전체 조회할 때 최신것부터 소팅하고, 상태도 모두 포함하도록 변경

* refactor: 포인트 충전 관련 메서드명 개선

* test: 포인트 로그 엔티티 테스트 추가

* feat: 출금 할때 수수료 적용

* fix: 치명적 PointLogRepository 오타 수정

* feat: Point 엔티티에 deduct() 메서드내부 로직에 보유한 포인트보다 더 많은 포인트 차감 요청에 대한 예외 추가
* chore: 멤버 더미데이터 추가

* chore: 포인트, 콜백 더미데이터 추가
- 패키지 옮김

* chore : 리뷰 더미데이터 추가

* chore : 더미데이터 수정

* chore: GuardGuideline 더미데이터 추가

* chore: twilio 테스트 위한 시니어(성훈) 추가

* chore: 안부전화 더미데이터 추가 및 refreshToken 레디스에 저장 로직 실행, 콘솔에 출력

---------

Co-authored-by: Seonghun Jeong <[email protected]>
Co-authored-by: 2iedo <[email protected]>
Co-authored-by: eunsoni <[email protected]>
chore: 안부전화 더미데이터 15세트 추가
* fix: 단순 상태변경 요청 POST -> PUT 으로 변경

* feat: 스웨거 example Value 추가
* feat: 이미 진행중 or 대기중 인 콜백이 있으면 추가적인 콜백 생성 안되게 함

* refactor: 이상해진 테스트 메서드 이름 정상화

* refactor: 결제 고객에대한 메시지 순화

* fix: withdraw 관리자 페이지에서 출금요청을 취소상태로 전환 요청하는 URL을 이상한것으로 설정해놓은 치명적 버그 FIX
* refactor: 상태 조건문 검증 로직 효율적인 방식으로 리팩토링

* refactor: 중복된 메서드 통일

* feat: 카카오 토큰 int형 필드에 NotNull추가

* fix: 오타 수정

* refactor: JsonFormat 시간 타입 kk:mm -> HH:mm으로 변경

* feat: restTemplate connection pool 설정
chore: 포인트 로그 더미데이터 추가
* feat: 콜백 단건 조회 기능 추가

* refactor: 포인트 로그 멘트 수정
* feat: 출금 요청할때 요청한 사람이 시니또인지 검증하는 로직 추가

* fix: 만에 하나 시니또가 존재하지 않으면 예외를 던지는게 아니라 임시 시니또 객체 생성하는것으로 변경(관리자 페이지 Whitelabel Error Page 방지)
…리 구현 (#89)

* refactor: Sinitto 엔티티 이름 변경 및 위치 변경

* refactor: 안부전화 시니또 객체 멤버 타입으로 변경 및 비즈니스 로직 래팩토링

* fix: 더미데이터 수정 및 예외처리 조건 추가

* feat: 포인트 예외 추가 및 더미데이터 수정

* test: 테스트코드 수정 및 확인

* chore: 안부전화에 시니또 할당 더미데이터 추가

* refactor: 코드 자동 리팩토링 적용

* fix: 메서드명 대소문자 오류 수정

* refactor: 시니또 계좌 정보 및 시니또 여부 검증 로직 순서 조정

* fix: 인텔리제이 자동 리팩토링으로 인해 바뀐 속성값 수동 변경
@zzoe2346 zzoe2346 merged commit 65a340f into Develop Oct 18, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants