20240903 본캠프 37일차 TIL

본캠프 37일차 내용 간단요약

• 09:00 ~ 10:00 : 오전회의
• 10:00 ~ 11:00 : JWT 강의
• 11:00 ~ 12:00 : 팀과제 리팩토링
• 12:00 ~ 13:00 : 점심시간
• 13:00 ~ 14:00 : 문서 수정
• 14:00 ~ 15:00 : 오류 제거
• 15:00 ~ 18:00 : 회의 및 튜터님 피드백
• 18:00 ~ 19:00 : 저녁시간
• 19: 00 ~ 25:00 : 팀과제 구현

---

오늘 해야할 일 ✔️ 🔺 ❌

 

✔️팀프로젝트 나머지 완료

---

오전회의

회의 안건

1. Dto 네이밍 관련
2. 팀원간 Dto 회의
3. 지난 밤동안 생긴 변동사항
4. pull request를 다른 팀원의 승인이 있도록 수정
5. 코드 리뷰 활성화

회의 내용

1. Dto 네이밍 관련해서 DTO를 사용할지 Dto를 사용할지 Dto를 뺄지 회의함
2. 다른 팀원(도메인)의 Dto를 사용해야 할 일이 있음, 하지만 다른팀원의 Dto를 만들어 사용하면 conflict날일이 충분함, 팀원간의 회의를 통해 Dto를 정리하기로 함
3. build.gradle에 bycrypt 추가해야함
   common > config >PasswordEncoder추가
   jwt헤더에서 userId를 받아오는 방법 설명
   회원탈퇴 기능 수정
   api명세 수정
4. pull request를 하며 다른 팀원의 승인이 있어야 merge할 수 있도록 수정함
5. pull request간 팀원간의 코드리뷰를 활성화함

의결사항

1. Dto 통일 사용
2. 해당 팀원의 Dto양식ㅇ을 따름
3. jwtUtil.getUserIdFromToken메서드를 통해 id를 가져온다
4. 금일부터 적용
5. 금일부터 적용

---

수정사항

무수히 많은 코드리뷰를 받아벌인

 

✔️ url.startWith("/api/v1/users")로 하면 /api/v1/users로 시작하는 다른 url도 필터를 거치지 않고 통과될 것 같습니다.
✔️ 사용하지 않는 주석은 삭제하시는게 좋을 것 같습니다.
✔️ Transactional 어노테이션이 빠진 것 같습니다.
✔️ void 응답의 경우에도 ResponseEntity로 감싸서 반환 할 수 있다는것 알고갑니다, 제 작업부분에도 반영하겠습니다.
✔️ 로그인 시 HttpStatus.OK가 돼야할 것 같습니다!

 

혼자 코드를 짰으면 몰랐을텐데 같이 짜니까 배우는점도 많은것 같고 놓친부분도 많았던것 같다

---

튜터님 피드백

 

🔺  상태 코드 및 에러 문구도 명시
🔺 지금은 hard delete로 되어있는데 맞을까요?

 

내일 팀원들이랑 물어볼 얘기

---

오늘의 회고 & 12시간 몰입했는가? 

회의하다 딴길로새고 회의하다 딴길로새고 껄껄.. 그래도 내 파트는 구현을 했다..!