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.

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

✒️ 2차 스프린트 #59

Open
wants to merge 266 commits into
base: main
Choose a base branch
from
Open

✒️ 2차 스프린트 #59

wants to merge 266 commits into from

Conversation

Sehee-Lee-01
Copy link
Member

@Sehee-Lee-01 Sehee-Lee-01 commented Jan 9, 2024

✅ 요구 사항

🚀 주요 update

기상 관련 검증 및 객체 구조 변경

  • 게시글 저장할 때 request body에 날씨를 불러올 좌표값을 함께 보내야합니다! -> postman에서 확인할 수 있습니다.
  • WeatherService에 오늘의 일교차를 불러오는 메서드(getCurrentTemperatureArrange) 추가
  • Weather, WeatherDTO를 저장하는 정보에 맞게 TemperatureArrange, TemperatureArrangeDto로 이름 변경

좋아요 기능

  • api 요청 시 처음 좋아요를 누르면 좋아요 객체를 생성, 두 번째 이후부터는 좋아요의 status를 수정하는 방식으로 적용
  • 테스트 점검 완료

아바타 이미지의 validation 추가

  • ConstraintValidator를 이용한 Custom Validator 생성
    • DTO가 모두 annotation 기반으로 validation을 진행하고 있음을 고려해 통일성을 주기 위해 이 방식 채택
    • @ImageValid, ImageValidator를 사용하여 검증
  • Image Validation 적용
  • 파일 업로드 시 tomcat default 용량이 최대 1MB인 것을 10MB로 변경
    • 4K 사진의 경우 5~7MB 정도라는 정보를 토대로 기준 선정

buide ci action이 올바르게 작동하지 않은 버그 수정

  • github secrets 전면 수정
  • build CI action script 수정
  • build script 실행 시 secrets 및 environment 검증 추가
  • properties, docker-compse file 내 value 값 수정
  • 모든 브렌치를 대상으로 pull request 시 실행되도록 설정
  • Pull request 시 jacoco test report를 추가하는 scipt 추가
  • Pull request 시 reviewer, assignee, project에 대한 자동 설정
  • github project 설정 및 action script를 통해 실행
  • auto assign action이 실행되지 않아 code owner를 지정하는 방식을 변경
  • refactoring issue에 대한 template 추가
  • pr시 jacoco 결과 연동

인증 Validation 고도화

  • RFC 5322 기준의 정규 표현식 검증
  • Password 검증 기능 추가
    • 최소 한 개 이상의 영문자, 숫자, 특수문자가 각각 필요
    • 최소 길이 8, 최대 길이 30
    • DTO에서 @password 커스텀 어노테이션 기반으로 진행
  • Nickname 검증 기능 추가
    • NonNull, NotBlank
    • 회원가입 시 image 데이터가 필요 없도록 수정

💡 기타사항

  • 이메일 인증 서비 구현 중
  • likeCount 구현 예정
  • 이미지 업로드 시 이미지 저장도 트랜잭션 적용 예정
  • 게시글 수정 및 삭제 기능 구현 예정

ASak1104 and others added 30 commits January 9, 2024 06:55
email에 대한 thread 명 설정
추후 service 로직 추가 예정
🚀 좋아요 요청 및 취소 기능 작성
Copy link

📑 Jacoco 테스트 결과

Overall Project 98.48% -0.82% 🍏
Files changed 98.34% 🍏

File Coverage
Post.java 100% 🍏
SendCodeDto.java 100% 🍏
SignupDto.java 100% 🍏
LoginDto.java 100% 🍏
CertifyDto.java 100% 🍏
AvatarItemResponse.java 100% 🍏
AvatarItemRequest.java 100% 🍏
LikeResponse.java 100% 🍏
LikeRequest.java 100% 🍏
ForecastCategory.java 100% 🍏
AvatarItemController.java 100% 🍏
PostValidator.java 100% 🍏
AvatarItemService.java 100% 🍏
OotwApplication.java 100% 🍏
CoordinateValidator.java 100% 🍏
CertificateService.java 100% 🍏
UserService.java 100% 🍏
ImageControllerAdvice.java 100% 🍏
WeatherService.java 100% 🍏
PostUpdateRequest.java 100% 🍏
PostSaveRequest.java 100% 🍏
PermissionException.java 100% 🍏
LikeController.java 100% 🍏
AsyncConfig.java 100% 🍏
HttpRequestsConfigurer.java 100% 🍏
ForecastApiClient.java 100% 🍏
ForecastResultItem.java 100% 🍏
ItemType.java 100% 🍏
AvatarItem.java 100% 🍏
Sex.java 100% 🍏
WeatherController.java 100% 🍏
IncorrectCertificateException.java 100% 🍏
NonCertifiedUserException.java 100% 🍏
UserControllerAdvice.java 100% 🍏
AlreadyCertifiedUserException.java 100% 🍏
TokenProvider.java 100% 🍏
PostController.java 100% 🍏
ImageFile.java 100% 🍏
CertificateController.java 100% 🍏
UserController.java 100% 🍏
Like.java 100% 🍏
User.java 100% 🍏
Certificate.java 100% 🍏
WeatherResponse.java 100% 🍏
OotwMailSender.java 100% 🍏
AssertUtil.java 100% 🍏
DateTimeProvider.java 100% 🍏
SkyType.java 100% 🍏
Temperature.java 100% 🍏
TemperatureArrange.java 100% 🍏
Coordinate.java 100% 🍏
PtyType.java 100% 🍏
PostReadResponse.java 100% 🍏
WriterDto.java 100% 🍏
PostSaveUpdateResponse.java 100% 🍏
ImageValidator.java 96.55% -3.45% 🍏
PostService.java 96.54% -3.46% 🍏
MiniOImageServiceImpl.java 94.59% -4.5% 🍏
ImageAnnotationValidator.java 94.44% -5.56% 🍏
AvatarImageValidator.java 94% -6% 🍏
LikeService.java 93.98% -6.02% 🍏
ForecastResultErrorManager.java 93.55% 🍏
GlobalControllerAdvice.java 92.91% -0.79% 🍏
ExceptionHandlerConfigurer.java 81.82% 🍏

Copy link

📑 Jacoco 테스트 결과

Overall Project 98.48% -0.82% 🍏
Files changed 98.34% 🍏

File Coverage
Post.java 100% 🍏
SendCodeDto.java 100% 🍏
SignupDto.java 100% 🍏
LoginDto.java 100% 🍏
CertifyDto.java 100% 🍏
AvatarItemResponse.java 100% 🍏
AvatarItemRequest.java 100% 🍏
LikeResponse.java 100% 🍏
LikeRequest.java 100% 🍏
ForecastCategory.java 100% 🍏
AvatarItemController.java 100% 🍏
PostValidator.java 100% 🍏
AvatarItemService.java 100% 🍏
OotwApplication.java 100% 🍏
CoordinateValidator.java 100% 🍏
CertificateService.java 100% 🍏
UserService.java 100% 🍏
ImageControllerAdvice.java 100% 🍏
WeatherService.java 100% 🍏
PostUpdateRequest.java 100% 🍏
PostSaveRequest.java 100% 🍏
PermissionException.java 100% 🍏
LikeController.java 100% 🍏
AsyncConfig.java 100% 🍏
HttpRequestsConfigurer.java 100% 🍏
ForecastApiClient.java 100% 🍏
ForecastResultItem.java 100% 🍏
ItemType.java 100% 🍏
AvatarItem.java 100% 🍏
Sex.java 100% 🍏
WeatherController.java 100% 🍏
IncorrectCertificateException.java 100% 🍏
NonCertifiedUserException.java 100% 🍏
UserControllerAdvice.java 100% 🍏
AlreadyCertifiedUserException.java 100% 🍏
TokenProvider.java 100% 🍏
PostController.java 100% 🍏
ImageFile.java 100% 🍏
CertificateController.java 100% 🍏
UserController.java 100% 🍏
Like.java 100% 🍏
User.java 100% 🍏
Certificate.java 100% 🍏
WeatherResponse.java 100% 🍏
OotwMailSender.java 100% 🍏
AssertUtil.java 100% 🍏
DateTimeProvider.java 100% 🍏
SkyType.java 100% 🍏
Temperature.java 100% 🍏
TemperatureArrange.java 100% 🍏
Coordinate.java 100% 🍏
PtyType.java 100% 🍏
PostReadResponse.java 100% 🍏
WriterDto.java 100% 🍏
PostSaveUpdateResponse.java 100% 🍏
ImageValidator.java 96.55% -3.45% 🍏
PostService.java 96.54% -3.46% 🍏
MiniOImageServiceImpl.java 94.59% -4.5% 🍏
ImageAnnotationValidator.java 94.44% -5.56% 🍏
AvatarImageValidator.java 94% -6% 🍏
LikeService.java 93.98% -6.02% 🍏
ForecastResultErrorManager.java 93.55% 🍏
GlobalControllerAdvice.java 92.91% -0.79% 🍏
ExceptionHandlerConfigurer.java 81.82% 🍏

Copy link

📑 Jacoco 테스트 결과

Overall Project 98.48% -0.82% 🍏
Files changed 98.35% 🍏

File Coverage
Post.java 100% 🍏
SendCodeDto.java 100% 🍏
SignupDto.java 100% 🍏
LoginDto.java 100% 🍏
CertifyDto.java 100% 🍏
AvatarItemResponse.java 100% 🍏
AvatarItemRequest.java 100% 🍏
LikeResponse.java 100% 🍏
LikeRequest.java 100% 🍏
ForecastCategory.java 100% 🍏
AvatarItemController.java 100% 🍏
PostValidator.java 100% 🍏
AvatarItemService.java 100% 🍏
OotwApplication.java 100% 🍏
CoordinateValidator.java 100% 🍏
CertificateService.java 100% 🍏
UserService.java 100% 🍏
ImageControllerAdvice.java 100% 🍏
WeatherService.java 100% 🍏
PostUpdateRequest.java 100% 🍏
PostSaveRequest.java 100% 🍏
PermissionException.java 100% 🍏
LikeController.java 100% 🍏
AsyncConfig.java 100% 🍏
HttpRequestsConfigurer.java 100% 🍏
ForecastApiClient.java 100% 🍏
ForecastResultItem.java 100% 🍏
ItemType.java 100% 🍏
AvatarItem.java 100% 🍏
Sex.java 100% 🍏
WeatherController.java 100% 🍏
ExpiredCertificateException.java 100% 🍏
IncorrectCertificateException.java 100% 🍏
NonCertifiedUserException.java 100% 🍏
UserControllerAdvice.java 100% 🍏
AlreadyCertifiedUserException.java 100% 🍏
TokenProvider.java 100% 🍏
PostController.java 100% 🍏
ImageFile.java 100% 🍏
CertificateController.java 100% 🍏
UserController.java 100% 🍏
Like.java 100% 🍏
User.java 100% 🍏
Certificate.java 100% 🍏
WeatherResponse.java 100% 🍏
OotwMailSender.java 100% 🍏
AssertUtil.java 100% 🍏
DateTimeProvider.java 100% 🍏
SkyType.java 100% 🍏
Temperature.java 100% 🍏
TemperatureArrange.java 100% 🍏
Coordinate.java 100% 🍏
PtyType.java 100% 🍏
PostReadResponse.java 100% 🍏
WriterDto.java 100% 🍏
PostSaveUpdateResponse.java 100% 🍏
ImageValidator.java 96.55% -3.45% 🍏
PostService.java 96.54% -3.46% 🍏
MiniOImageServiceImpl.java 94.59% -4.5% 🍏
ImageAnnotationValidator.java 94.44% -5.56% 🍏
AvatarImageValidator.java 94% -6% 🍏
LikeService.java 93.98% -6.02% 🍏
ForecastResultErrorManager.java 93.55% 🍏
GlobalControllerAdvice.java 92.91% -0.79% 🍏
ExceptionHandlerConfigurer.java 81.82% 🍏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request refactoring code refactoring
Projects
Status: Review
Development

Successfully merging this pull request may close these issues.

3 participants