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

[Feat/house-service] 빈집 거래 API 구현 #181

Merged
merged 10 commits into from
Jun 20, 2023
Merged

Conversation

dldmsql
Copy link
Member

@dldmsql dldmsql commented Jun 19, 2023

주요 작업 내용

한 줄로 정리해주세요.

빈집 게시글 CRUD 기능 구현

작업 내용

  • 기능 정의서에 따른 ERD 설계 및 엔티티 정의 / 수정
  • 게시글 내용 글자수 10,000자 제한 유효성 검사 로직을 수행하기 위한 AOP 적용
  • 게시글 목록 조회 시, 필터링을 위한 동적 쿼리 작성
  • 컨트롤러 단위 테스트 코드 작성 및 문서 발행
  • prod 디비 서버에 테이블 생성

변경점

  • 사용자 테이블에 빈집 게시글, 스크랩 연관관계 추가

이유 : 게시글 작성자와 스크랩의 주체를 파악하기 위함입니다.

  • 게시글 정적 태그 사이에서 값 추출 함수 전역으로 변경

이유 : 커뮤니티 게시판에서만 사용하던 지역 함수를 빈집 게시글에서도 사용하기 위해 전역함수로 변경하였습니다.

  • 유효성검사 AOP 등록

이유 : 컨트롤러에 요청이 바인딩되기 이전에 유효성 검사를 실행함으로써 쓰레드 자원 소모를 막습니다.

To-Do

서비스 계층 테스트 코드 작성

CheckList

  • CI를 통과했나요?
  • 리뷰어를 등록했나요?
  • 참고 레퍼런스가 있을 경우, PR 혹은 댓글로 남겼나요?

dldmsql added 9 commits June 19, 2023 23:29
기능 명세서에 따른 엔티티 생성 및 변경
글로벌 사용을 위해 함수 접근 범위를 지역에서 전역으로 수정
컨트롤러에 요청이 바인딩 되기 이전에 유효성 검사 진행
빈집 거래에 대한 상태처리는 미정의
테스트 코드 작성에 사용될 더미 데이터 정의
@dldmsql dldmsql added docs 문서 수정 feature 기능 개발 이슈 test 테스트 코드 labels Jun 19, 2023
@dldmsql dldmsql self-assigned this Jun 19, 2023
@dldmsql
Copy link
Member Author

dldmsql commented Jun 19, 2023

AOP를 이용한 유효성 검사 를 참고해서 구현하였습니다.

@dldmsql
Copy link
Member Author

dldmsql commented Jun 19, 2023

코틀린과 스프링부트 기반 유효성 검사에 대한 기초 내용은 해당 링크를 통해 공부했습니다.

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 9 Code Smells

87.2% 87.2% Coverage
0.0% 0.0% Duplication

@dldmsql dldmsql merged commit 4437806 into dev Jun 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs 문서 수정 feature 기능 개발 이슈 test 테스트 코드
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant