충남대 BE_윤정훈 6주차 과제 (2단계 & 3단계) #216
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
옮겨온다.
동시성 테스트 추가
ci-cd 파일 추가
Schema-hidden=true 설정을 통해 응답에서 제외했다.
API 명세 통일 작업 수행
진행한 부분 체크작업 수행
body를 반환하도록 함
사용하지 않는 예외코드 삭제
List to Page 변경
DTO로 응답 타입 변환 과정 거침
API 명세서 변경
API 수정 사항 반영
addCorsMapping 적용
yunjunghun0116
commented
Aug 1, 2024
There was a problem hiding this comment.
이와 같이 ProductPageResponse, WishProductPageResponse, GiftOrderPageResponse 를 지정하여 Page 단위로 조회를 요청하는 메서드에 대해서 페이지정보(현재 페이지, 총 페이지 수, 총 컨텐츠 수, 컨텐츠 등)를 담아 함께 반환하도록 하였습니다.
yunjunghun0116
commented
Aug 1, 2024
There was a problem hiding this comment.
Repository에서는 List -> Page 를 반환하도록 변경하여 이를 Service계층에서는 페이지정보를 바탕으로 PageResponse를 만들어 반환하도록 하였습니다.
yunjunghun0116
commented
Aug 1, 2024
There was a problem hiding this comment.
기존에 존재했던 Member 의 name 필드가 사라졌습니다
yunjunghun0116
commented
Aug 1, 2024
There was a problem hiding this comment.
WishProduct의 quantity 필드가 사라졌습니다.
누가 무엇을 몇개나? 에서 누가 무엇을?
로 명세를 변경하였고, 그 이유는 quantity가 사용되지 않는 필드임을 확인하고 삭제하기로 하였습니다.
Member 필드에 point 추가
포인트 관련 테스트 추가
포인트 API 명세 추가
asebn1
approved these changes
Aug 2, 2024
Comment on lines
+37
to
+48
| #### 2단계 | ||
|
|
||
| - [X] 배포 스크립트를 작성한다. | ||
| - [X] 클라이언트와 API 연동시 발생하는 CORS 문제에 대응한다. | ||
| - [X] 배포하여 클라이언트와 테스트를 진행한다. | ||
|
|
||
| #### 3단계 | ||
|
|
||
| - [X] 포인트는 사용자별로 보유한다. | ||
| - [X] 포인트 차감 방법 : 주문 요청시 사용할 포인트도 함께 전달받아 차감한다. | ||
| - [X] 포인트를 충전할 수 있다. | ||
|
|
Comment on lines
+30
to
+32
| .allowedOrigins("http://localhost:3000", "localhost:3000") | ||
| .allowedMethods("GET", "POST", "PUT", "DELETE") | ||
| .allowedHeaders("*") |
There was a problem hiding this comment.
제 경우 content-type : application/json 이라서 POST 요청임에도 preflight 요청이 발생하는 것을 확인하였습니다.
이를 허용해주기 위해서 OPTIONS 메서드를 추가해주었습니다!
Comment on lines
+16
to
+37
| @RestController | ||
| @RequestMapping("/api/points") | ||
| public class PointController implements PointApi { | ||
|
|
||
| private final PointService pointService; | ||
|
|
||
| public PointController(PointService pointService) { | ||
| this.pointService = pointService; | ||
| } | ||
|
|
||
| @PostMapping | ||
| public ResponseEntity<PointResponse> addPoint(@RequestAttribute("memberId") Long memberId, @Valid @RequestBody PointRequest pointRequest) { | ||
| var point = pointService.addPoint(memberId, pointRequest.point()); | ||
| return ResponseEntity.ok(point); | ||
| } | ||
|
|
||
| @GetMapping | ||
| public ResponseEntity<PointResponse> getPoint(@RequestAttribute("memberId") Long memberId) { | ||
| var point = pointService.getPoint(memberId); | ||
| return ResponseEntity.ok(point); | ||
| } | ||
| } |
Comment on lines
+33
to
+39
|
|
||
| public PointResponse getPoint(Long memberId) { | ||
| var member = memberRepository.findById(memberId) | ||
| .orElseThrow(() -> new NotFoundElementException(memberId + "를 가진 이용자가 존재하지 않습니다.")); | ||
| return PointResponse.of(member.getPoint()); | ||
| } | ||
| } |
asebn1
merged commit 30eb525
into
kakao-tech-campus-2nd-step2:yunjunghun0116
0 of 2 checks passed
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.
과제 수행과정
아마 https:배포된 주소:3000 추가하지 않을까 싶습니다.)
추가(3단계) 과제 수행과정
API 명세 일부 변경