Ecosyste.ms: Timeline
Browse the timeline of over 8.76 Billion events for every public repo on GitHub, all the way back to 2015. Data updated hourly from GH Archive.
jelee2555 created a review comment on a pull request on next-step/spring-basic-roomescape-playground
stream을 사용하여 코드를 작성하셨네요! 저도 참고하겠습니다!
jelee2555 created a review comment on a pull request on next-step/spring-basic-roomescape-playground
public이 아닌 protected를 사용하신 이유가 궁금합니다! 혹시 public으로 선언하였을 때 발생하는 문제가 있을까요?
jelee2555 created a review comment on a pull request on next-step/spring-basic-roomescape-playground
reservation controller에서 time이랑 theme가 비어있는 값인지 확인하는 코드가 있는데 service에도 작성한 이유가 무엇인지 궁금합니다!
jelee2555 created a review on a pull request on next-step/spring-basic-roomescape-playground
안녕하세요! 리뷰가 많이 늦어져서 죄송합니다 ㅠㅠ 제가 미션을 뒤늦게 제출하게 되어 리뷰도 늦게 제출하게 되었습니다. 부족한 실력이지만, 최선을 다해 리뷰해 보았습니다. 잘 부탁드립니다!!
jelee2555 created a review on a pull request on next-step/spring-basic-roomescape-playground
안녕하세요! 리뷰가 많이 늦어져서 죄송합니다 ㅠㅠ 제가 미션을 뒤늦게 제출하게 되어 리뷰도 늦게 제출하게 되었습니다. 부족한 실력이지만, 최선을 다해 리뷰해 보았습니다. 잘 부탁드립니다!!
jelee2555 opened a pull request on next-step/spring-basic-roomescape-playground
[Spring Data JAP] 이지은 미션 제출합니다
안녕하세요, 리뷰어님! 미션을 일주일이나 늦게 제출해서 죄송합니다 ㅠㅠ 이전 미션을 다 완료하지 못하여 이전 미션을 수행하고 step4까지는 마무리 했었는데 pr을 올린 줄 알았는데 안올린걸 이제야 확인했습니다 ㅠㅠ 죄송합니다. 지금은 step5까지 미션을 진행한 상태입니다. 부족한 코드 리뷰 잘 부탁드립니다!!
whxogus215 opened a pull request on next-step/spring-basic-roomescape-playground
[Spring Core] 조태현 제출합니다.
안녕하세요 리뷰어님! 오늘로 4개월 간 진행된 스터디 미션이 마무리가 됐습니다. 그동안 미션 구현과 더불어 많은 분들의 코드 리뷰하시느라 수고 많으셨습니다. 남아있는 해커톤에서도 좋은 결과 있길 바라겠습니다 😄 저는 이번 미션을 통해 컴포넌트 스캔 방식과 스프링 프로필 설정에 대해 배울 수 있었습니다. 특히 JWT 토큰 로직이 담긴 클래스의 패...
zhy2on created a review comment on a pull request on next-step/spring-basic-roomescape-playground
주신 의견을 바탕으로 리팩토링 해보겠습니다!! 감사합니다
zhy2on created a review comment on a pull request on next-step/spring-basic-roomescape-playground
``JOIN FETCH``를 사용해서 ``지연 로딩``을 하지 않고 ``즉시 로딩``을 하기 위해서입니다..! 원래 JPA에서는 연관관계에 있는 객체의 데이터를 즉시 불러오지 않는 ``지연 로딩``을 사용합니다. https://velog.io/@jinyeong-afk/기술-면접-즉시-로딩과-지연-로딩의-차 그런데 여기선 해당 메서드를 이용...
whxogus215 created a review comment on a pull request on next-step/spring-basic-roomescape-playground
감사합니다!
whxogus215 created a review comment on a pull request on next-step/spring-basic-roomescape-playground
도메인 별로 커스텀 예외를 생성하고 예외 메시지를 `Enum`으로 분리해봤습니다! 예를 들어, 예약을 예외하는 `ReservationService`의 `save()`의 경우, 사용자의 요청 값에 따라 `time`, `theme`, `member` 값을 Repository에서 가져오는데요. 이 때, 조회한 결과 값이 존재하지 않을 경우, 예외 메시지...
whxogus215 created a review comment on a pull request on next-step/spring-basic-roomescape-playground
감사합니다 😄
whxogus215 created a review comment on a pull request on next-step/spring-basic-roomescape-playground
사용자의 입장에서 어플리케이션의 흐름을 고려했을 때, 그렇게 생각해볼 수 있을 것 같네요! 좋은 의도인 것 같습니다. 다만 리뷰이님의 패키지 구조를 봤을 때, `예약 대기`와 `예약` 간의 의존 관계가 형성되어 있는 것처럼 보일 수도 있지 않을까하는 저의 개인적인 생각이 들었습니다. 답변 감사드립니다!
kokodak opened a pull request on next-step/spring-basic-roomescape-playground
[Spring Core] 원태연, 이승용 미션 제출합니다.
kokodak closed a pull request on next-step/spring-basic-roomescape-playground
[Spring Data JPA] 원태연, 이승용 미션 제출합니다.
[Spring Data JPA 미션 커밋](https://github.com/next-step/spring-basic-roomescape-playground/pull/48/files/a9260e934d94b060f64f3336e8c382ed4352ec62..f0a357c4eb8f200e1775870de0268a6e73ef3677) 늦어서 죄송해요 🥲
day024 created a review comment on a pull request on next-step/spring-basic-roomescape-playground
저는 `waiting`을 구현하면서 복잡해지고 코드가 지저분해졌는데 예린님의 `save` 메서드는 이해하기 쉬웠던 것 같아요. 전체적으로 구조를 잘 나누셔서 `save`가 깔끔하고 직관적인 메서드가 된 것 같습니다. 코드 구조를 먼저 생각하면서 짜야한다는 것을 다시 느꼈어요👍👍
day024 created a review comment on a pull request on next-step/spring-basic-roomescape-playground
저는 WaitingController에서 대기예약을 삭제하는 `@DeleteMapping("/waitings/{id}")`를 구현하였습니다. 또한 `Reservation`에서도 ``` @DeleteMapping("/reservations/{id}")``` 를 작성했었는데요. 저는 지금 삭제기능에 대한 오류를 겪고있어 제 코드로 확인이 불가능...
day024 created a review comment on a pull request on next-step/spring-basic-roomescape-playground
`Theme`과 `Time` 에 대해서도 `Optional`을 사용하셨는데 여기서도 사용하신 이유가 뭘까요? 궁금해요!
day024 created a review comment on a pull request on next-step/spring-basic-roomescape-playground
공통 피드백이였던 `profiles`을 사용하셨군요. 확실히 보안면에서 더 좋은 것 같네요!
day024 created a review on a pull request on next-step/spring-basic-roomescape-playground
안녕하세요! 이번 미션도 수고 많으셨습니다😊 저와 다르게 진행한 점이 많아서 여러 내용 찾아 보게 되어 공부가 많이 된 것 같아요! 예린님 코드 보면서 많이 배워갑니다. 특히 저번 피드백에서 언급되었던 점들이 많이 녹아있어 제가 공부해보고 적용해야할 점들이 많이 보인 것 같아요😊 😊 저에게 이번 미션이 많이 어려웠어요 예린님은 ...
day024 created a review on a pull request on next-step/spring-basic-roomescape-playground
안녕하세요! 이번 미션도 수고 많으셨습니다😊 저와 다르게 진행한 점이 많아서 여러 내용 찾아 보게 되어 공부가 많이 된 것 같아요! 예린님 코드 보면서 많이 배워갑니다. 특히 저번 피드백에서 언급되었던 점들이 많이 녹아있어 제가 공부해보고 적용해야할 점들이 많이 보인 것 같아요😊 😊 저에게 이번 미션이 많이 어려웠어요 예린님은 ...
kokodak opened a pull request on next-step/spring-basic-roomescape-playground
fix: time.html 내의 버튼 태그 문구 수정
어드민 페이지 중, 시간 관리 페이지에서 시간을 추가하는 버튼의 문구를 `예약 추가` 에서 `시간 추가`로 수정하였습니다.