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

✨ [Feat]: 장소 관련 API 대대적 수정 + 로직 변경 #28

Merged
merged 24 commits into from
Dec 26, 2023
Merged

✨ [Feat]: 장소 관련 API 대대적 수정 + 로직 변경 #28

merged 24 commits into from
Dec 26, 2023

Conversation

eomgerm
Copy link
Contributor

@eomgerm eomgerm commented Dec 10, 2023

📍 PR 타입 (하나 이상 선택)

  • 기능 추가
  • 버그 수정
  • 의존성, 환경 변수, 빌드 관련 코드 업데이트
  • 기타 사소한 수정

❗️ 관련 이슈 링크

Close #25

📌 개요

  • 독립 서점 관련 API 대거 수정
  • 디자인 패턴 사용하여 API 로직 대거 수정

🔁 변경 사항

  • 너무 많습니다... 언제 한 번 코드 리뷰 합시다!

📸 스크린샷

👀 기타 더 이야기해볼 점

✅ 체크 리스트

  • PR 템플릿에 맞추어 작성했어요.
  • [] 변경 내용에 대한 테스트를 진행했어요.
  • 프로그램이 정상적으로 동작해요.
  • PR에 적절한 라벨을 선택했어요.
  • 불필요한 코드는 삭제했어요.

eomgerm added 22 commits December 5, 2023 20:15
@eomgerm
🔥 [Update]: 불필요한 API 삭제
bfdfb35
@eomgerm
🚚 [Rename]: place DTO 파일 폴더로 정리
973699f
@eomgerm
🙈 [Update]: .env 파일들 무시
2a2687b
@eomgerm
🔨 [Update]: 로컬 개발 환경에서 NODE_ENV를 local로 설정하게 수정
83ef05e
@eomgerm
👷 [CI]: 환경에 따라 .env 동적으로 로드
3ec32fc
@eomgerm
🗃️ [Update]: DB 설정 수정
71ad121
@eomgerm
Merge branch 'ci/26-run-env' into update/25-place-list
cbe8d1b
@eomgerm
✨ [Feat]: 위경도 검증 파이프 추가
b6d8f7b
@eomgerm
✨ [Feat]: 페이징 할 장소 아이디 검증 파이프 추가
ee29e3a
@eomgerm
⬆️ [Update]: 장소 목록 API 기획안에 맞추어 수정
040d456
@eomgerm
✨ [Feat]: BaseResponse 생성 패턴 변경
45afc6c
@eomgerm
✨ [Feat]: DB에서 BigInt 조회시 number로 조회되도록 변경
8266f01
@eomgerm
✨ [Feat]: ValidationPipe 추가, path param, query string은 앞으로 이 방식으로 검증
a004203
@eomgerm
🐛 [Fix]: PlaceNews가 엉뚱한 BaseEntity를 상속하는 것 수정
0c7ba77
@eomgerm
✨ [Feat]: 소식 없는 경우 에러 추가
dd75d1d
@eomgerm
🚚 [Rename]: SortCodition dto 폴더로 이동
8bfff26
@eomgerm
🚚 [Rename]: PlaceListResponse -> PlacePreviewResponse
2c99c7f 
✨ [Feat]: DTO 생성을 Converter로 위임에 따라 필요 속성들 수정
@eomgerm
🚚 [Rename]: PlaceReview~로 이름 통일
558a4cb
@eomgerm
✨ [Feat]: 장소 관련 API 대대적 수정
65a5b19
@eomgerm
✨ [Feat]: 장소 리뷰 존재 검증 Pipe 추가
7ecd000
@eomgerm
🔥 [Remove]: 불필요한 테스트 파일 삭제
15fc6fb
@eomgerm
⬆️ [Update]: 사용자 엔티티 수정
0fd6651
@eomgerm eomgerm added 📄 Docs 문서 작성 및 수정 (README.md / Swagger) ✨ Feature 기능 추가 🏗️ Infra 필요 환경 세팅 ♻️ Refactor 코드 리팩터링 📡 API API 관련 ⬆️ Update 코드 수정 labels Dec 10, 2023
@eomgerm eomgerm self-assigned this Dec 10, 2023
1000chw
1000chw approved these changes Dec 26, 2023
View reviewed changes
Copy link
Contributor

@1000chw 1000chw left a comment

Choose a reason for hiding this comment

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

👍

@eomgerm eomgerm merged commit 8fdaec5 into master Dec 26, 2023
@eomgerm eomgerm deleted the update/25-place-list branch December 26, 2023 05:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@1000chw 1000chw 1000chw approved these changes

Assignees

@eomgerm eomgerm

Labels
📡 API API 관련 ⬆️ Update 코드 수정 📄 Docs 문서 작성 및 수정 (README.md / Swagger) ✨ Feature 기능 추가 🏗️ Infra 필요 환경 세팅 ♻️ Refactor 코드 리팩터링
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

⬆️ [Update]: 독립 서점 관련 API 수정
2 participants
@eomgerm @1000chw