Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
devhoya97 created a review comment on a pull request on woowacourse/java-coupon
헉 그렇네요.. `%,d`를 사용해서 1000단위로 ,가 붙도록 변경하여 테스트 코드 통과시켰습니다!
pricelees created a review comment on a pull request on woowacourse/java-coupon
> 이 빈은 어떤 목적으로 생성하셨는지 궁금해요! 스캔할 엔티티 패키지를 명시적으로 지정해서 엔티티는 coupon.entity 패키지에 생성한다는 것을 강제할 수도 있고, application.yml에 있는 설정값 및 아래에서 만든 DataSource 빈을 사용한다는 것을 명시하고 싶었어요~!
pricelees created a review comment on a pull request on woowacourse/java-coupon
> 테스트가 꼼꼼해서 좋네요 👍👍👍 감사합니닷 ~🙇
pricelees created a review comment on a pull request on woowacourse/java-coupon
> 현재 방법 외 다른 방법으로 해결하는 방법은 없었을까요? 고민하신 방법들을 공유해주시면 좋을 것 같아요! 캐시는 사용해보지 않아서 고려조차 안했고, 캐시를 사용하지 않는다면 1. 처음부터 쓰기 DB를 사용한다. 2. 실패하는 경우에만 쓰기 DB를 사용한다 로 방법이 나뉠 것 같아요. 현재 규모에서는 1번 방법도 부하가 거의 없을 것이라...
pricelees created a review comment on a pull request on woowacourse/java-coupon
> 테스트 깨지는 부분이 있어서 확인 부탁드려요~~~~ (위 테스트 하나만 깨집니다!) 이 부분은 한참 찾아봤는데.. 도커 재시작후 한번 더 해주실 수 있을까요..? 테스트 돌아가는거 확인하고 제출했고 여전히 제 환경에서는 잘 통과해서요..ㅎㅎ;;;
pricelees created a review comment on a pull request on woowacourse/java-coupon
> indent를 줄여보는 것은 어떨까요오 스트림 안에 쓰는 것은 인덴트 1에 해당되지 않나요..? (헷갈려서 여쭤봅니다!) 일단 가독성이 좋지 않다고 판단하신 것 같아서 별도의 변수로 분리는 해두겠습니다~~
pricelees created a review comment on a pull request on woowacourse/java-coupon
> 빈 이름을 메서드명이 아닌 어노테이션을 통해 별도로 설정하신 이유가 궁금합니다! 크게 복잡한게 아니라면 가급적 명시하는 편입니다 ㅎㅎ 말씀주신대로 메서드명만 지정해도 되긴 하지만 name을 지정하는게 더 파악하기 쉽더라구요
pricelees created a review comment on a pull request on woowacourse/java-coupon
> 검증을 Coupon 엔티티 내부가 아닌 service 단에서 진행하신 이유가 궁금해요! 저도 처음에는 엔티티에 검증 로직이 있었는데, 아래 이유들로 별도의 검증을 하도록 수정했어요~ 1. 쿠폰 발급시의 제약 조건은 언제든지 바뀔 수 있다고 생각했습니다. 개인적으로 엔티티 객체는 한번 만들면 가급적 수정하고 싶지 않았습니다. 2. 요구사...
pricelees created a review comment on a pull request on woowacourse/java-coupon
> 저의 경우는 타입을 지정하지 않아도 DataSource가 적절하게 생성되는데요! 특정 타입을 지정하신 이유가 있으신가요? application.yml에 type 정보가 있는걸 못봤네요 ㅎㅎ;; 이 부분은 지우겠습니다!
pricelees created a review on a pull request on woowacourse/java-coupon
에버 안녕하세요~ 프로젝트 진행으로 제출이 늦어 죄송합니다 ㅠㅠ 남겨주신 의견들은 하나하나 꼼꼼하게 확인했습니다 ㅎㅎ 답글 달면서 느끼는데 별 생각없이 사용한 코드들이 꽤나 있더라구요! 수정할 부분은 수정했으니 다시 한번 확인 부탁드립니다 ~!
pricelees created a review on a pull request on woowacourse/java-coupon
에버 안녕하세요~ 프로젝트 진행으로 제출이 늦어 죄송합니다 ㅠㅠ 남겨주신 의견들은 하나하나 꼼꼼하게 확인했습니다 ㅎㅎ 답글 달면서 느끼는데 별 생각없이 사용한 코드들이 꽤나 있더라구요! 수정할 부분은 수정했으니 다시 한번 확인 부탁드립니다 ~!
pricelees pushed 1 commit to eun-byeol woowacourse/java-coupon
- [1단계 - DB 복제와 캐시] 조조(조은별) 미션 제출합니다. (#62) * chore: 리드미 gitignore 제거 * docs: 1단계 요구사항 정리 * feat: 쿠폰 도메인 생성 *... e3dab2b