Skip to content

[8주차/에반] 워크북 제출합니다#56

Merged
Sangrix merged 1 commit into
UMC-Inha:evan/mainfrom
Sangrix:feature/week8
May 28, 2026
Merged

[8주차/에반] 워크북 제출합니다#56
Sangrix merged 1 commit into
UMC-Inha:evan/mainfrom
Sangrix:feature/week8

Conversation

@Sangrix
Copy link
Copy Markdown

@Sangrix Sangrix commented May 26, 2026

✅ 실습 체크리스트

  • 이론 학습을 완료하셨나요?
  • 미션 요구사항을 모두 이해하셨나요?
  • 실습을 수행하기 위한 공부를 완료하셨나요?
  • 실습 요구사항을 모두 완료하셨나요?

✅ 컨벤션 체크리스트

  • 디렉토리 구조 컨벤션을 잘 지켰나요?
  • pr 제목을 컨벤션에 맞게 작성하였나요?
  • pr에 해당되는 이슈를 연결하였나요?(중요)
  • 적절한 라벨을 설정하였나요?
  • 파트장에게 code review를 요청하기 위해 reviewer를 등록하였나요?
  • 닉네임/main 브랜치의 최신 상태를 반영하고 있는지 확인했나요?(매우 중요!)

📌 주안점

8주차 미션 추가해서 pr 다시 올렸습니다.

@Sangrix Sangrix requested a review from YoungJJun May 26, 2026 08:43
@Sangrix Sangrix self-assigned this May 26, 2026
Copy link
Copy Markdown
Collaborator

@YoungJJun YoungJJun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

8주차 피드백

  1. 미션 내용 중 SpringSecurity 적용 및 회원가입 API는 구현되었습니다!

    email, password가 DTO에 추가된것 같은데 Converter에서 DTO → Entity 변환 과정에 builder 패턴에 email, password 모두 누락되어 있습니다. 이 부분 수정 필요할 것 같아요.

    추가로 비밀번호를 BCrypt 암호화해서 저장하는 로직도 추가해야할 것 같습니다.

  2. 회원가입

    멤버가 DB에 저장되는 로직까지는 구현된 것 같은데 정책이나 선호 음식에 대한 코드가 없습니다. 이 부분도 추가해주셔야 할 것 같아요! 그리고 많은 분들이 정책을 입력 받지만 필수 약관에 대한 동의여부를 검증하는 로직을 놓치셨는데 이 점 고려해서 구현해주시면 될 것 같아요.

  3. API에 대해 필요에 따라 Public, Private 설정해주셨고 ExceptionHandling 또한 잘 구현되어서 응답또한 잘 통일될 것 같습니다.


에반 DB 구조부터 전체적으로 수정하신 것 같은데 수고하셨습니다!

이번주는 conflict로 인해 강제 머지를 시키느라 따로 수정사항만 볼 수 없어서 이번주 미션에 대해서만 피드백 드렸습니다!

다음주도 화이팅해주세요~

@Sangrix Sangrix merged commit 9383ef5 into UMC-Inha:evan/main May 28, 2026
@YoungJJun YoungJJun linked an issue May 30, 2026 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Chapter09_Spring Security - JWT, OAuth

2 participants