Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
bbbang105 created a review comment on a pull request on KUSITMS-30th-TEAM-A/backend
네 매핑되어서 주입이 됩니다! 다만 기존 yaml 파일에서는 ```yaml jwt: secret: ${JWT_SECRET} redirect: onboarding: ${JWT_REDIRECT_ONBOARDING} base: ${JWT_REDIRECT_BASE} register-token: ...
bbbang105 created a review comment on a pull request on KUSITMS-30th-TEAM-A/backend
`주석을 작성 안하고도 알아볼 수 있는 코드가 좋다는 말` 을 이전에 봐서 그동안에는 주석을 최대한 작성하지 않으려 했는데요, 아무리 잘 짜려고 해도 남이 보기에는 이해하기 쉽지 않을 것 같더라구요! 그래서 API나 메서드 상단에는 앞으로 `Javadoc`을 활용해서 주석을 달아보려고 합니다 😁
bbbang105 created a review comment on a pull request on KUSITMS-30th-TEAM-A/backend
네 저는 그렇게 하고 있어요!
bbbang105 created a review comment on a pull request on KUSITMS-30th-TEAM-A/backend
찾아보니 이 부분은 사람마다 차이가 많은 것 같아서 저희가 결정하면 될 것 같습니다!! `static` : 항상 같은 값을 반환, 내부 상태가 없음, 단순한 로직일 때 유용 `Bean 등록` : 비즈니스 로직과 연관이 깊음, 내부 상태가 있음 ..
bbbang105 created a review comment on a pull request on muhandojeon/The-Software-Craftsman
감사합니다 ㅎㅎ 👍🏻 👍🏻 👍🏻
bbbang105 created a review comment on a pull request on muhandojeon/The-Software-Craftsman
맞아요 저도 공부하면서 재미있을 때도 실제로 개발할 때 도움이 된 적도 있는 것 같습니다!! 해야 하는 것에는 동의하기 때문에... 우선은 열심히 해야겠네요!! 😁
bbbang105 created a review comment on a pull request on KUSITMS-30th-TEAM-A/backend
OAuth 핸들러에서 모두 처리하던 부분을 UserService에서 위임하도록 하였습니다! 하고 보니 여기서는 서비스 레이어에서 cookie를 설정하고 있네요...이 부분 어떻게 개선하면 좋을지 고민해봐야겠습니다 🤔
bbbang105 created a review comment on a pull request on KUSITMS-30th-TEAM-A/backend
불필요한 트랜잭션 제거하였습니다.
bbbang105 created a review on a pull request on KUSITMS-30th-TEAM-A/backend
리팩토링 한 부분들에 대해서 설명 달아두었습니다!
bbbang105 created a review on a pull request on KUSITMS-30th-TEAM-A/backend
리팩토링 한 부분들에 대해서 설명 달아두었습니다!
bbbang105 opened a pull request on KUSITMS-30th-TEAM-A/backend
[refactor] : Auth, User, Jwt 관련 코드 리팩토링을 진행한다
## ✅ PR 유형 어떤 변경 사항이 있었나요? - [x] 새로운 기능 추가 - [ ] 버그 수정 - [ ] 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경) - [x] 코드 리팩토링 - [ ] 주석 추가 및 수정 - [ ] 문서 수정 - [ ] 빌드 부분 혹은 패키지 매니저 수정 - [ ] 파일 혹은 ...
bbbang105 created a review comment on a pull request on KUSITMS-30th-TEAM-A/backend
해당 로직에서는 필요가 없다고 판단하여 제거하였습니다! 트랜잭션은 데이터 무결성 보장, 동시성 제어 등에서 이점이 있다고 알고 있습니다.
bbbang105 created a review comment on a pull request on KUSITMS-30th-TEAM-A/backend
이 부분 고민해보겠습니다..! 클라이언트 측으로 친절한 응답을 해 주는 것이 좋다고 생각하여 만들고 쓰고 있는데요, 혹시 다른 좋은 방법이 있을지 궁금합니다 🤔
bbbang105 created a review comment on a pull request on KUSITMS-30th-TEAM-A/backend
해당 부분은 트랜잭션을 제거하고, 유저 서비스와 역할을 분리하는 방식으로 리팩토링하였습니다! 핸들러에서는 요청 처리와 리다이렉트만을 담당합니다.
bbbang105 pushed 8 commits to feature/#78/first-refactor KUSITMS-30th-TEAM-A/backend
- #78 [feat] : Jwt 토큰 관련 설정값을 관리한다 5f1f97d
- #78 [feat] : Global Error를 세분화한다 ba9091e
- #78 [feat] : 주석을 추가하고 유효기간을 반환하는 기능을 추가한다 09c9a03
- #78 [refactor] : 핸들러에서는 요청 처리와 리다이렉트만을 담당하도록 변경한다 4947f22
- #78 [refactor] : HTTP 프로토콜은 Presentation Layer에서 처리하도록 변경한다 a4b16f3
- #78 [refactor] : 매직넘버를 제거하고 기능을 추가한다 fbb2e2a
- #78 [feat] : Redis 관련 에러 코드를 추가한다 ded0bf2
- #78 [feat] : 테스트 코드를 수정한다 2aa9548
bbbang105 created a branch on KUSITMS-30th-TEAM-A/backend
feature/#78/first-refactor - ⚾️ 히트존(HitZone) 백엔드 레포지토리
bbbang105 created a review comment on a pull request on KUSITMS-30th-TEAM-A/backend
application layer는 어떤 프로토콜이 사용되는지 몰라도 동작할 수 있어야 하기 때문에 알 필요가 없다고 생각합니다!