Open
Conversation
[#1] Add dependency
[#3] update README.md
first main merge
이슈 템플릿 업데이트
Update issue templates title
issue templates title update
sync main to dev branch
sync main to dev branch
기본 Entity, Service, DTO 구현
기본 Entity, Service, DTO 구현
store 생성 기능 개발 완료
store 생성 기능 개발 완료
StoreJpaRepository,ProductJpaRepository 반환형 Optional -> List (여러개 가질 수 있기 때문)
StoreJpaRepository,ProductJpaRepository 반환형 Optional -> List (여러개 가질 수 있기 때문)
오탈자 수정으로 상품 등록 버그 수정
오탈자 수정으로 상품 등록 버그 수정
주문 생성 주문 내역 조회 주문 상세 조회 주문 취소
DTO 보강 작업
DTO 보강 작업
상품 목록 조회, 상품 상세조회, 상품 등록, 상품 삭제, 상품 수정 기능 개발 완료
Machanyoung/8 상품, 상점, 판매자 기능 개발 완료
병합 이후 도메인별 패키지 리팩토링/누락 패키지 import
@operation 어노테이션으로 swagger 설명 추가
Son/10 회원가입 및 인증
Member(회원 관리 기능 추가/병합)
Machanyoung/19 회원기능 병합 후 버그 수정
Role Enum, SecurityConfig 역할 통일
Machanyoung/28
엔티티 fk 어노테이션 추가
각 도메인별 엔티티 수정 및 서비스 로직->api 컨트롤러 수정
반환 타입과 여러가지 문제를 수정하였습니다.
권한 검증 시 로그인 유저 식별 로직 개선
store name 끌어오기
- 빌드 성공 - 실행 실패
store_id 인식을 위해 id로 리팩토링
Cart,Order,Seller,Store,Product 도메인 개발 완료
- [#39] 코드 설명 주석 추가
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
대부분의 도메인이 Controller, Service, Repository 처럼 REST API 의 3계층 구조로 명확히 나눠져 있어 이해하기 좋습니다.
CRUD 의미에 맞춰
insertXxx,selectXxx,selectXxxList,updateXxx,deleteXxx처럼 이름을 명확히 하면 가독성이 향상되어 읽는 사람이 동작을 바로 이해할 수 있습니다.이전 멘토링에서 언급한 것처럼 단순 조회 기능은 테이블 데이터 변경이 없으므로 트랜잭션 처리 부분은 크게 신경 쓰지 않아도 됩니다.
현재 JUnit 과 Spring Boot Test 기반 단위, 통합 테스트코드가 없습니다.
이전 멘토링에서 언급한 것처럼 추후 테스트 일정에 맞춰 핵심 기능 중심으로 추가하시면 됩니다.
개인적인 의견으로 Swagger를 사용하여 주요 API 요청/응답 예시를 포함하면 더 완벽한 문서가 될 것 같습니다.
추후 Frontend 개발자와 협업할 것을 염두해 둔다면 이 점도 고려해주세요.