Skip to content

Feat/#4#5

Merged
ojy0903 merged 4 commits intodevelopfrom
feat/#4
Jan 18, 2026
Merged

Feat/#4#5
ojy0903 merged 4 commits intodevelopfrom
feat/#4

Conversation

@kingmingyu
Copy link
Collaborator

📌 관련 이슈

🚀 개요

스웨거 설정, 응답 구조 추가 및 예시 컨트롤러, 코드 컨벤션을 바탕으로 한 디렉터리 구조 추가

📄 작업 내용

구체적인 작업 내용을 설명해주세요.

  • 응답 구조 추가 및 도메인 별 에러 코드 추가
  • 스웨거 설정 추가
  • 예시 컨트롤러 및 코드 컨벤션을 바탕으로 한 디렉터리 구조 추가

📸 스크린샷 / 테스트 결과 (선택)

결과물 확인을 위한 사진이나 테스트 로그를 첨부해주세요.

image image image image

✅ 체크리스트

  • 브랜치 전략(GitHub Flow)을 준수했나요?
  • 메서드 단위로 코드가 잘 쪼개져 있나요?
  • 테스트 통과 확인
  • 서버 실행 확인
  • API 동작 확인

🔍 리뷰 포인트 (Review Points)

리뷰어가 중점적으로 확인했으면 하는 부분을 적어주세요. (P1~P4 적용 가이드)

  • 스웨거는 Docs로 따로 빼자고 했던 것 같은데 노션에는 없어서 임의로 추가해 봤습니다. 괜찮으신가요?

💬 리뷰어 가이드 (P-Rules)
P1: 필수 반영 (Critical) - 버그 가능성, 컨벤션 위반. 해결 전 머지 불가.
P2: 적극 권장 (Recommended) - 더 나은 대안 제시. 가급적 반영 권장.
P3: 제안 (Suggestion) - 아이디어 공유. 반영 여부는 드라이버 자율.
P4: 단순 확인/칭찬 (Nit) - 사소한 오타, 칭찬 등 피드백.

@kingmingyu kingmingyu self-assigned this Jan 17, 2026
@kingmingyu kingmingyu linked an issue Jan 17, 2026 that may be closed by this pull request
3 tasks
@ojy0903
Copy link
Collaborator

ojy0903 commented Jan 17, 2026

P4: 스웨거 해당 방식으로 하는 것도 좋은거 같습니다!. 그런데 build.gradle 에서 security 의존성 주석 처리 하신건 스웨거 접속 문제 때문일까요? 맞으면 제가 회원가입 구현할때 security 의존성 다시 추가하고 SecurityConfig 에서 스웨거 경로 열어두는 방식으로 해볼게요!

Copy link
Collaborator

@jinnieusLab jinnieusLab Jan 17, 2026

Choose a reason for hiding this comment

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

P3: usecase는 저번 컨벤션 회의 때 사용하지 않기로 논의했던 것 같아서 빼도 될 것 같습니다!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

빼겠습니다!

jwt:
token:
secretKey: ZGh3YWlkc2F2ZXdhZXZ3b2EgMTM5ZXUgMDMxdWMyIHEyMiBAIDAgKTJFVio=
expiration:
Copy link
Collaborator

@jinnieusLab jinnieusLab Jan 17, 2026

Choose a reason for hiding this comment

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

P2: 주요 민감 정보를 환경 변수로 관리하려는 것 같은데, 그렇다면 여기도 ${JWT_SECRET} 이런 식으로 수정해야할 것 같습니다!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

.gitignore에 application.yml이 있어서 신경 안썼었는데 application.yaml로 만든 것 같습니다 삭제하겠습니다!

@jinnieusLab
Copy link
Collaborator

jinnieusLab commented Jan 17, 2026

P4: 스웨거 구현도 논의했던 것처럼 인터페이스로 만들어서 잘 해주셨고, 응답 구조 코드도 깔끔한 것 같습니다!
그리고 계층별로 예시 코드들도 작성해주셔서 작업할 때 편리하겠네요 감사합니다!!

+) 이슈랑 PR에 labels 세팅해두는 걸 깜빡해서 세팅 후 feature로 걸어두었습니다!

@kingmingyu
Copy link
Collaborator Author

P4: 스웨거 해당 방식으로 하는 것도 좋은거 같습니다!. 그런데 build.gradle 에서 security 의존성 주석 처리 하신건 스웨거 접속 문제 때문일까요? 맞으면 제가 회원가입 구현할때 security 의존성 다시 추가하고 SecurityConfig 에서 스웨거 경로 열어두는 방식으로 해볼게요!

스웨거 편하게 보려고 빼놨었습니다 다시 추가해주시면 감사하겠습니다. 아 그리고 엔티티랑 SecurityConfig 올려주시면 저도 소셜 로그인 작업 시작하겠습니다!

@ojy0903
Copy link
Collaborator

ojy0903 commented Jan 18, 2026

P4: 스웨거 해당 방식으로 하는 것도 좋은거 같습니다!. 그런데 build.gradle 에서 security 의존성 주석 처리 하신건 스웨거 접속 문제 때문일까요? 맞으면 제가 회원가입 구현할때 security 의존성 다시 추가하고 SecurityConfig 에서 스웨거 경로 열어두는 방식으로 해볼게요!

스웨거 편하게 보려고 빼놨었습니다 다시 추가해주시면 감사하겠습니다. 아 그리고 엔티티랑 SecurityConfig 올려주시면 저도 소셜 로그인 작업 시작하겠습니다!

아 넵 그러면 제가 지금 단순 회원가입 Controller 부분이랑 SecurityConfig 수정만 조금 하면 될거 같아서 이거 PR 머지 해주시면 제가 반영해서 바로 PR 올릴게요! 문제 없으면 머지 진행해주세요!!

@ojy0903 ojy0903 merged commit ee652fa into develop Jan 18, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ Feature 새로운 기능 추가

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: 응답 구조 - 응답 구조 및 디렉터리 세팅

3 participants