Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
lgm1007 pushed 16 commits to master lgm1007/hhplus_ecommerce
- feat: 각 비즈니스 별 발생할 수 있는 에러코드 ApiResponse로 정리 및 누락돈 예외 케이스 추가 e60de39
- test: 통계성 상위 상품 조회 중 상품 존재하지 않은 경우에 대한 예외케이스 통합 테스트 작성 a44fd06
- feat: RestControllerAdvice를 사용하여 GlobalExceptionHandler 구현 6cc13be
- Merge branch 'feature/step8' into feature/step9 0d09190
- Merge branch 'feature/step8' into feature/step9 e4254a0
- docs: 애플리케이션의 Error Code 정의서 문서 정리 0a7a8d4
- docs: 애플리케이션의 Error Code 관리 정의서 문서 - 에러 코드 정의 및 관리 체계 내용 정리 d4ae235
- build: kotlin-logging - 로깅을 위한 라이브러리 추가 540442c
- feat: GlobalExceptionHandler 예외에 대한 에러 로그 추가 8e54257
- feat: ApiLoggingInterceptor 추가 - 인터셉터로 API Request 대한 로깅 남기기 d278e39
- modify: 저번 주 과제 피드백 반영 - 주문 로직에서 Service에게 더 많은 책임 부여 d9be469
- feat: CorsFilter 추가 - FE 연동한다는 가정하에 CORS 에러 대비하는 필터 추가 0c378a7
- docs: 필터와 인터셉터 내용 문서 정리 5e349b2
- modify: 커스텀 Exception을 data class로 정의 bb32751
- refactor: 불필요한 세미콜론 제거 53b7b59
- Merge pull request #5 from lgm1007/feature/step9 [Step9/Step10] 에러 관리 체계, GlobalErrorHandler, Filter 및 Interceptor 활... 7dd4ed4
lgm1007 closed a pull request on lgm1007/hhplus_ecommerce
[Step9/Step10] 에러 관리 체계, GlobalErrorHandler, Filter 및 Interceptor 활용, 시나리오 API 요구사항 기능 제공
## 작업사항 ### Default * 비즈니스 별 발생할 수 있는 에러 코드 정의 및 체계 구축 * 비즈니스에서 발생 가능한 상황에 대해 에러 관리 정의서 문서로 정리했습니다. ([Error 관리 정의서](https://github.com/lgm1007/hhplus_ecommerce/blob/feature/step9/docs/ERROR_...
lgm1007 created a review comment on a pull request on lgm1007/hhplus_ecommerce
ㅎㅎ 정기님 감사합니다!! 정기님도 이번 주 과제 수고하셨습니다!
ltnscp9028 created a review comment on a pull request on lgm1007/hhplus_ecommerce
우리의 DomainError 에 HttpStatus를 담지 않고,Handler에서 처리하여 보다 domain이 잘 격리된 것 같습니다 👍
ltnscp9028 created a review comment on a pull request on lgm1007/hhplus_ecommerce
* 이 아니라 특정 엔드포인트에서 동작하게 한게 좋은 것 같습니다 ~
LeeJeongGi created a review on a pull request on lgm1007/hhplus_ecommerce
이번주 과제도 역시 잘 마무리 하셨군요. 저는 들어오는 모든 요청과 응답에 대한 로깅을 필터로 구현했는데, 규명님은 인터셉터로 구현 하셨네요! 이번 과제도 잘 배우고 갑니다 :)
LeeJeongGi created a review on a pull request on lgm1007/hhplus_ecommerce
이번주 과제도 역시 잘 마무리 하셨군요. 저는 들어오는 모든 요청과 응답에 대한 로깅을 필터로 구현했는데, 규명님은 인터셉터로 구현 하셨네요! 이번 과제도 잘 배우고 갑니다 :)
lgm1007 pushed 1 commit to feature/step9 lgm1007/hhplus_ecommerce
- modify: 커스텀 Exception을 data class로 정의 bb32751
lgm1007 opened a pull request on lgm1007/hhplus_ecommerce
[Step9/Step10] 에러 관리 체계, GlobalErrorHandler, Filter 및 Interceptor 활용, 시나리오 API 요구사항 기능 제공
## 작업사항 ### Default * 비즈니스 별 발생할 수 있는 에러 코드 정의 및 체계 구축 * 비즈니스에서 발생 가능한 상황에 대해 에러 관리 정의서 문서로 정리했습니다. ([Error 관리 정의서](https://github.com/lgm1007/hhplus_ecommerce/blob/feature/step9/docs/ERROR_...
lgm1007 pushed 1 commit to feature/step8 lgm1007/hhplus_ecommerce
- modify: 주문 API ApiResponse description 수정 f98dc0e
lgm1007 pushed 1 commit to feature/step8 lgm1007/hhplus_ecommerce
- modify: 장바구니 API 발생하지 않을 예외 명세 제외 2dbd854
lgm1007 pushed 23 commits to master lgm1007/hhplus_ecommerce
- feat: 잔액 API 서비스 로직 주입하기 424877a
- Merge branch 'feature/step7' into feature/step8 66fe999
- feat: 장바구니 API 서비스 로직 주입하기 8ad1ff7
- feat: 주문 API 서비스 로직 주입하기 1d19ebd
- feat: 결제 API 서비스 로직 주입하기 d383cbd
- feat: 상품 API 서비스 로직 주입하기 24ab901
- feat: 상위 상품 통계 API 서비스 로직 주입하기 6238827
- Merge branch 'feature/step7' into feature/step8 0959c59
- Merge branch 'feature/step7' into feature/step8 8209cfc
- test: 잔액 비즈니스 통합 테스트 작성 ebfcd6d
- test: 상품 비즈니스 기능 통합 테스트 작성 461d81a
- Merge branch 'feature/step7' into feature/step8 4e7e7d8
- feat: 주문 Facade 로직에 대한 통합 테스트 작성 ecdde02
- feat: 주문 Service 로직에 대한 통합 테스트 작성 68360df
- Merge branch 'feature/step7' into feature/step8 ff54661
- test: 결제 Facade 기능에 대한 통합테스트 작성 26786e4
- test: 장바구니 비즈니스 로직 통합테스트 작성 dfe389c
- test: 통계성 상위 주문 상품 조회 기능 통합테스트 작성 7704213
- modify: 외부 플랫폼 요청 중 예외 발생했다고 가정 6f4544a
- Merge branch 'feature/step7' into feature/step8 eadb37e
- and 3 more ...
lgm1007 pushed 21 commits to feature/step7 lgm1007/hhplus_ecommerce
- feat: 잔액 API 서비스 로직 주입하기 424877a
- Merge branch 'feature/step7' into feature/step8 66fe999
- feat: 장바구니 API 서비스 로직 주입하기 8ad1ff7
- feat: 주문 API 서비스 로직 주입하기 1d19ebd
- feat: 결제 API 서비스 로직 주입하기 d383cbd
- feat: 상품 API 서비스 로직 주입하기 24ab901
- feat: 상위 상품 통계 API 서비스 로직 주입하기 6238827
- Merge branch 'feature/step7' into feature/step8 0959c59
- Merge branch 'feature/step7' into feature/step8 8209cfc
- test: 잔액 비즈니스 통합 테스트 작성 ebfcd6d
- test: 상품 비즈니스 기능 통합 테스트 작성 461d81a
- Merge branch 'feature/step7' into feature/step8 4e7e7d8
- feat: 주문 Facade 로직에 대한 통합 테스트 작성 ecdde02
- feat: 주문 Service 로직에 대한 통합 테스트 작성 68360df
- Merge branch 'feature/step7' into feature/step8 ff54661
- test: 결제 Facade 기능에 대한 통합테스트 작성 26786e4
- test: 장바구니 비즈니스 로직 통합테스트 작성 dfe389c
- test: 통계성 상위 주문 상품 조회 기능 통합테스트 작성 7704213
- modify: 외부 플랫폼 요청 중 예외 발생했다고 가정 6f4544a
- Merge branch 'feature/step7' into feature/step8 eadb37e
- and 1 more ...
lgm1007 closed a pull request on lgm1007/hhplus_ecommerce
[Step8] 비즈니스 로직 통합테스트 및 API 로직 주입하기
### 변경 사항 * 비즈니스 로직 통합 테스트 작성 * 비용 충전 / 차감 * 상품 조회 / 재고 차감 * 주문 * 결제 * 장바구니 상품 추가 / 제거 * 상위 상품 통계 조회 * API에 Facade 및 Service 주입하여 비즈니스 로직 붙이기 * 테스트 진행하면서 오류가 발생했던 문...
lgm1007 pushed 72 commits to master lgm1007/hhplus_ecommerce
- docs: API Spec 내 Swagger 캡쳐본 첨부 fd05449
- Merge branch 'feature/step6' into feature/step7 068689d
- feat: 상품 조회 비즈니스 로직 작성 - 도메인 및 JPA 엔티티 레파지토리 추가 b855863
- Merge branch 'feature/step6' into feature/step7 7a3e707
- feat: 상품 조회 비즈니스 로직 - ProductService 로직 구현 c98d589
- Merge branch 'feature/step6' into feature/step7 dd3dfa6
- docs: API SPEC 명세 문서에 특정 상품 조회 Swagger 캡쳐본 첨부 09d4c9b
- Merge branch 'feature/step6' into feature/step7 5349b1f
- feat: 특정 상품을 조회하는 비즈니스 로직 - 특정 상품 세부정보 조회 시 공유락을 걸어 Read에 대한 Lock 추가 0a0c5dd
- modify: 상품 목록 조회 시 상품 목록이 없는 경우 예외 NotFoundException 던지기 bb8209b
- modify: ProductApi 에서 구현한 비즈니스 로직 호출하도록 수정 e514287
- test: ProductDto와 ProductDetailDto의 정적 팩토리 메서드 단위 테스트 추가 a63b051
- test: ProductInfo의 정적 팩토리 메서드 단위 테스트 추가 4e0c382
- test: ProductService 상품조회 API에 대한 단위 테스트 추가 101a19e
- Merge branch 'feature/step6' into feature/step7 c46e9c6
- modify: AuditingEntity 사용하여 @CreatedDate, @LastModifiedDate 제대로 적용하도록 엔티티 수정 7556b22
- feat: 주문 Service, Repository 및 JPA Entity, 주문 저장 로직 추가 c414009
- feat: 주문 아이템 Service, Repository 및 JPA Entity, 주문 저장 로직 추가 6685ef4
- feat: 주문 및 주문 아이템 등록하는 (저장하는) Service 로직 구현 68fcc39
- Merge branch 'feature/step6' into feature/step7 7402ba8
- and 52 more ...
lgm1007 closed a pull request on lgm1007/hhplus_ecommerce
[Default/Step7] 비즈니스 로직 구현 및 단위 테스트 작성 외
### 변경 사항 * API SPEC 문서에 Swagger 캡쳐본 첨부 * [API_SPEC.md](https://github.com/lgm1007/hhplus_ecommerce/blob/feature/step7/docs/API_SPEC.md) * 비즈니스 로직 (Facace & Service) 구현 * 상품 조회 * 주...
lgm1007 pushed 3 commits to feature/step8 lgm1007/hhplus_ecommerce
lgm1007 pushed 1 commit to feature/step7 lgm1007/hhplus_ecommerce
- test: OrderFacade 단위테스트에 CartService 모킹 추가 5720ea5
lgm1007 created a comment on a pull request on lgm1007/hhplus_ecommerce
ㅋㅋㅋㅋㅋ자주 제 코드 봐주셔서 감사합니다 정기님!! 저도 아직 부족해서 팀원분들에게 많이 배우고 있습니다ㅎㅎ😌
LeeJeongGi created a review on a pull request on lgm1007/hhplus_ecommerce
시나리오 별로 통합 테스트가 너무 잘 구현되어 있는거 같아요. 그래도 자주 보던 코드 스타일?이여서 그런지 코드가 훨씬 잘 읽히고 좋았습니다 ㅎㅎ 많이 배웁니다!! :)