Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
tackyu created a review comment on a pull request on woowacourse/java-coupon
정확성을 생각해서 `BigDecimal`을 고려했었습니다!
youngsu5582 created a comment on a pull request on woowacourse/java-coupon
클로버.. 정말정말 오랜만에 연락 드리네요.. 🥲🥲 프로젝트 데모데이 때문에 바빴는데 이번에는 채용 일정이네요,, 말해주신 문법 부분 체크해서 수정하고, 검사하는 용 Util 함수만 추가했습니다! 더 궁금한 점 있으면 남겨주세용 감사합니다! 🫡🫡
GIVEN53 pushed 1 commit to linirini woowacourse/java-coupon
- [1단계 - DB 복제와 캐시] 리니(이예린) 미션 제출합니다. (#42) * feat: 쓰기, 읽기 data source 분리 * feat: 쿠폰 도메인 구현 * refactor: 쿠폰 이름 객체... c1c02a8
GIVEN53 closed a pull request on woowacourse/java-coupon
[1단계 - DB 복제와 캐시] 리니(이예린) 미션 제출합니다.
안녕하세요, 프린! 저는 토미의 강의가 없었으면 복제 지연 해결 방안을 고민하는 흐름을 잡는 것도 어려웠을 것 같은데요, 프린은 어떠셨나요? 🥲 저는 Writer DB로 바로 쿠폰을 조회하는 방법으로 복제 지연을 해결해보려고 했는데요, 아래는 제가 고민했던 방안들과 선택의 이유를 정리해보았어요! 물론 실제 사용자의 데이터를 기반으로 판단해야...
GIVEN53 created a review comment on a pull request on woowacourse/java-coupon
<img width="916" alt="image" src="https://github.com/user-attachments/assets/f235b326-b471-4ba3-8569-e90bcccae063"> 저는 수동으로 bean에 등록하지 않아서 디버깅해봤는데 *JpaBaseConfiguration*에서 자동으로 등록하더라구요 리니도 한 번 디...
GIVEN53 created a review comment on a pull request on woowacourse/java-coupon
저도 DiscountAmount에서 검증해보려고 했는데 리니 의견처럼 생성자에서 `MinimumOrderAmount`를 불필요하게 알아야하는 점이 걸리더라구요 따로 검증 메서드를 빼고 Coupon에서 호출하게끔 구현해봤는데 이것도 썩 내키지 않아서 최종적으로는 Coupon에서 검증하도록 구현했어요 쿠폰마다 할인율 제약이 달라질 수 있다는 점도 충분히...
GIVEN53 created a review on a pull request on woowacourse/java-coupon
안녕하세요 리니~~ 런칭데이 잘 마무리하셨나요? 다사다난했던 일주일이었네요 🥲 빠르게 2단계 가보죠 화이팅!!
GIVEN53 created a review on a pull request on woowacourse/java-coupon
안녕하세요 리니~~ 런칭데이 잘 마무리하셨나요? 다사다난했던 일주일이었네요 🥲 빠르게 2단계 가보죠 화이팅!!
jongmee opened a pull request on woowacourse/java-coupon
[2단계 - DB 복제와 캐시] 미아(이종미) 미션 제출합니다.
안녕하세요 짱수! 이번 단계도 잘 부탁드립니다 🙇🏻♀️ 저는 쿠폰을 캐싱할 때 읽기 전략은 **Look aside**, 쓰기 전략은 **Write around**로 선택했습니다. 현재 사용자가 발급받은 쿠폰 목록을 조회할 때 쿠폰 정보를 캐시에서 조회하고 있습니다. 따라서 쿠폰 발급 기간에만 쿠폰 정보가 조회될 것이므로 쿠폰을 생성할 때...
JINU-CHANG pushed 1 commit to tsulocalize woowacourse/java-coupon
- [1단계 - DB 복제와 캐시] 우주(김성현) 미션 제출합니다. (#5) * feat: reader, writer datasource route 기능 구현 * feat: coupon, member, me... 08b070d
JINU-CHANG closed a pull request on woowacourse/java-coupon
[1단계 - DB 복제와 캐시] 우주(김성현) 미션 제출합니다.
안녕하세요, 제제! 리뷰이 우주입니다. 이번 1단계 미션에서 도메인 제한 사항이 영향을 주지 않는 것 같아 우선 제외하고 구현해보았습니다! 잘 부탁드리고 방끗 웃는 하루 되시길 바래요 😄
JINU-CHANG created a review on a pull request on woowacourse/java-coupon
이번단계 요구사항은 모두 충족한 것 같아 이만 머지하겠습니다!
tackyu created a review comment on a pull request on woowacourse/java-coupon
아직 프로덕션 환경에서 적용될 코드라고 생각을 안 했던 것 같습니다! validate로 수정하는게 적절할 것 같아요 감사합니다!
ehBeak created a review comment on a pull request on woowacourse/java-coupon
네 감사합니다! 코멘트 내용을 보니, 지금 구현으로는 `23:59:59.000999999`가 되겠네요.. 원래 의도했던 `23:59:59.999999`가 되기 위해서 `withNano(999999000)`으로 설정했습니다 :)
ehBeak created a review comment on a pull request on woowacourse/java-coupon
공유 감사합니다 :) 저도 덕분에 찾아보고 학습할 수 있었습니다😊 테스트가 통과하는 이유는 `couponService.create(coupon)` 실행 시, JPA에서는 생성한 엔티티를 저장할 때, id값을 생성해서 `coupon` 객체에 넣어주기 때문입니다. 이렇게 생성해서 넣어주는 이유는 id 생성 전략을 `AutoIncrement`로 지정했...
HoeSeong123 pushed 1 commit to zeus6768 woowacourse/java-coupon
- [1단계 - 복제 지연] 제우스(신재우) 미션 제출합니다. (#47) * 기능 요구사항 작성 * coupon service test * add requirements specifications ... 7096e3e
HoeSeong123 closed a pull request on woowacourse/java-coupon
[1단계 - 복제 지연] 제우스(신재우) 미션 제출합니다.
안녕하세요 초롱~! ⚡⚡⚡ 먼저 다음의 서비스 코드에서 복제 지연을 확인했어요.  ![스크린샷 2024-10-18...
ehBeak created a review comment on a pull request on woowacourse/java-coupon
오 그렇네요! 꼼꼼한 리뷰 감사합니다 🥹🥹 넵 `TransactionSynchronizationManager.isActualTransactionActive()`을 확인하여, 중첩트랜잭션일 경우 `source`에 바로 접근하고 그렇지 않을 경우 `replica`에 접근하도록 수정했습니다 :)
HoeSeong123 created a review on a pull request on woowacourse/java-coupon
우형, DH 준비로 바쁠텐데도 정말 꼼꼼하게 잘해주셨네요!!! 이번 단계는 여기서 마무리하도록 하겠습니다ㅎㅎ 바쁘겠지만 2단계도 화이팅입니다!! 이제 진짜 우테코 마지막 미션!!!!