[BE] 수정 시, 현재 인원 초기화 + 방 삭제 / 수정 시 오픈 상태일때만 수정하게 변경(#637) #639

github-actions · 2024-10-19T13:21:37Z

📌 관련 이슈

closed : [BE] 수정 시, 현재 인원 초기화 + 방 삭제 / 수정 시 오픈 상태일때만 수정하게 변경 #637

✨ PR 세부 내용

이슈에 대한 내용과 똑같고
Reader / Writer 를 본격 도입했습니다.

( 애쉬가 작업하는 부분과 충돌 뜰거 같아서 Writer 에 DTO 그대로 넣었습니당 )

github-actions · 2024-10-19T14:08:04Z

Test Results

57 files 57 suites 8s ⏱️
176 tests 172 ✅ 4 💤 0 ❌
183 runs 179 ✅ 4 💤 0 ❌

Results for commit 99d72f7.

♻️ This comment has been updated with latest results.

jcoding-play

수고하셨습니당~

jcoding-play · 2024-10-21T10:53:10Z

backend/src/main/java/corea/room/domain/RoomWriter.java

+    public Room create(Member member, RoomCreateRequest request) {
+//        validateDeadLine(request.recruitmentDeadline(), request.reviewDeadline());
+        Room room = roomRepository.save(request.toEntity(member));
+        log.info("방을 생성했습니다. 방 생성자 id={}, 요청한 사용자 id={}", room.getId(), room.getManagerId());
+        return room;
+    }
+
+    public Room update(Room room, Member member, RoomUpdateRequest request) {
+        validate(room, member);
+        return roomRepository.save(request.toEntity(room, member));
+    }


Controller 단에서 전달받은 dto를 그대로 전달받았는데, 이 부분 급하다고 하니 TODO로 남기고 넘어가보시죠!~

hjk0761

다들 잘 짚어줬고, 구두로도 전달 했으니 로직 문제는 없는 것 같아 approve 합니다!!

ashsty

안녕하세요 조이썬~
로직적으로 크게 수정해야 할 것 같은 부분은 보이지 않아서 approve 드립니다!

말씀 드렸던 부분도 모두 코멘트로 남겨 두었어요. 원하시는 만큼 반영하시면 될 것 같습니다.

아쉬운 점이 있다면

기존에 잡지 못한 에러가 있어서 변경점이 있었던 건데,
이번에야말로 해당 부분에 대한 테스트가 추가되면 더 좋을 것 같으네요~~!
(수정/삭제 시 오픈 상태여야 함, 수정한 부분만 수정됨... 등등 조이썬의 상상의 나래를 응원)

ashsty · 2024-10-21T11:59:54Z

backend/src/main/java/corea/exception/ExceptionType.java

@@ -26,7 +25,7 @@ public enum ExceptionType {
    TOKEN_EXPIRED(HttpStatus.UNAUTHORIZED, "토큰이 만료되었습니다."),
    INVALID_TOKEN(HttpStatus.UNAUTHORIZED, "토큰이 올바르지 않습니다."),
    AUTHORIZATION_ERROR(HttpStatus.UNAUTHORIZED, "인증에 실패했습니다."),
-    ROOM_DELETION_AUTHORIZATION_ERROR(HttpStatus.UNAUTHORIZED, "방 삭제 권한이 없습니다. 방 생성자만 방을 삭제할 수 있습니다."),
+    ROOM_MODIFY_AUTHORIZATION_ERROR(HttpStatus.UNAUTHORIZED, "방 변경 권한이 없습니다. 방 생성자만 방을 변경할 수 있습니다."),


방 정보 변경? ㅠㅠ

ashsty · 2024-10-21T12:00:08Z

backend/src/main/java/corea/room/controller/RoomControllerSpecification.java

@@ -54,7 +54,7 @@ ResponseEntity<RoomParticipantResponses> participants(@Parameter(description = "
                    "JWT 토큰에서 추출된 사용자 정보는 방을 생성한 사용자와 일치하는지 파악합니다. " +
                    "<br><br>**참고:** 이 API를 사용하기 위해서는 유효한 JWT 토큰이 필요하며, " +
                    "토큰이 없거나 유효하지 않은 경우 인증 오류가 발생합니다.")
-    @ApiErrorResponses(value = ExceptionType.ROOM_DELETION_AUTHORIZATION_ERROR)
+    @ApiErrorResponses(value = ExceptionType.ROOM_MODIFY_AUTHORIZATION_ERROR)


방 수정 명세서에도 추가해주세요!

ashsty · 2024-10-21T12:00:31Z

backend/src/main/java/corea/room/dto/RoomUpdateRequest.java

    private static final RoomStatus INITIAL_ROOM_STATUS = RoomStatus.OPEN;

-    public Room toEntity(Member manager) {
+    public Room toEntity(Room room, Member member) {


member -> manager 👍

ashsty · 2024-10-21T12:03:05Z

backend/src/main/java/corea/room/domain/RoomWriter.java

+    private final RoomRepository roomRepository;
+    private final ParticipationWriter participationWriter;
+
+    public Room create(Member member, RoomCreateRequest request) {


마찬가지로

create, update, delete 모두
member -> manager로 변경되어도 좋을 것 같습니다~

* [BE] 방 매칭 실패 시 매칭 실패 원인을 전달하는 기능 구현(#562) (#575) * feat: 반환된 예외를 통해 실패 원인을 찾는 기능 구현 * feat: 실패한 매칭 정보를 저장하는 엔티티 구현 * feat: 매칭을 실패했을 경우 실패한 매칭 정보를 저장하는 기능 구현 * refactor: 클래스명 변경 * feat: 반환된 예외를 통해 매칭 실패 원인에 대한 메세지를 얻는 기능 구현 * refactor: FailedMatching 생성자 변경 * feat: 매칭 실패한 방을 조회시, 실패 원인을 같이 전달하는 기능 구현 * refactor: 맵 구현체 변경 * refactor: 피드백 반영 --------- Co-authored-by: gyungchan Jo <[email protected]> * [BE] 방 종료 시, 코드 리뷰 완료 버튼 동작 안하도록 하는 기능 구현(#579) (#580) * feat: 방 종료 후 코드 리뷰 완료 버튼 검증 로직 구현 * fix: 잘못된 요청값 수정 (#566) Co-authored-by: youngsu5582 <[email protected]> * refactor: 리뷰 완료 요청 검증 로직 변경 --------- Co-authored-by: gyungchan Jo <[email protected]> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: youngsu5582 <[email protected]> * [BE] 참여했던 방이 종료되면 참여 중 탭에서 제거(#581) (#584) * feat: 참여했던 방이 종료되면 참여 중 탭에서 제거하는 기능 구현 * [BE] 중복으로 매칭되던 상황 해결(#587) (#588) * refactor: 도메인 수정 * feat: 리뷰어, 리뷰이 조회 API 기능 구현 * refactor: 중복된 기능 코드 제거 * docs: 메서드 시그니쳐 통일 * feat: 시연용 데이터 추가 * refactor: 패키지 이동으로 인한 오류 수정 * feat: 시연용 데이터 추가 * fix: REMOTE_ORIGIN 그냥 변수로 변경 * feat: 데이터 추가 * feat: 서브모듈 반영 * feat: response 생성 때 reviewer, reviewee 분리 * feat: application 설정 변경 * feat: 데모 데이터 함수로 분리 * fix: 누락된 saveAll 추가 * fix: 데이터 정합성 수정 * fix: roomId 상수 변경 * feat: 피드백 키워드 뒤 .제거 * refactor: 3차 데모데이 데이터 변경 * feat: room 4에 대한 케이스도 추가 * feat: room 4 매칭 추가 * fix: 응답 내 프로필 링크로 변경 * 최신 브랜치 병합 * feat: submodule 업데이트 * refactor: 서브모듈 변경 * fix: 매칭 중복 예외 처리 안되던 오류 해결 * fix: 리뷰어 매칭 안되던 오류 수정 * test: participationFilter 로직 수정에 따른 테스트 수정 --------- Co-authored-by: hjk0761 <[email protected]> Co-authored-by: youngsu5582 <[email protected]> * feat: pr 제출 안 했을 때 문구 추가 (#590) Co-authored-by: jinsil <[email protected]> * [BE] 방 생성 검증 로직 주석 처리(#593) (#594) * refactor: 방 생성 검증 로직 주석 처리 * refactor: 운영 환경 설정 변경 * refactor: 방 생성 검증 로직 테스트 disabled 처리 --------- Co-authored-by: gyungchan Jo <[email protected]> * [BE] 방 매칭 실패 시 매칭 실패 원인을 전달하는 기능 구현(#562) (#575) * feat: 반환된 예외를 통해 실패 원인을 찾는 기능 구현 * feat: 실패한 매칭 정보를 저장하는 엔티티 구현 * feat: 매칭을 실패했을 경우 실패한 매칭 정보를 저장하는 기능 구현 * refactor: 클래스명 변경 * feat: 반환된 예외를 통해 매칭 실패 원인에 대한 메세지를 얻는 기능 구현 * refactor: FailedMatching 생성자 변경 * feat: 매칭 실패한 방을 조회시, 실패 원인을 같이 전달하는 기능 구현 * refactor: 맵 구현체 변경 * refactor: 피드백 반영 --------- Co-authored-by: gyungchan Jo <[email protected]> * [BE] 방 종료 시, 코드 리뷰 완료 버튼 동작 안하도록 하는 기능 구현(#579) (#580) * feat: 방 종료 후 코드 리뷰 완료 버튼 검증 로직 구현 * fix: 잘못된 요청값 수정 (#566) Co-authored-by: youngsu5582 <[email protected]> * refactor: 리뷰 완료 요청 검증 로직 변경 --------- Co-authored-by: gyungchan Jo <[email protected]> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: youngsu5582 <[email protected]> --------- Co-authored-by: gyungchan Jo <[email protected]> Co-authored-by: ashsty <[email protected]> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: hjk0761 <[email protected]> Co-authored-by: youngsu5582 <[email protected]> Co-authored-by: jinsil <[email protected]> * refactor: 로컬 서버용 깃허브 토큰 변경 (#600) Co-authored-by: gyungchan Jo <[email protected]> * [BE] Room Controller 역할 분리(#595) (#596) * refactor: Room Controller 역할 분리 * refactor: 피드백 반영 --------- Co-authored-by: youngsu5582 <[email protected]> * [BE] 자동 예약 및 자동 업데이트 동시성 제어(#586) (#604) * feat: Lock을 통한 자동 매칭 동시성 제어 기능 구현 * feat: Lock을 통한 자동 업데이트 동시성 제어 기능 구현 * feat: 자동 매칭 Lock 범위 최소화를 위한 클래스 구현 * feat: 자동 예약 Lock 범위 최소화를 위한 클래스 구현 * test: 자동 매칭 테스트 중복 코드 추상 클래스로 이동 * test: 자동 예약 테스트 중복 코드 추상 클래스로 이동 * refactor: Transactional 어노테이션 제거 * refactor: 피드백 반영 --------- Co-authored-by: gyungchan Jo <[email protected]> * [BE] 방 수정 기능 & 스케줄러 부분 분리 및 리팩토링(#605) (#606) * feat: repository 의존성 제거, 테스트 명확하게 변경 * feat: Reader/Writer 통해 조회 로직 분리 * feat: 룸 수정 기능 구현 * refactor: 피드백 반영 * fix: 충돌 해결 --------- Co-authored-by: youngsu5582 <[email protected]> * [FE] 콜리네 서비스 제공후 QA(#597) (#602) * fix: 모집 마감 -> 종료된 방으로 변경 * fix: 매칭 후 pr 제출 안 해서 실패했을 때 바로 문구 띄우기 * design: 모달 이름에 width 변경 * design: 프로필 드롭다운에서 이름 다 보여지게 하기 * design: 배너 medium일 때 높이 수정 * feat: timeDropdown에서 선택된 시간이 제일 위에 떠있게 하기 * feat: 피드백 모아보기에서 세부 피드백에 scroll 추가 * design: 세부 피드백 높이 지정 * fix: content 길이 길어졌을 때 ... 로 보이게 하기 * design: 피드백 모달 엔터처리, line-height 추가 * feat: 분류 드롭다운으로 선택하게 변경 * feat: 키워드 없을 때 ui 처리 * feat: 방 매칭 실패 시 서버가 준 message 띄우기 * [BE] Room Controller 역할 분리(#595) (#596) * refactor: Room Controller 역할 분리 * refactor: 피드백 반영 --------- Co-authored-by: youngsu5582 <[email protected]> * refactor: ALL 타입 추가 * feat: 수정된 api에 맞게 message 타입 추가 * fix: 방 생성 시 키워드 입력을 안했을 때 렌더링 조건 수정 * feat: 바뀐 RoomInfo 데이터 형식에 맞게 storybook 수정 --------- Co-authored-by: jinsil <[email protected]> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: youngsu5582 <[email protected]> Co-authored-by: 00kang <[email protected]> * [BE] 참여했던 방이 종료 된 후, 종료 탭 및 마이페이지에서 안보이는 문제 해결(#607) (#611) * refactor: 참여한 방을 조회하는 기능 수정 * refactor: 방을 조회하는 기능 메서드 분리 * refactor: 방을 조회 메서드 클래스 분리 * test: Nested 어노테이션을 통한 RoomServiceTest 관심사 분리 * feat: 참여했던 방이 종료되면 종료 탭에서 보이도록 하는 기능 구현 * feat: 종료 탭에서 자신이 참여했던 방도 나타내는 기능 구현 * test: 테스트 수정 * refactor: 조회 쿼리 수정 * refactor: 피드백 반영 * refactor: 변수명 변경 --------- Co-authored-by: gyungchan Jo <[email protected]> * [BE] 로그 중복 작성 제거, 비즈니스 로직 내 로그 추가(#608) (#610) * feat: additivity false 로 변경 * refactor: 참가,취소 로직 분리 * chore: 로그 레벨 변경 * feat: 로그 추가 * refactor: 피드백 반영 --------- Co-authored-by: youngsu5582 <[email protected]> * [FE] 24시간 이하로 남으면 디데이 말고 시간 보여주기(#598) (#609) * fix: 디데이가 1일 남은경우에도 24시간 미만 남은경우 남은 dday로 반환하도록 변경 * refactor: areDatesEqual 함수를 dateFormatter 유틸 함수로 이동 * fix: 중복된 함수 제거 * test: 데이터 포메팅 유틸 함수 테스트코드 추가 * test: 테스트 시 이미지 모듈은 mock string 값이 반환되도록 모킹 * test: 룸 카드의 남은 기간 정보 UI 렌더링 테스트 추가 * fix: 진행중인 방의 D-Day 가 잘못 표기되는 문제 해결 * test: 테스트 명세 오타 수정 * test: 방 상세정보의 모집/리뷰 마감까지 남은 시간 렌더링 테스트 추가 * test: jest 환경에서 시간대를 한국 시간대를 사용하도록 변경 * feat: SEO 최적화를 위해 html에 메타태그 추가 --------- Co-authored-by: Lee sang Yeop <[email protected]> * [FE] 생성페이지 달력, 시간 입력 리팩토링(#603) (#614) * design: 방 생성 페이지 모바일, 태블릿 대응 * refactor: error props 제거 * feat: DateTimePicker 생성하여 날짜, 시간 한번에 관리 * refactor: formatCombinedDateTime 인자 하나로 받기 * refactor: handleInputChange 함수로 통합하기 * refactor: 분류 type 맞추기 * feat: 방 생성 유효성 처리 --------- Co-authored-by: jinsil <[email protected]> * refactor: 초기 데이터 주석 처리 * refactor: 초기 참여자 사이즈 변경 * feat: 매칭 실패 원인을 전달하는 기능 구현 * [BE] 방 응답 시, CLASSIFICATION 반환 추가(#621) (#623) * feat: 방 정보 응답 시, classification 정보 추가 * feat: 개발 서버 액세스 토큰 변경 * test: 테스트 통과하도록 주석 처리 --------- Co-authored-by: gyungchan Jo <[email protected]> * refactor: 요청 dto 제약 조건 변경 * [FE] 방 수정 기능 추가(#616) (#620) * feat: hostType node_env로 판별 * feat: 검색 title 변경 * fix: test에 message 추가하여 오류 해결 * feat: 방 수정 api 추가 * refactor: classification 속성 추가하기 * refactor: 방 상세 정보 가져오는 함수 훅으로 분리 * feat: 방 생성, 수정 페이지 분리 * fix: put 요청으로 변경 * refactor: getInitialFormState 함수로 분리 * design: 드롭다운 선택된 것 다르게 보여주기 * refactor: 불필요한 코드 제거 * fix: roomId로 변경 * feat: roomId로 변경 --------- Co-authored-by: jinsil <[email protected]> * [FE] 참여 중인 방에 대해 바뀐 명세서 반영하기(#612) (#622) * refactor: 수정된 API명세서에 맞게 includeClosed 쿼리 추가 * feat: 참여중 라벨 추가 * feat: 참여중, 모집중 라벨 같이 보여주기 * feat: 종료, 매칭실패 라벨 같이 보여주기 * feat: 참여중 라벨을 범용적으로 사용하기 위해 참여로 수정 * refactor: 참여 라벨, 매칭실패 라벨 고려하여 추가하기 * test: roomStatus, participationStatus, message 상태에 따라 다르게 보여지는 라벨의 테스트 구현 * feat: 서비스 어느 곳에서든지 참여 라벨이 있을 경우 상세페이지로 리다이렉팅 --------- Co-authored-by: 00kang <[email protected]> Co-authored-by: Lee sang Yeop <[email protected]> * [BE] 쓴 피드백은 피드백 모아보기에서 바로 볼 수 있도록 변경 + 리팩터링(#613) (#617) * refactor: 작성한 피드백은 바로 피드백 모아보기에서 보도록 수정 * refactor: 피드백 가져오는 역할 클래스 분리 * refactor: 개발 피드백 작성 기능 수정 * refactor: 개발 피드백 수정 기능 변경 * refactor: 개발 피드백 조회 기능 수정 * refactor: 생성, 수정 dto 분리 * refactor: 피드백을 조회하는 기능 수정 * refactor: 피드백 반영 * refactor: 피드백 반영 --------- Co-authored-by: gyungchan Jo <[email protected]> * [BE] 매칭 로직 원복 및 수정(#624) (#627) * refactor: 도메인 수정 * feat: 리뷰어, 리뷰이 조회 API 기능 구현 * refactor: 중복된 기능 코드 제거 * docs: 메서드 시그니쳐 통일 * feat: 시연용 데이터 추가 * refactor: 패키지 이동으로 인한 오류 수정 * feat: 시연용 데이터 추가 * fix: REMOTE_ORIGIN 그냥 변수로 변경 * feat: 데이터 추가 * feat: 서브모듈 반영 * feat: response 생성 때 reviewer, reviewee 분리 * feat: application 설정 변경 * feat: 데모 데이터 함수로 분리 * fix: 누락된 saveAll 추가 * fix: 데이터 정합성 수정 * fix: roomId 상수 변경 * feat: 피드백 키워드 뒤 .제거 * refactor: 3차 데모데이 데이터 변경 * feat: room 4에 대한 케이스도 추가 * feat: room 4 매칭 추가 * fix: 응답 내 프로필 링크로 변경 * 최신 브랜치 병합 * feat: submodule 업데이트 * refactor: 서브모듈 변경 * feat: submodule 업데이트 * refactor: 매칭 로직 원복 * fix: 매칭 로직 오류 수정 * refactor: 변수명 직관적으로 수정 * refactor: 리뷰어를 제외하고 matchingSize 만큼 추가 매칭하도록 수정 * refactor: 리뷰어가 matchingSize 만큼 리뷰이와 매칭되도록 수정 * feat: submodule 업데이트 * refactor: 피드백 반영 * refactor: 변수명 수정 --------- Co-authored-by: hjk0761 <[email protected]> Co-authored-by: youngsu5582 <[email protected]> Co-authored-by: HyunJoong Kim <[email protected]> * [BE] 참여한 전체 방들 조회할 때, 실패 원인 같이 보내는 기능 구현(#625) (#629) * refactor: ParticipationWriter 패키지 변경 및 코드 포맷팅 적용 * refactor: 작은 단위로 클래스 분리 * refactor: 예외 타입명 변경 * refactor: 참여했지만 매칭에 실패했던 모든 방들을 조회할 때 실패 메시지를 같이 보내는 기능 구현 * refactor: 방 조회 기능 수정 * test: 방 관련 테스트 추가 --------- Co-authored-by: gyungchan Jo <[email protected]> * [FE] FallbackImage 잘 안 뜨는 문제 해결하기(#631) (#633) * fix: 로딩 전에 src 먼저 빈 문자열인지 판별 * fix: useState 초깃값으로 src 넣어주기 --------- Co-authored-by: jinsil <[email protected]> * [FE] 콜리네 서비스 제공 후 받은 피드백 반영하기 (#628) (#630) * feat: 평가 옵션의 텍스트를 아쉬움과 만족만 표시하도록 수정 * design: 피드백 모아보기 페이지의 피드백카드 디테일 수정 * feat: 피드백 모아보기 페이지의 피드백카드에 textarea 추가 * feat: 피드백 모아보기 페이지에서 키워드가 없을 때 처리 * design: 피드백 폼의 갭 수정 * design: 피드백 카드 컴포넌트에 textarea를 사용하면서 불필요한 디자인 속성 삭제 * fix: 보안 관련 경고를 해결하기 위해 target="_blank"를 사용하는 곳에 rel="noreferrer" 추가 * fix: 모달 문구 방 생성과 수정 다르게 렌더링 * refactor: EllipsisText라는 공통 컴포넌트 만들어서 텍스트가 온전히 보이지 않는 문제 해결 * design: Icons 사이즈 조정 및 단위 통일 * feat: 방카드모달에서 키워드 없을 때 처리 * design: 리뷰어, 리뷰이 테이블 아이디 가운데 정렬 * feat: 리뷰어/리뷰이 피드백 작성 모달 확인시 textarea로 보여주기 * feat: 리뷰어/리뷰이 피드백 모달 내에 키워드 없을 때 처리 * design: 키워드 정렬 수정 * design: 코드리뷰완료 버튼 색 수정 * design: 리뷰어/리뷰이 피드백 모달에 gap 조정 * feat: 코드 추천 항목은 비공개임을 명시 * refactor: HoverStyledLink를 이용해서 가이드페이지 제외 모든 컨텐츠 링크 스타일링 통일 * refactor: HoverStyledLinkForGuide를 이용해서 가이드페이지 내부 링크 스타일링 통일 * fix: MemoryRouter를 이용해 테스트 환경에 라우터 컨텍스트 제공 * feat: 놓친 EllipsisText 추가 * fix: Textarea의 글자수표시영역이 div에 같이 안 잡히는 에러 해결 * refactor: 피드백모달의 view모드일 때 분기를 Textarea 내부에서 하기 --------- Co-authored-by: 00kang <[email protected]> * [FE] 모달페이지 웹 접근성 추가(#618) (#641) * feat: 선언적으로 지정된 요소에만 Tab키 선택이 가능하도록 하는 FocusTrap 컴포넌트 추가 * fix: 버튼이 disabled 인 경우 tab으로 선택이 안되도록 수정 * feat: dropdown의 경우 open되면 해당 옵션만 tab으로 선택 가능하도록 추가 * feat: Modal 에 FocusTrap 추가 * style: OptionSelect 컴포넌트가 focus 시 outline이 보이지 않는 문제 수정 * fix: 룸카드 컴포넌트를 버튼으로 만들어 tab으로 접근 및 보이스오버에서 한 번에 읽을 수 있도록 변경 * fix: 헤더의 요소를 클릭 가능하도록 변경 및 시멘틱 태그 추가 * chore: 피드백 프로세스 버튼을 tab으로 선택이 가능하도록 태그를 변경 및 애니메이션 추가 * feat: 피드백 모아보기 페이지에 tab키로 사용 가능하도록 추가 * fix: 타입명과 컴포넌트명이 일치한 문제 해결 * fix: a 링크에 noreferrer 추가 * style: RoomCard의 요소를 button -> div 로 롤백 * feat: 프로필 드롭다운시 해당 요소만 포커싱이 되도록 추가 * feat: 피드백 모달에서 탭키로 사용이 가능하도록 추가 * refactor: 이전요소 포커싱 함수 리팩토링 * chore: 헤더에 불필요한 주석 제거 --------- Co-authored-by: Lee sang Yeop <[email protected]> Co-authored-by: 최진실 <[email protected]> * [BE] 리뷰 서비스 리팩토링(#615) (#636) * refactor: 리뷰 클라이언트 패키지 이동 및 리팩토링 * refactor: 리뷰 서비스 리팩토링 * fix: 충돌 해결 * refactor: 피드백 반영 --------- Co-authored-by: youngsu5582 <[email protected]> * [BE] 수정 시, 현재 인원 초기화 + 방 삭제 / 수정 시 오픈 상태일때만 수정하게 변경(#637) (#639) * refactor: 리뷰 클라이언트 패키지 이동 및 리팩토링 * refactor: 리뷰 서비스 리팩토링 * fix: 충돌 해결 * chore: ExceptionType 명확하게 변경 * fix: 잘못된 매개변수 변경 * refactor: reader,writer 통해 기능 분리 * refactor:피드백 반영 * test: 테스트 예제 추가 * fix: 방 상태를 그대로 가지게 변경 --------- Co-authored-by: youngsu5582 <[email protected]> --------- Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: gyungchan Jo <[email protected]> Co-authored-by: youngsu5582 <[email protected]> Co-authored-by: ashsty <[email protected]> Co-authored-by: hjk0761 <[email protected]> Co-authored-by: jinsil <[email protected]> Co-authored-by: 00kang <[email protected]> Co-authored-by: HyunJoong Kim <[email protected]>

youngsu5582 added 6 commits October 19, 2024 21:55

refactor: 리뷰 클라이언트 패키지 이동 및 리팩토링

27f2933

refactor: 리뷰 서비스 리팩토링

e340649

fix: 충돌 해결

c9993d2

chore: ExceptionType 명확하게 변경

7243020

fix: 잘못된 매개변수 변경

9de6817

refactor: reader,writer 통해 기능 분리

30ed363

github-actions bot added BE 백엔드 개발 관련 작업 수정 오류 수정 관련 작업 labels Oct 19, 2024

github-actions bot requested review from ashsty, hjk0761 and jcoding-play October 19, 2024 14:06

github-actions bot assigned youngsu5582 Oct 19, 2024

jcoding-play approved these changes Oct 21, 2024

View reviewed changes

hjk0761 approved these changes Oct 21, 2024

View reviewed changes

ashsty approved these changes Oct 21, 2024

View reviewed changes

youngsu5582 added 4 commits October 21, 2024 21:54

Merge branch 'develop' into feat/#637

81cdd29

refactor:피드백 반영

87a7ff0

test: 테스트 예제 추가

06976c7

fix: 방 상태를 그대로 가지게 변경

99d72f7

youngsu5582 merged commit 49d8942 into develop Oct 21, 2024
4 checks passed

youngsu5582 deleted the feat/#637 branch October 21, 2024 13:31

youngsu5582 mentioned this pull request Oct 21, 2024

코레아 1.21v 배포 #649

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[BE] 수정 시, 현재 인원 초기화 + 방 삭제 / 수정 시 오픈 상태일때만 수정하게 변경(#637) #639

[BE] 수정 시, 현재 인원 초기화 + 방 삭제 / 수정 시 오픈 상태일때만 수정하게 변경(#637) #639

github-actions bot commented Oct 19, 2024 •

edited by youngsu5582

Loading

github-actions bot commented Oct 19, 2024 •

edited

Loading

jcoding-play left a comment

jcoding-play Oct 21, 2024

hjk0761 left a comment

ashsty left a comment •

edited

Loading

ashsty Oct 21, 2024

ashsty Oct 21, 2024

ashsty Oct 21, 2024

ashsty Oct 21, 2024

[BE] 수정 시, 현재 인원 초기화 + 방 삭제 / 수정 시 오픈 상태일때만 수정하게 변경(#637) #639

[BE] 수정 시, 현재 인원 초기화 + 방 삭제 / 수정 시 오픈 상태일때만 수정하게 변경(#637) #639

Conversation

github-actions bot commented Oct 19, 2024 • edited by youngsu5582 Loading

📌 관련 이슈

✨ PR 세부 내용

github-actions bot commented Oct 19, 2024 • edited Loading

Test Results

jcoding-play left a comment

Choose a reason for hiding this comment

jcoding-play Oct 21, 2024

Choose a reason for hiding this comment

hjk0761 left a comment

Choose a reason for hiding this comment

ashsty left a comment • edited Loading

Choose a reason for hiding this comment

ashsty Oct 21, 2024

Choose a reason for hiding this comment

ashsty Oct 21, 2024

Choose a reason for hiding this comment

ashsty Oct 21, 2024

Choose a reason for hiding this comment

ashsty Oct 21, 2024

Choose a reason for hiding this comment

github-actions bot commented Oct 19, 2024 •

edited by youngsu5582

Loading

github-actions bot commented Oct 19, 2024 •

edited

Loading

ashsty left a comment •

edited

Loading