Feature:: Admin 회원관리 api 추가 #291
Conversation
HyunJunSon
requested review from
linglong67,
chan99k and
gunsight1
as code owners
| @Column(name= "wash_count") | ||
| private Integer washCount; | ||
| @Column(name = "monthly_expense") | ||
| private Integer monthlyExpense; | ||
| @Column(name = "interest") |
There was a problem hiding this comment.
혹시 컬럼네임 명시하지 않아서 발생한 queryDSL 이슈가 어떤 종류였는지 말씀해주실수 있나요?
There was a problem hiding this comment.
위와 같이 에러가 발생합니다
| package com.kernel360.member.repository; | ||
|
|
||
| import com.kernel360.member.entity.Admin; | ||
| import org.springframework.data.jpa.repository.JpaRepository; | ||
|
|
||
| public interface AdminRepository extends JpaRepository<Admin,Long> { | ||
| } |
There was a problem hiding this comment.
패키지 구분하지 않고 Admin 계정을 member 패키지 내부에서 관리한다고 생각하면 될까요?
There was a problem hiding this comment.
아? 그러네요
근데 admin은 관리자 계정 관리하는 거 아니였나요?
There was a problem hiding this comment.
member 는 사용자고 admin은 관리자로 따로 둬야해서 이렇게 만드신거 같은데요. 사용자 관련 통계나, 리뷰나 댓글 작성자에 대한 정보를 열람할 수 있어야 해서요
| @RequiredArgsConstructor | ||
| public enum MemberErrorCode implements ErrorCode { | ||
|
|
||
| FAILED_NOT_MAPPING_ORDINAL_TO_NAME(HttpStatus.INTERNAL_SERVER_ERROR.value(), "EMC001", ""); |
There was a problem hiding this comment.
메시지 내용을 안 써주신 별다른 이유가 없으시다면..? 채워주셔야겠는걸요?
|
|
||
| @RestController | ||
| @RequiredArgsConstructor | ||
| @RequestMapping("/admin") |
There was a problem hiding this comment.
갑자기 눈에 들어와서 의견을 남기자면.. 사실 admin은 별도의 서비스 모듈로 띄울거라 admin이라는 경로를 추가로 넣을 필요가 없을 것 같아요
서브도메인도 받게 되면 아마도 admin.washfit.site 이런 식일 텐데 request url에 admin이 또 들어갈 필요는 없을지도??
(의견이니까 일단 넘어가셔도 됩니다ㅋㅋ)
| import java.time.LocalDate; | ||
|
|
||
| /** | ||
| * DTO for {@link com.kernel360.member.entity.Member} |
There was a problem hiding this comment.
찬규님꺼 머지된 상태라 이번 PR에 올린 코드 상에 LocalDate로 된 부분들 찾아서 수정이 필요한 변수라면 LocalDateTime 으로 변경하시고 commit 추가해주세요~
There was a problem hiding this comment.
(음.. Enum들이 중복되는 걸 보다보니, 이것도 domain에 위치해도 괜찮을 수 있었겠다는 생각이 드네요)
| private static BooleanExpression IsCarInfoNotNull() { | ||
| return member.carInfo.carNo.isNotNull(); | ||
| } | ||
|
|
||
| private static BooleanExpression IsWashInfoNotNull() { | ||
| return member.washInfo.washNo.isNotNull(); | ||
| } |
There was a problem hiding this comment.
메서드명은 소문자로 시작하는 게.. (수정해주실거죠?)
There was a problem hiding this comment.
Review 쪽이 왜 있나했는데 이거였군요 👏
| package com.kernel360.member.repository; | ||
|
|
||
| import com.kernel360.member.entity.Admin; | ||
| import org.springframework.data.jpa.repository.JpaRepository; | ||
|
|
||
| public interface AdminRepository extends JpaRepository<Admin,Long> { | ||
| } |
There was a problem hiding this comment.
아? 그러네요
근데 admin은 관리자 계정 관리하는 거 아니였나요?
💡 Motivation and Context
🔨 Modified
🌟 More
---
📋 커밋 전 체크리스트
🤟🏻 PR로 완료된 이슈
closes #277