Ecosyste.ms: Timeline

Browse the timeline of over 8.21 Billion events for every public repo on GitHub, all the way back to 2015. Data updated hourly from GH Archive.

next-step/spring-basic-roomescape-playground

byjo pushed 1 commit to yeon2lee next-step/spring-basic-roomescape-playground
  • [Spring Data JPA] 이나연 미션 제출합니다. (#14) * docs: 4단계 요구사항 분석 * build: jpa 의존성 추가 * feat: 엔티티 매핑 * feat: 연관관계 매... 6d77899

View on GitHub

byjo closed a pull request on next-step/spring-basic-roomescape-playground
[Spring Data JPA] 이나연 미션 제출합니다.
이번에 DB 스키마에서 계속 오류가 나서 미션 진행을 많이 못했네요..ㅠㅠ 5단계까지 구현 완료했고 시간 되는대로 6단계까지 구현할게요!
byjo pushed 1 commit to hafnium1923 next-step/spring-basic-roomescape-playground
  • [Spring Data JPA] 송지은 미션 제출합니다. (#13) * docs: step4 요구사항 업데이트 * feat: gradle 의존성 추가 * feat: Member 객체 엔티티 매핑 ... 0cbba1f

View on GitHub

byjo closed a pull request on next-step/spring-basic-roomescape-playground
[Spring Data JPA] 송지은 미션 제출합니다.
안녕하세요 가연~! 돌고돌아 리뷰를 받게 되었네요! JPA가 처음이라 개념을 이해하느라 시간을 많이 쓴 것 같아요,,, 대충 어떻게 돌아가는지는 알겠지만 응용할 머리가 없어서ㅋ홐ㅎㅋㅎㅋ 이것 저것 보면서 걍 클론코딩...... 왜 코드를 읽는건 쉬운데 짜는건 어려운걸까요? ㅠㅠ 저는 step5까지밖에 못했어요.. 그래도 테스트는 다 통과하더라구...
byjo pushed 1 commit to joamksh next-step/spring-basic-roomescape-playground
  • refact : 4단계 jpa 전환 (#12) c6595e8

View on GitHub

byjo closed a pull request on next-step/spring-basic-roomescape-playground
[Spring Data JPA] 김수현 미션 제출합니다
안녕하세요? 저는 이번주 과제는 jpa로 변환하면서 굉장히 빠르게 끝날 줄 알았는데... 예상치 못한 오류를 만나서 계속 고민하고 있네요 제가 봤던 기초 책에서는 jpa로만 진행을 해서 이전까지 jdbc가 오히려 낯설고 jpa는 익숙했는데 4단계는 처음부터 jpa로 하는게 아니라 jdbc->jpa로 변경하는 것이었는데 변경하는게 처음이어서 그런지 ...
Martin0o0 created a comment on a pull request on next-step/spring-basic-roomescape-playground
> 안녕하세요 현영님 미션 수고하셨습니다 !! 질문 주신 것에 대한 제 생각을 말하기 전에 먼저 [해당 글](https://www.mimul.com/blog/oop-alternative-to-utility-classes/)을 참고 해보셔서 한번 더 고민해보시면 좋을 것 같습니다 ㅎㅎ 반영하시면서 현영님 만의 답을 다시 내려보시고 제 생각도 공유 드리겠...

View on GitHub

Martin0o0 created a review comment on a pull request on next-step/spring-basic-roomescape-playground
수정하였습니다!

View on GitHub

Martin0o0 created a review on a pull request on next-step/spring-basic-roomescape-playground

View on GitHub

Martin0o0 created a review comment on a pull request on next-step/spring-basic-roomescape-playground
Default Thema의 기본키가 1이라면 private Long theme = 1L;로 설정하면 될 것 같습니다!

View on GitHub

Martin0o0 created a review on a pull request on next-step/spring-basic-roomescape-playground

View on GitHub

Martin0o0 created a review comment on a pull request on next-step/spring-basic-roomescape-playground
application.properties에 jwt.secret.key=Yn2kjibddFAWtnPJ2AFlL8WXmohJMCvigQggaEypa5E=로 설정하고, 해당 필드에 @Value("${jwt.sercret.key}")로 읽어드려 감출 수 있을 것 같습니다. 허나, application.properties 역시 그대로 노출되어 있기에, jwt....

View on GitHub

Martin0o0 created a review on a pull request on next-step/spring-basic-roomescape-playground

View on GitHub

Martin0o0 created a review comment on a pull request on next-step/spring-basic-roomescape-playground
JwtUtil 클래스가 스프링 빈으로 등록되어 스프링 컨테이너에서 각각의 객체들이 의존성 주입을 할 수 있는 환경이니, @PostContructor를 사용해서 스프링 빈으로 등록될 시점에 JwtParser를 초기화하는 방식으로 진행하면 될 것 같습니다! 따라서 공통된 JwtParser부분을 init()메서드에 초기화되도록 수정하였습니다!

View on GitHub

Martin0o0 created a review on a pull request on next-step/spring-basic-roomescape-playground

View on GitHub

Martin0o0 created a review comment on a pull request on next-step/spring-basic-roomescape-playground
바로 수정하겠습니다!

View on GitHub

Martin0o0 created a review on a pull request on next-step/spring-basic-roomescape-playground

View on GitHub

Martin0o0 created a review comment on a pull request on next-step/spring-basic-roomescape-playground
key값은 고정되었다고 생각한다면 보다 직관적일 것 같습니다!

View on GitHub

Martin0o0 created a review on a pull request on next-step/spring-basic-roomescape-playground

View on GitHub

Martin0o0 created a review comment on a pull request on next-step/spring-basic-roomescape-playground
엔드포인트 요청 시 쿠키의 토큰 값을 추출해서 사용자의 정보를 반환하여 파마리터로 들어오도록 구현한 Dto 객체입니다.!

View on GitHub

Martin0o0 created a review on a pull request on next-step/spring-basic-roomescape-playground

View on GitHub

Martin0o0 created a review comment on a pull request on next-step/spring-basic-roomescape-playground
처음에 쿠기와 jwt관련된 메서드를 Memberservice에 담아두었는데 Util클래스로 따로 빼는 과정에서 남아있던 것 같습니다. 빠르게 수정하였습니다!

View on GitHub

Martin0o0 created a review on a pull request on next-step/spring-basic-roomescape-playground

View on GitHub

dlrkdus created a review comment on a pull request on next-step/spring-basic-roomescape-playground
쿼리문 안에 생성자가 들어갈 수 있다는 사실은 저도 처음 안 것 같은데 신기하네요 .. 제 생각에는 테스트는 통과된다는 것 보면 치명적인 문제는 아닐 것 같고, 오류 문구에 ![image](https://github.com/next-step/spring-basic-roomescape-playground/assets/99721126/be86ed0...

View on GitHub

dlrkdus created a review on a pull request on next-step/spring-basic-roomescape-playground

View on GitHub

dlrkdus created a review on a pull request on next-step/spring-basic-roomescape-playground

View on GitHub

dlrkdus created a review comment on a pull request on next-step/spring-basic-roomescape-playground
오 이건 저도 처음 안 사실이네요 ... 근데 아마 명시적 목적으로 @Repository 태그를 다는 것 같아요 !

View on GitHub

dlrkdus created a review on a pull request on next-step/spring-basic-roomescape-playground

View on GitHub

dlrkdus created a review comment on a pull request on next-step/spring-basic-roomescape-playground
와우 정말 저도 1도 몰랐던 것

View on GitHub

dlrkdus created a review on a pull request on next-step/spring-basic-roomescape-playground

View on GitHub

Load more