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.

Sign up for GitHub

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

Jump to bottom

[Feat] Benefit 조회, 사용, 저장 구현 #12

Merged
merged 10 commits into from
Jan 20, 2024
Merged

[Feat] Benefit 조회, 사용, 저장 구현 #12

merged 10 commits into from
Jan 20, 2024

Conversation

choeun7
Copy link
Contributor

@choeun7 choeun7 commented Jan 20, 2024

✨ PR 유형

어떤 변경 사항이 있나요??

  • 새로운 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제

🛠️ 작업내용

  • 내가 가진 혜택 조회 api (10개 기준 페이징 나눴으니 추후 회의 통해 수정 가능)
  • 게임 성공 시 혜택 저장 api
  • 혜택 사용 (이미 사용되었는지, 만료되었는지 에러 체킹)
  • 저장된지 한 달이 지났는지 1시간마다 확인. 지났다면 expired = true로 바꿔줌 (1시간마다 확인도 회의 통해 수정 가능)

📋 추후 진행 상황

Benefit 도메인에서 할 수 있는 쪽은 끝나서, 이제 Game으로 넘어가려고 합니다. 최대한 빨리 끝내고 장소 쪽으로 넘어가겠습니다..!

📌 리뷰 포인트

예외가 발생할 수 있는 상황들을 골라내긴 했는데, 놓친 부분이 있다면 말씀해주세요

✅ Checklist

  • 커밋 메시지 컨벤션에 맞게 작성했습니다
  • 유지-보수를 위해 주석 처리를 잘 작성했습니다

@choeun7
[Feat] MemberBenefit 도메인 초기 설정
3a36fff 
Member와 Benefit을 연결하는 테이블을 초기 설정했습니다.
@choeun7
[Feat] MemberBenefitRepository 초기 설정
c430078 
Repository 파일을 만들었습니다.
@choeun7
[Chore] optional 변수명 수정
ababa7c 
userOptional -> memberOptional
@choeun7
[Feat] 혜택 저장 api
07354ba 
store 쪽은 주석처리 했고, 혜택 중복 방급 방지 예외처리 했습니다.
@choeun7
[Chore] BaseEntity 변수명 수정
4cee49b 
CreatedDate -> CreatedDate
modifiedDate -> modifiedAt
@choeun7
[Chore] Null check 방법 수정
976fd8a 
optional 방법 -> exception throw 방법 수정
@choeun7
[Feat] 혜택 사용 api
6c4c6ba 
멤버가 보유하고 있는 혜택을 사용하는 api입니다.
해당 혜택을 이미 사용한 경우, 해당하는 혜택이 없는 경우 예외 처리했습니다.
@choeun7
[Feat] 시간마다 혜택 사용기간 검사
915bdad 
멤버가 혜택을 저장한지 1달이 지났다면 만료기간 지남 체크
1시간마다 체크(임시)
@choeun7
[Feat] 내 혜택 조회 api
c055c84 
10개 기준으로 페이지 나누어서 내가 보유한 혜택을 조회하는 api입니다.
사용 가능, 사용 완료, 만료 혜택을 모두 조회합니다.
@choeun7
[Chore] build.gradle 원상태 복구
7029945 
테스트용 MariaDB 사용하던 것을 원상태로 복구하는걸 깜빡했습니다
@choeun7 choeun7 added 🚶‍♀️초은 초은이 담당 ✨ feature 기능 추가 labels Jan 20, 2024
@choeun7 choeun7 self-assigned this Jan 20, 2024
arinming
arinming approved these changes Jan 20, 2024
View reviewed changes
Copy link
Member

@arinming arinming left a comment

Choose a reason for hiding this comment

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

확인했습니다, 수고하셨습니다~!

sanggae4133
sanggae4133 approved these changes Jan 20, 2024
View reviewed changes
Copy link
Contributor

@sanggae4133 sanggae4133 left a comment

Choose a reason for hiding this comment

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

코드 확인했습니다 LGTM~

Jeongmin39
Jeongmin39 approved these changes Jan 20, 2024
View reviewed changes
Copy link
Contributor

@Jeongmin39 Jeongmin39 left a comment

Choose a reason for hiding this comment

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

확인했습니다! 고생하셨어요 ~~

@choeun7 choeun7 merged commit b1d6118 into dev Jan 20, 2024
2 checks passed
@arinming arinming deleted the feature/2 branch March 10, 2024 10:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@arinming arinming arinming approved these changes

@Jeongmin39 Jeongmin39 Jeongmin39 approved these changes

@sanggae4133 sanggae4133 sanggae4133 approved these changes

Assignees

@choeun7 choeun7

Labels
✨ feature 기능 추가 🚶‍♀️초은 초은이 담당
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@choeun7 @sanggae4133 @Jeongmin39 @arinming