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

Review #102

Merged
merged 213 commits into from
Nov 11, 2024
Merged

Review #102

merged 213 commits into from
Nov 11, 2024

Conversation

jjh4450
Copy link
Contributor

@jjh4450 jjh4450 commented Nov 10, 2024

No description provided.

peacefullyquietly and others added 30 commits November 1, 2024 14:56
@peacefullyquietly
feat : ProductController 클래스 작성
9fa94e1
@peacefullyquietly
feat : ProductService 로직 내 전체 상품 조회(getAllProducts) 메서드 추가
7af8385
@peacefullyquietly
refactor : 정렬 로직 레이어 이동(controller -> service)
88bbe51
@peacefullyquietly
feat : @EqualsAndHashCode에 callSuper 값 false로 설정
fe4f5f7
@peacefullyquietly
chore : 주석 수정
b1997d5
@peacefullyquietly
feat : ProductRepository에 필요한 메서드 추가
8e6096b
@peacefullyquietly
refactor : String 타입의 allergy, freeFroms를 List로 수정
5e8553d
@peacefullyquietly
feat : getAllProducts 메서드 분리
9ff4de7
@peacefullyquietly
chore : 주석 수정
a5f67e9
@peacefullyquietly
feat : ProductRepository 커스텀 쿼리 추가
d6a3d01
@jjh4450
Merge pull request #67 from peacefullyquietly/weekly/9
8f47798 
feat : 상품 컨트롤러
@peacefullyquietly
feat : 상품 명세에 따라 pageInfo DTO 생성
757144d
@peacefullyquietly
feat : 추가된 dto에 따른 코드 수정
7ee39d5
@peacefullyquietly
refactor : 코드 컨벤션에 따른 코드 수정
b1635e3
Merge remote-tracking branch 'upstream/weekly/9' into test/entity
af99817
Merge remote-tracking branch 'upstream/tmp' into test/entity
85dfbe0
@peacefullyquietly
refactor : DTO를 RequestBody로 받도록 수정
b1b5edb
@peacefullyquietly
refactor : productAllergies, productFreeFroms 필드 빈 리스트로 초기화
df6983d
@peacefullyquietly
refactor : 변수명 오타 수정 및 파라미터 추가
556c6e4
@peacefullyquietly
feat : 상품 생성에 관한 테스트 코드 작성
d8135a0
@peacefullyquietly
refactor : 불필요한 DTO 제거
ee18a0e
@jjh4450
Merge pull request #70 from peacefullyquietly/weekly/9
e1f353d 
refactor : product에 관한 코드 수정
Merge remote-tracking branch 'upstream/tmp' into test/entity
bc7f8a2
@nimunsang
Merge pull request #69 from kakao-tech-campus-2nd-step3/weekly/9
ee67cb7 
Weekly/9
Merge remote-tracking branch 'upstream/weekly/9' into test/entity
1ba4e18
Merge remote-tracking branch 'upstream/weekly/9' into test/entity
2efee30
@peacefullyquietly
chore : 불필요한 필드 삭제
3e84fb3
@jjh4450
Merge pull request #76 from peacefullyquietly/weekly/9
92f0587 
chore :  불필요한 필드 삭제
Merge remote-tracking branch 'upstream/weekly/9' into fix/productResp…
113ce4d 
…onseDTO
@peacefullyquietly
chore : productResponseDTO 필드 삭제에 따른 테스트 코드 수정
ec238f1
youcastle03 and others added 24 commits November 10, 2024 21:37
@youcastle03
refactor: cookie 반환 취소
ee0507e
@youcastle03
feat: blackList
2a02980
@youcastle03
feat:sameSite:None
06cedc0
@youcastle03
fix: 리프레시토큰 쿠키 만료싲간 재설정
6af155f
@youcastle03
fix:ExceptionHandler Return raw type
0e3eba3
@youcastle03
fix:wishListNotFoundException httpStatus type
362c3b6
@youcastle03
feat: redis config
5d28de9
@youcastle03
feat: redis config
7590486
@youcastle03
fix:오타 수정
4e8205e
@youcastle03
feat:BlackListRepository
a217d0c
@youcastle03
feat:TokenService
bd74d07
@youcastle03
feat:로그아웃시 블랙리스트 등록
5b7670c
@youcastle03
refactor: cookie 반환 취소
ac02e1a
@youcastle03
feat: blackList
1f526f6
@youcastle03
chore: 빈칸 수정
ead10d9
@youcastle03
충돌 해결
ce7b582
fix: merge 컨플릭트 해결
491241b
@youcastle03
fix: test용 url수정
7823f9f
@youcastle03
fix: url수정
c89b328
@youcastle03
feat:testyml에 redis정보 수정
b340dd2
@youcastle03
fix:test yml수정
d4717e2
@jjh4450
Merge pull request #103 from youcastle03/weekly/10
6e158a6 
Feat: blackList구현
@jjh4450
Merge pull request #104 from humpose/fix/mergeConflict
587004d 
refactor: 리뷰 반영
@jjh4450
Merge pull request #105 from kakao-tech-campus-2nd-step3/weekly/10
b202bbc 
동기화
@jjh4450 jjh4450 requested a review from youcastle03 November 11, 2024 04:43
@jjh4450 jjh4450 merged commit 5b24210 into dev Nov 11, 2024
1 check passed
@jjh4450 jjh4450 deleted the Review branch November 12, 2024 03:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@youcastle03 youcastle03 youcastle03 approved these changes

@humpose humpose humpose approved these changes

@peacefullyquietly peacefullyquietly peacefullyquietly approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@jjh4450 @youcastle03 @humpose @peacefullyquietly @nimunsang