Place crud - 먼저 처리해야 할 게 있어서 일단 보류 #33

JIWEON-JEONG · 2022-09-22T13:59:33Z

Summary

Place CRU 기능.
Delete 기능은 ADMIN 기능으로 추가할 예정.

Jira Link

https://ttbkk.atlassian.net/browse/TBK-4

Background (해당 PR을 리뷰하는데 필요한 지식/키워드)

JPA
Spring
Java

PR checklist

아래의 요구사항을 충족했다면, 체크 해주세요.
체크박스 체크 방법

작업한 내용에 대해 테스트를 진행했음.
Jira에 해당 작업물이 최신화 되어있음.
untracked files나 unstaged files에 미처 반영하지 않은 코드가 없음. (git status로 누락된 코드가 있는지 체크합니다.)

PR type

어떤 종류의 Pull Request 인가요?

Breaking change 발생 여부

데이터베이스에 존재하는 table 또는 column의 변경/삭제
사용중인 API의 변경 (하위호환이 불가능한) 하위호환이란?
기타 (내용을 적어주세요)

어떻게 테스트 할 수 있나요?

# Conflicts: # build.gradle

# Conflicts: # src/main/java/com/ttbkk/api/place/PlaceService.java

JIWEON-JEONG added 30 commits July 8, 2022 17:25

[ADD] 테스트 코드 수정 및 yml h2 서버 추가

ab9beb7

[ADD] repo placeRepo 생성(DataJpa,querydsl 동기화) , repo test code 작성

dc120bc

[CHANGE] Bigdecimal 생성 오류 방지

25d00e3

[ADD] grid api 구현 & 테스트코드 작성

e7c317e

[FIX] Controller Dto maaping error -> @NoArgsConstructor 로 해결

18f7a2d

[feat] grid api 구현 & 단위테스트 & 예외처리 생태계 구축

8a8919b

[fix] 주석 제거

eea7467

pull origin grid api

d95feb9

RestApiException 사용법 주석 추가

cf5cb0f

[ADD] 좌표 데이터 포맷 error 시 메세지에 예시 추가

5bc580f

[CHANGE] final keword 삭제

5622670

[CHANGE] finalkeword 에 대한 부분 주석 추가

fe7e1e5

[CHANGE] CommonErrorCode 수정. 예상치못한 오류 -> 500

22cb084

[CHANGE] 생성자에서 타입캐스팅 -> 서비스 단 유효성 체크하면서 타입캐스팅 으로 변경

10732eb

[ADD] GridRequestDto 주석 추가

9626eb1

Merge remote-tracking branch 'origin/main' into grid_api

027ce6b

# Conflicts: # build.gradle

[UPDATE] pull origin main

bbe212d

Merge remote-tracking branch 'origin/main' into grid_api

40585b6

[UPDATE] main pull updated

08d9efc

[CHANGE] Custom Repo -> Query Repo

58cf037

[CHANGE] Dto 변환 Service layer 에서 처리

df019ae

[ADD] 예외 테스트 추가

327b8be

[DELETE] ORM 기능 테스트 삭제

2357b13

setting pr 분리

911f262

[REMOVE] 예외처리 로직은 새로운 pr 처리

873b927

[REFACTOR] grid api get method 처리& params 유효성 처리

4da9870

[ADD] grid size 검사 로직 추가

f63f927

[ADD] grid size 검사 로직 단위테스트

9fd96c7

[ADD] 에러 환경 구축

2bcf8bc

[ADD] 예상치 못한 에러 로직 추가

fca7f37

JIWEON-JEONG added 30 commits August 24, 2022 11:27

[ADD] 기존 코드에 restControllerAdvice 적용

842b045

[UPDATE] 클래스 이름 변경 RestApi -> Base

bd21d80

디렉토리 구조 변경

de992cb

[REMOVE] 정규식 예외 발생으로 제거

61b24d6

[ADD] 좌표 form 검사

2aa3034

[ADD] 위도 경도 정수부분 검사 메서드

61d91dc

[UPDATE] compare To logic 변경

fd5ba21

[UPDATE] 줄바꿈 코드가독성 업데이트

12670c9

[UPDATE] 줄바꿈 코드가독성 업데이트

c014bd7

[REFACTOR] getPlacesAndCountInGrid

98b68a3

[UPDATE] 추가 예외 처리

05ec2b4

[FIX] AND -> OR

5073355

[ADD] checkLocationFormAndIntegerPart 테스트 코드 작성

d0dd38a

[UPDATE] 가독성 부분 수정

84a9a45

[REFACTOR] between logic 적용 , 주석추가

549b154

[REFACTOR] 변수명 변경 줄이기 X

7ad5e8f

[REMOVE] test directory

010971b

[REFACTOR] 예외처리 logic

fea1bbd

Merge branch 'handle_exception_setting' into place_crud

0b19a72

Merge branch 'grid_api_jjw' into place_crud

6530934

# Conflicts: # src/main/java/com/ttbkk/api/place/PlaceService.java

[ADD] checkLocationDecimalPart

f7b8c89

[ADD] 장소 생성 API

e5421bc

[ADD] TEST 전용 yml

777e2a0

[UPDATE] schema name

6fc1ed1

commit test

3d081eb

[REMOVE] oldException

16d6046

commit

69e9201

Merge remote-tracking branch 'origin/test_yml_setting' into place_crud

f985f12

checkout

f6f0db2

[ADD] Update Place - 중간commit

3efdcc0

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Place crud - 먼저 처리해야 할 게 있어서 일단 보류 #33

Place crud - 먼저 처리해야 할 게 있어서 일단 보류 #33

JIWEON-JEONG commented Sep 22, 2022

Place crud - 먼저 처리해야 할 게 있어서 일단 보류 #33

Are you sure you want to change the base?

Place crud - 먼저 처리해야 할 게 있어서 일단 보류 #33

Conversation

JIWEON-JEONG commented Sep 22, 2022

Summary

Jira Link

Background (해당 PR을 리뷰하는데 필요한 지식/키워드)

PR checklist

PR type

Breaking change 발생 여부

어떻게 테스트 할 수 있나요?