Ecosyste.ms: Timeline
Browse the timeline of over 8.71 Billion events for every public repo on GitHub, all the way back to 2015. Data updated hourly from GH Archive.
whxogus215 created a review comment on a pull request on next-step/spring-basic-roomescape-playground
해당 빈은 `MemberService`를 주입받고 있지만, 로직에서는 `MemberService`를 활용하고 있지 않는 것 같아요!
whxogus215 created a review comment on a pull request on next-step/spring-basic-roomescape-playground
```suggestion Member member = memberRepository.findById(loginMember.getId()) .orElseThrow(() -> new IllegalArgumentException("회원이 존재하지 않습니다.")); ``` Optional 객체를 바로 꺼내올 ...
whxogus215 created a review comment on a pull request on next-step/spring-basic-roomescape-playground
해당 메서드에서 `waitings`는 사용되고 있지 않은 것 같아요! 기존에는 `waitings`가 사용되다가 코드를 리팩토링하는 과정에서 더이상 사용하지 않게 된걸까요?
whxogus215 created a review comment on a pull request on next-step/spring-basic-roomescape-playground
`DataLoader`와 `TestDataLoader`는 스프링 부트가 실행된 이후로 필요한 데이터를 저장하는 역할을 하는 공통점을 갖고 있습니다. 두 클래스를 하나의 패키지로 묶어서 이들의 역할을 나타낼 수 있으면 좋을 것 같아요!
whxogus215 created a review comment on a pull request on next-step/spring-basic-roomescape-playground
`Waiting` 패키지만 유일하게 이름이 대문자로 시작하는 것 같아요!
whxogus215 created a review on a pull request on next-step/spring-basic-roomescape-playground
리뷰이님 미션 진행하느라 고생많으셨습니다 😄 롬복을 적절히 활용하셔서 코드가 좀 더 깔끔해보였습니다. 10주차 요구사항도 만족하는 것 같습니다! ### 브랜치 이름에 대한 저의 생각 리뷰어의 입장에서 생각했을 때는 브랜치의 이름이 `main3`이라는 것이 정확히 어떠한 커밋이 담겨있는지 유추하기 어려울 수 있을 것 같아요. 어떠한 작업을 ...
whxogus215 created a review on a pull request on next-step/spring-basic-roomescape-playground
리뷰이님 미션 진행하느라 고생많으셨습니다 😄 롬복을 적절히 활용하셔서 코드가 좀 더 깔끔해보였습니다. 10주차 요구사항도 만족하는 것 같습니다! ### 브랜치 이름에 대한 저의 생각 리뷰어의 입장에서 생각했을 때는 브랜치의 이름이 `main3`이라는 것이 정확히 어떠한 커밋이 담겨있는지 유추하기 어려울 수 있을 것 같아요. 어떠한 작업을 ...
PlusUltraCode created a review comment on a pull request on next-step/spring-basic-roomescape-playground
동일하게 추적을 높이기 위해 로깅을 추가하면 좋을거 같아요 : )
PlusUltraCode created a review comment on a pull request on next-step/spring-basic-roomescape-playground
데이터 로딩 작업에 로깅을 추가하면 추적에 용이 할거 같아요 : ) logger.info("Loading initial data..."); 처음 logger.info("Data loading completed."); 마지막
PlusUltraCode created a review comment on a pull request on next-step/spring-basic-roomescape-playground
반환값이 null 일 수도 있어 예외처리를 해주면 더욱 좋을거 같아요
PlusUltraCode created a review on a pull request on next-step/spring-basic-roomescape-playground
안녕하세요 수민씨! 리뷰어 이동호입니다. 전반적으로 패키지 분리가 잘 되어 있고, 변수명도 잘 만들어져 있어서 코드를 읽기 수월했습니다. 좋은 구조와 명확한 네이밍 덕분에 이해하기 쉬웠습니다 놓치기 쉬운 부분들에서 중점적으로 리뷰를 남겼습니다. 고생 많으셨습니다 : )
PlusUltraCode created a review on a pull request on next-step/spring-basic-roomescape-playground
안녕하세요 수민씨! 리뷰어 이동호입니다. 전반적으로 패키지 분리가 잘 되어 있고, 변수명도 잘 만들어져 있어서 코드를 읽기 수월했습니다. 좋은 구조와 명확한 네이밍 덕분에 이해하기 쉬웠습니다 놓치기 쉬운 부분들에서 중점적으로 리뷰를 남겼습니다. 고생 많으셨습니다 : )
PlusUltraCode created a review comment on a pull request on next-step/spring-basic-roomescape-playground
예전 미션 이지만 findByEmailAndPassword 는 null Pointer 예외가 터질수도 있어 null 체크 해보는게 좋을거 같아요 : ) findByEmailAndPassword 함수의 반환값을 Optional 사용하면 쉽게 처리할 수 있습니다 : )
PlusUltraCode created a review comment on a pull request on next-step/spring-basic-roomescape-playground
필드 주입 대신 생성자 주입은 어떠신가요? 생성자 주입을 하게 순환 의존성 문제를 방지할 수 있다고 합니다. 또한 final 로 선언하면 불변성을 보장할 수 있을거 같아요 : )
Anhye0n opened a pull request on next-step/spring-basic-roomescape-playground
[Spring Core] 안정현 미션제출합니다.
안녕하세요! 리뷰어님. 미션 자체는 저번보다 괜찮았지만, 7단계 분리하는 부분에서 제가 잘 분리한게 맞는지.. 하는 의심이 듭니다. 이 부분에 관해서 의견 주시면 감사할 것 같습니다!
zhy2on opened a pull request on next-step/spring-basic-roomescape-playground
[Spring Core] 오지현 미션 제출합니다.
안녕하세요 리뷰어님! 최종 미션까지 정말 수고 많으셨습니다! Jwt 관련 기능이 잘 분리가 되었는지 DataLoader 사용이 잘 되었는지 봐주시면 감사하겠습니다!
Kyuwon-Choi opened a pull request on next-step/spring-basic-roomescape-playground
[Spring Core] 최규원 미션 제출합니다.
안녕하세요 리뷰어님! 마지막 과제까지 고생 많으셨습니다! 스텝 7이 이해가 잘 안되네요.. bean에 대해 공부를 좀 더 해야할 것 같습니다! spring이 익숙치 않아 코드가 난잡하고 저도 제 코드를 100퍼센트 이해하는게 아니라서 감안하고 봐주세요!ㅠㅠ
jelee2555 opened a pull request on next-step/spring-basic-roomescape-playground
[Spring Core] 이지은 미션 제출합니다.
안녕하세요 리뷰어님! 마지막 미션까지 수고하셨습니다!! 이번 미션을 진행하면서 부족한 부분이 많았던 것 같은데 부족한 저의 코드 리뷰 잘 부탁드립니다!!
mete0rfish opened a pull request on next-step/spring-basic-roomescape-playground
[Spring Core] 윤성원 미션제출합니다.
드디어 미션이 모두 마무리됐습니다🎉 그동안 정말 고생많으셨어요~ 이번 미션은 비교적 쉬운 미션이라 금방 진행했네요. CommandLineRunner와 Profile을 통해 테스트와 프로덕션의 데이터 삽입을 나눌 수 있다는게 신기했어요 이상한 부분이나 비효율적인 부분을 남겨주시면 많은 도움이 될 것 같아요 😁
day024 opened a pull request on next-step/spring-basic-roomescape-playground
[Spring Core] 정다영 미션제출합니다.
안녕하세요 리뷰어님 이번 미션도 수고많으셨습니다! 🍀[Spring Core]()
GoToBILL opened a pull request on next-step/spring-basic-roomescape-playground
[Spring Core]주병주 미션 제출합니다.
마지막 미션인데 그동안 수고 많으셨고 남은 해커톤도 화이팅해봐요!
che7371 opened a pull request on next-step/spring-basic-roomescape-playground
[Spring Core] 오채현 미션 제출합니다.
드디어 마지막 과제가 끝이 났네요! 정말 수고 많으셨습니다. 이제 배포랑 해커톤이 남았는데 끝까지 파이팅 입니다!! *테스트는 다 통과 했습니다. 혹시 안 지켜진 요구사항이나, 수정이 필요한 부분이 보이면 말씀해주세요!
nyeroni opened a pull request on next-step/spring-basic-roomescape-playground
[Spring Core] 신예린 미션 제출합니다.
안녕하세요! 문제를 제대로 이해했는지 모르겠네요. 테스트는 모두 통과하긴 하였지만, 원하는 요구 사항을 잘 구현했는지 확인 부탁드려요!
PlusUltraCode opened a pull request on next-step/spring-basic-roomescape-playground
[Spring Core] 이동호 미션 제출합니다.
AuthConfig 및 @Import 를 통해 전역적으로 JwtUtils을 설정할 수 있었습니다. CommandLineRunner 을 활용하여 특정 프로파일만 적용할 수 있었습니다. 처음 본 코드이고 미숙하지만 리뷰 잘부탁드립니다. :)
boyekim opened a pull request on next-step/spring-basic-roomescape-playground
[Spring Core] 김수민 미션 제출합니다.
auth 패키지를 분리하며 RoomescapeApplication이 최상단이 아니게 되었고 따라서 Component Scan의 범위에서 벗어나게 되었습니다. 따라서 AuthConfig로 빈을 등록하고 싶은 클래스를 설정해준 후 `@Import`를 통해 스캔 범위 밖도 Component Scan을 해 주었습니다. 이번 미션을 통해 `@Compone...
jelee2555 created a review comment on a pull request on next-step/spring-basic-roomescape-playground
orElse(null)을 사용하신 이유가 궁금합니다! loginMember가 존재하지 않으면 waiting을 생성할 수 없어야 한다고 생각하는데 그렇다면 orElseThrow()를 사용해야 하지 않을까요?