Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[#28] 동네 인증 Controller 구현 및 Service 수정 #29

Merged
merged 4 commits into from
Oct 6, 2024
Merged

[#28] 동네 인증 Controller 구현 및 Service 수정 #29

merged 4 commits into from
Oct 6, 2024

Conversation

Srltas
Copy link
Collaborator

@Srltas Srltas commented Sep 30, 2024 •
edited
Loading

📌 Summary

  • 동네 인증 결과를 반환하는 NeighborhoodVerificationResponse를 추가하고, 새로운 동네 인증 Controller를 구현했습니다.

📝 Description

  1. Controller 구현
    • POST로 동네 인증 요청 구현
    • Session에 user id 값을 얻어서 사용하는 방식으로 구현
  2. Service, Repository 리팩토링
    • NeighborhoodVerificationResponse를 반환하도록 수정
    • @Service, @Repository를 붙여 Spring의 의존주입을 받을 수 있도록 수정
  3. 동네 인증 테스트 코드 작성
    • Parameterize를 사용해 여러 개의 파라미터 값을 검증할 수 있도록 구현

📚 References

✅ Checklist

  • 새로운 기능이나 수정된 기능에 대해 충분한 테스트를 작성했습니다.
  • 코딩 스타일 가이드를 준수했습니다.
  • 문서(주석, README 등)가 필요하다면 업데이트했습니다.

@Srltas Srltas self-assigned this Sep 30, 2024
@Srltas Srltas removed the request for review from f-lab-bradley October 1, 2024 11:18
f-lab-bradley
f-lab-bradley reviewed Oct 4, 2024
View reviewed changes
Copy link
Collaborator

@f-lab-bradley f-lab-bradley left a comment

Choose a reason for hiding this comment

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

main 에 리베이스 한 번 해주세요

@Srltas
build: springboot validation 추가
22f2a99
@Srltas
feature: 동네인증 Controller 구현
8e530c4 
- Session에서 user id만 있는 상태로 가정해 Controller 구현
- Controller 응답을 위해 application에 response DTO 생성
- Controller 응답을 위해 application의 Service 수정
- Spring의 의존주입을 받기 위해 @service, @repository 추가
@Srltas
test: 동네인증 Controller 테스트 코드
4f52305 
- parameterized를 이용해 성공 케이스만 추가
@Srltas
Copy link
Collaborator Author

Srltas commented Oct 5, 2024

main 에 리베이스 한 번 해주세요

리베이스 완료 했습니다!

@Srltas Srltas marked this pull request as ready for review October 5, 2024 01:11
@Srltas
refactor: DTO 필드 유효성 검사 추가
ab08b8a 
- 위도, 경도 최소, 최대값 설정
- neighborhood 정수값만 사용할 수 있도록 설정
@Srltas Srltas mentioned this pull request Oct 6, 2024
Merged
3 tasks
f-lab-bradley
f-lab-bradley approved these changes Oct 6, 2024
View reviewed changes
Copy link
Collaborator

@f-lab-bradley f-lab-bradley left a comment

Choose a reason for hiding this comment

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

사용자 인증 부분 바로 이어서 진행 해 주세요

@Srltas Srltas added the feature Task to add new features label Oct 6, 2024
@Srltas Srltas merged commit c126c51 into main Oct 6, 2024
1 check passed
@Srltas Srltas deleted the feature_issue_28 branch October 6, 2024 13:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@f-lab-bradley f-lab-bradley f-lab-bradley approved these changes

Assignees

@Srltas Srltas

Labels
feature Task to add new features
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Srltas @f-lab-bradley