지도/코스의 장소 수정 API , 지도 썸네일 수정 API #184
Conversation
Miensoap
requested review from
koomchang,
1119wj and
hyohyo12
as code owners
| ) { | ||
| const { comment } = updatePlaceInCourseRequest; | ||
| if (updatePlaceInCourseRequest.isEmpty()) { | ||
| throw new BadRequestException('수정할 정보가 없습니다.'); |
There was a problem hiding this comment.
q: nest 제공 에러를 쓰면 에러 코드는 어떻게 들어가나요?
There was a problem hiding this comment.
필터에서 999 로 들어갑니다!
여러 도메인에서 사용되는 에러인데, common exception 으로 만드는게 나을까요?
| @Transactional() | ||
| async setPlacesOfCourse( | ||
| id: number, | ||
| setPlacesOfCourseRequest: SetPlacesOfCourseRequest, | ||
| ) { | ||
| const course = await this.courseRepository.findById(id); | ||
| if (!course) throw new CourseNotFoundException(id); | ||
| const course = await this.getCourseOrElseThrowNotFound(id); |
| @IsOptional() | ||
| @IsUrl() | ||
| thumbnailUrl?: string; |
There was a problem hiding this comment.
p2: IsOptional 은 클라에서 아예 필드를 주지 않을 때 동작합니다.
클라에서 만약 thumnailUrl: "" 로 보내면 에러 납니다.
저는 그래서 아래와 같이 해결했었어요. 클라이언트에서도 없으면 빈값으로 보내기로 예전에 이야기 했었습니다!
// CreateCourseRequest 참고
@ValidateIf((object, value) => value !== '')
@IsUrl()
thumbnailUrl?: string;There was a problem hiding this comment.
아래와 같이 수정하겠습니다.
thumbnailUrl필드가 존재하지 않으면 업데이트에 반영하지 않는다.thumbnailUrl필드가 존재하나 빈 문자열인 경우 기본 url 로 변경한다.thumbnailUrl필드가 존재하며 유효한 url 인 경우 해당 url 로 변경한다.
backend/src/map/map.controller.ts
Outdated
| if (updateMapInfoRequest.isEmpty()) { | ||
| throw new BadRequestException('수정할 정보가 없습니다.'); | ||
| } |
There was a problem hiding this comment.
q: Request 검증은 controller의 역할이라 생각해서 service에 넣지 않고 여기 넣으신건가요?
| if (!(await this.placeRepository.existById(placeId))) { | ||
| throw new InvalidPlaceToMapException(placeId); | ||
| } | ||
| @Transactional() |
There was a problem hiding this comment.
새 객체로 교체하고 aggregate root를 저장하는 방식이라, insert, delete 쿼리가 별개로 나가더라구요
| async deletePlace(id: number, placeId: number) { | ||
| const map = await this.mapRepository.findById(id); | ||
| if (!map) throw new MapNotFoundException(id); | ||
| const map = await this.getMapOrElseThrowNotFound(id); | ||
|
|
||
| map.deletePlace(placeId); | ||
| await this.mapRepository.save(map); | ||
|
|
||
| return { deletedId: placeId }; |
There was a problem hiding this comment.
p1: 만약 위에서 필요하다면 여기도 필요할 것 같네요!
There was a problem hiding this comment.
여긴 delete 쿼리 하나만 실행되어서 없어도 될 것 같습니다.
( 삭제할 장소가 없는 경우 delete 쿼리 실행 x )
Miensoap
force-pushed
the
feature/#177
branch
from
411dbe1 to
9cecdbf
Compare
Miensoap
force-pushed
the
feature/#177
branch
from
9cecdbf to
c5ea3da
Compare
📄 Summary
🙋🏻 More
🕰️ Actual Time of Completion
close #177