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

v0.0.1 배포 테스트 #19

Merged
merged 135 commits into from
Sep 9, 2023
Merged

v0.0.1 배포 테스트 #19

merged 135 commits into from
Sep 9, 2023

Conversation

Shin-Jae-Yoon
Copy link
Collaborator

📌 개발 내용

  • 배포 테스트

📑 PR 포인트


👥 협업을 위한 코드리뷰

  1. 세상에 바보같은 질문은 없다.
  2. 실수를 예방하는 팀이 아니라, 실수를 잘 다루는 팀이 되자.
  3. 분업이 아닌 협업을 하자. 우린 모두 같은 문제를 이겨내기 위해 모였다.
  4. 영원한 것은 없으니 대화를 하자.
  5. 서로 존중하자.
  6. 많이 부딪치고 깨닫고 배우자.
  7. 부드럽게 설득하고 열린 마음으로 설득당해보자.

✅ 리마인더

  • 본인의 로컬에서 정상 동작하는지 확인해주세요.
  • 최신 브랜치를 Pull 받고 PR을 요청했는지 확인해주세요.
  • API가 추가되었을 경우 테스트를 하고 PR을 올려주세요.
  • Conflict가 났을 때, UI상에서 해결하지 말고, 본인 local에서 해결해주세요.
  • Commit 메시지 제대로 작성해주세요.

⚙️ 코드리뷰 룰

  • R(Request Change): 해당 블럭은 꼭 변경해주셨으면 좋겠습니다.
  • C(Comment): 웬만하면 고려해주시면 좋겠습니다.
  • Q(Question) : 해당 라인이 궁금합니다.
  • A(Approve): 반영해도 좋고 넘어가도 좋습니다. 혹은 사소한 의견입니다.

hongdosan and others added 30 commits August 31, 2023 20:32
- iamport gradle 셋팅 (DI)
- iamport YAML 셋팅 (key, secret)
- payment controller Class 및 iamport 필드 생성
- application.yml s3 profile 분리
- 결제 성공 시, 주문서 생성 API 호출하도록 함
- application-s3.yml 설정 수정
- 불필요 파일 삭제
- BaseEntity에서 BaseTimeEntity로 이름 변경
- Git EOF 방지 마지막 줄 추가
- Javadoc 설명 추가
- 명확한 핸드폰 번호 정규식으로 교체
- 테스트 코드 추가
- build.gradle 수정
[JT-33] 결제 페이지 구축 PR입니다.
- 클래스명 변경
- @controller -> @RestController 어노테이션 변경
- URL 등록
- 원인 : form 태그로 인해 요청이 가지 않음
- 해결 : form 태그 삭제
- 추가 : 요청 성공 후, 즉 결제 승인 후 서버단 추가 로직 작성
- 결제 엔티티 생성
- 결제 저장소 생성
- 결제사를 담을 Enum 생성
- 핵심필드에 대한 null 검증 메서드 생성
- Enum unmodifiable Map 수정
- Enum private 생성자 추가
kmebin and others added 29 commits September 7, 2023 18:35
- 요일로 필터링 구현
- 웹툰 제목, 작가명으로 검색 구현
- exception -> error.exception
- module-domain.application -> module-domain.service 패키지로 변경
- module.application에 application 패키지 생성
- ExceptionCode -> ExceptionStatus
[JT-57] 예외 핸들러 구현하였습니다.
@Shin-Jae-Yoon Shin-Jae-Yoon merged commit c5d3d3d into main Sep 9, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants