[All] release: 리뷰미 v1.1.0 배포 #723
Merged
Conversation
This file contains 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
* test: fixture 추가 * test: fixture 인자에 position 추가
* fix: 테스트 오류 수정 - 목록,상세 api 핸들링 훅 테스트 실패 수정 : reviewRequestCode가 들어가면서 이게 반영이 안돼서 실패해서 이를 추가함 - 비밀번호 조회 테스트 실패 수정 : 비밀번호 조회는 api response 타입이 변경된게 반영안돼서 실패함, 이를 수정 * fix: jest에서 import 못 읽는 오류 수정 - babel-jest 설치 및 jest.config.js 설정 변경 - jest.poly.fills import를 require롤 변경 (ES Modules -> COMMON JS Modules) * fix : jest에서 svg관련 SyntaxError 오류 수정 * refactor: useGetReviewList 확장자변경 (tsx- > ts)
* test: static 적용 * refactor: 개행
* feat: 버튼 연속 클릭 시, 트랜지션 코드가 실행되지 않게 하기 위해 clicked 상태 추가 * feat: 버튼 클릭 가능 시점에 0.01초 딜레이 추가 * fix: 자동 슬라이드 기능에 clicked 상태 의존성 추가해서 버튼 클릭 시 정상 작동하도록 수정
* refactor: 비밀번호 검증 훅에 onBlur 여부를 false로 초기화하는 함수 추가 * refactor: URLGeneratorForm의 input값들을 atom으로 관리 및 Input 관련 필드들을 별도의 컴포넌트로 분리 * chore: 불필요한 import 삭제 * refactor: input 상태값을 보다 구체적으로 수정 * refactor: urlGeneratorForm에서 사용하는 타입을 별도의 type 파일로 분리 * chore: 타입 import문 수정 * refactor: 기존 컴포넌트 분리를 기반으로 전역 상태 제거 리팩토링 * chore: 사용하지 않는 전역 상태 관리 파일 제거 * chore: 사용하지 않는 key 삭제 * chore: 주석 제거 * fix: label이 잘못 들어간 문제 수정
* refactor: 타입명 변경 (ReviewWritingFrom -> ReviewWirtingFormData) * docs: 타입 설명을 위한 jsDoc 추가 * refactor: 선택 해제하려는 강점 카테고리에 대한 상태,함수 네이밍 변경 - unCheckTargetOption -> unCheckTargetCategoryOption - unCheckTargetOptionId -> unCheckTargetCategoryOptionId) * refactor: 훅 네이밍 변경 (useCancelAnswerdCategory -> useCheckTailQuestionAnswer) * refactor: 타입명 변경 ( MakeNewSelectedOptionList -> MakeNewSelectedOptionListParams) ) * refactor: useUpdateMultipleChoiceAnswer에서 useOptionSelection 분리 - 객관식에서 선택된 문항에 대한 관리 책임은 useOptionSelection에 부여 * refactor: useMultipleChoice에서 useUnCheckCategoryOption 훅 분리 - 선택 해제하려는 강점 카테고리에 대한 관리 책임을 useUnCheckCategoryOption에 부여 * refactor:useMultipleChoice의 handleCheckboxChange 리팩토링 - 선택,해제의 경우에 따른 액션을 함수로 나눔 - 객관식에서 사용하는 훅 리팩토링에 따른 결과 적용 - updateOptionChoice 생성 * refactor: 객관식 훅 파일의 위치 변경 (hook -> MultipleChoiceQuestion) * style : usePasswordValidation export 방식 컨벤션에 맞게 변경 * refactor: useModals, useModalClose, usePasswordvalidation export 경로 리팩토링 * refactor: 컴포넌트 네이밍 수정 (MultipleChoiceQuesion -> MultipleChoiceAnswer) * refactor: 서술형 컴포넌트, 훅 리팩토링 - LongReviewItem -> TextAnswer - useLongReviewItem 삭제 , useTextAnswer에서 useLongReviewItem에서 담당하던 change, blur에 따른 오류 메세지 관리하는 역할 맡음 - TextAnswer, useTextAnswer 의 위치를 ReviewWirtingCardFormPage 로 이동 * refactor: 리뷰 작성 페이지 컴포넌트,파일명 수정 (ReviewWritingPage로 변경) * refactor: useTextAnswer 코드 위치 변경 및 주석 내용 변경 * refactor: Progressbar 상태 리팩토링 - 리팩토링 이유: 작성 폼의 상태들(currentCardIndex, cardSectionList등)과 밀접한 관계가 있지만, 작성 폼 상태 관리하는 훅들 속에 프로그래스바 상태 변경 함수들이 같이 있어서 관리에 좋지 않다고 판단 - 리팩토링 방향: 프로그래스바 상태들은 프로그래스바안에서 관리하자 - 방법: visitedCardListAtom 삭제, useStepList 훅 생성 * refactor: useStepList에서 상태명 변경 (visitedCardList -> visitedCardIdList) * refactor: 코드 설명 주석 추가 * feat: Carousel 컴포넌트 생성 * refactor: CardForm에서 캐러셀 관련 기능은 CardSlider로 분리 - 훅 네임 변경 : useCheckNextStepAvailability -> useMocinfStepAvailability - 관련 훅(useMocinfStepAvailability, useSlideWidthAndHeight)들 CardSlider hooks 폴더로 이동 및 캐러셀 공통 컴포넌트 사용 - ReviewWritingCard에 있는 CardSliderController를 CardSlider로 이동 * refactor: CardForm에서 모달 코드 및 컴포넌트들 CardModalContainer로 이동 * refactor: 리뷰 작성 페이지 폴더 구조 변경 - 작성 페이지의 기능 폴더를 두고 그 하위에 컴포넌트, 훅 폴더를 두는 방식으로 변경함 - 변경 이유: 페이지 안에 많은 기능이 있기 때문에, 기능별로 묶는것이 유지 보수에 좋을 거라고 판단 * chore: 필요 없어진 useReivewForm 삭제 * refactor: 서버에 답변을 제출하는 기능을 useSubmitAnswer 훅으로 묶음 * refactor: CardForm의 모달을 관리하는 useCardFormModal 훅 생성 및 useNavigateBlocker props 변경 * fix: useCardForm에서 isOpen 반환하도록 수정 * refactor: useSubmitAnswer 설명 주석 추가 및 export 경로 정리 * refactor: useLoadAndPrepareReview 훅 분리 - CardForm에서 useLoadAndPrepareReview 훅 분리 * feat: 리뷰 질문지 받아오는 query staleTime 5분으로 설정 * refactor: 훅, 함수 네이밍 변경 - useSubmitAnswer-> useSubmitAnswers, submitAnswer-> submitAnswers * refactor: 리뷰작성페이지 목데이터 리팩토링 - 필요없어진 reviewWritingData 삭제 - 폴더명 변경 : writingCardForm -> reviewWriting - 파일명 변경 : reviewWritingCardFormData -> reviewQuestionData - 상수명 변경: REVIEW_WRITING_FORM_CARD_DATA- > REVIEW_QUESTION_DATA * fix: Sidebar에 상수 변경에 따른 오류 수정 * fix: 테스트 오류 수정 - 목록,상세 api 핸들링 훅 테스트 실패 수정 : reviewRequestCode가 들어가면서 이게 반영이 안돼서 실패해서 이를 추가함 - 비밀번호 조회 테스트 실패 수정 : 비밀번호 조회는 api response 타입이 변경된게 반영안돼서 실패함, 이를 수정 * fix: jest에서 import 못 읽는 오류 수정 - babel-jest 설치 및 jest.config.js 설정 변경 - jest.poly.fills import를 require롤 변경 (ES Modules -> COMMON JS Modules) * fix : jest에서 svg관련 SyntaxError 오류 수정 * refactor: useGetReviewList 확장자변경 (tsx- > ts) * ci : jest에서 emotion theme을 읽을 수 있도록 관련 설정 및 패키지 추가 * ci: jest-dom 관련 셋팅 jest.setup에 추가 * test: 객관식의 최대 선택 개수 관련 테스트 추가 * fix: Button 컴포넌트 props 수정 - button의 다른 속성들이 적용될 수 있도록 수정 * refactor: CardSlider props에서 cardSectionList 삭제 * refactor: useUpdateMultipleChoiceAnswer, useMovingStepAvailablity 리팩토링 * test: CardSlider에서 객관식 질문의 유형과 답변에 따른 다음 버튼 활성화 테스트 추가 - 테스트 추가 및 테스트를 위한 data-testid 추가 * test: CardSlider 테스트에 질문 순서별 버튼 유형 테스트 추가 * refactor: REVIEW_QUESTION_DATA 에서 피드백과 추가 리뷰 상수로 분리 * refactor: useTextAnswer에서 TEXT_ANSWER_LENGTH 내보냄 * test: CardSlider 테스트 파일에 객관식 관련 테스트 추가 * refactor:REVIEW_QUESTION_DATA에서 강점에 대한 꼬리 질문 상수로 분리 * test: CardSlider 테스트 케이스에 강점 꼬리 질문 추가되는 테스트 추가 * refactor: 훅 네임 변경 (useCombinedAnswerState -> useCombinedReviewWritingState) * refactor: CardSlider 테스트 코드에서 recoil render 변수명 수정 * test: 객관식 유형,필수 여부별 선택된 문항에 따른 답변 유효성 검사 테스트 추가 * feat: CardSlider 테스트 코드에서 CardSlider 랜더링 시, 필요한 recoil 상태도 같이 렌더링 되게 수정 * refactor: 불필요한 주석 삭제 * chore: 객관식 테스트 문항 수정 * test: 서술형 테스트 추가 * test: cardSectionList에 대한 테스트 추가 * refactor: CardSlider 테스트에서 불필요해진 테스트 코드 삭제 및 useCombinedReviewWritingState 훅 삭제 - 객관식,서술형, 강점 선택에 따른 질문지 테스트를 분리함에 따라 CardSlider 테스트에서 해당 테스트 코드 삭제 * refactor: useStepList의 props에서 cardSectionList 제거 * test: 프로그레스 바에 대한 테스트 추가 * fix: usePasswordValidation 경로 오류 수정
* feat: 선택인 주관식에서 최소 글자 수 적용하지 않도록 변경 * test: 선택인 주관식에서 최소 글자 수 적용하지 않도록 변경한 것을 테스트에 반영 * refactor: useTextAnswer의 validateTextLength 리팩토링 - 질문의 필수 여부에 따른 다른 유효성 조건이 코드에서 직관적으로 보이게 리팩토링 * feat: 선택인 객관식 질문에서 최소 문항 개수 조건 적용하지 않는 것으로 변경 * test: 선택인 객관식 질문에서 최소 문항 개수 조건 적용하지 않는 기능 변경으로 인한 테스트 변경 * fix: textarea에 공백만 있는 입력이 주어진 경우 에러 메세지를 띄우고 유효하지 않은 입력으로 설정 및 상수화 * feat: 선택 질문인 경우에 대한 개수 안내 문구 변경 * design: 선택 문구에 margin-left 설정 * chore: QnABox에 불필요한 주석 삭제 * fix: change시 공백 오류 메세지 나오는 것 수정 * chore: 오타 수정 --------- Co-authored-by: ImxYJL <[email protected]>
* refactor: TEXT_ANSWER_LENGTH 상수화 * refactor: 서술형 필수 직문에 최소/최대 글자 수 표시 및 textarea의 placeholder 제거 * test: 리팩토링에 따른 테스트 import 수정 * refactor: 주관식 선택 문항에 대한 가이드라인 추가 * refactor: 질문별 안내 문구를 설정하는 if문 개선
* refactor: 요소 드래그 방지를 위한 prevent-drag 클래스 추가 * refactor: 클릭 요소 드래그 방지를 위한 prevent-drag 클래스 적용 * chore: 불필요한 클래스 제거 * feat: 드래그를 막아주는 PreventDrag 컴포넌트 구현 * refactor: PreventDrag에 children으로 클릭 요소 넣기 * design: Breadcrumb display 속성 추가 * refactor: UndraggableWrapper로 컴포넌트명 변경
* feat: 상세 리뷰 데이터 요청 useQuery의 staleTime 지정 * feat: 리뷰 목록 데이터 요청 useQuery의 staleTime 지정 * feat: 리뷰 그룹 데이터 요청 useQuery의 staleTime 변경
* fix: webpack-dev-server 삭제, webpack에 내장된 development 사용 * ci: 불필요한 start 명령어 삭제 - webpack의 mode를 설정하지 않아서 기본값이 development가 들어가는데 이 부분이 기존에 있는 dev와 일치해서 역할이 겹침 - dev라는 명령어가 development 모드 실행이라는 의미를 명시적으로 보여줘서 start를 삭제함 * ci: dist 폴더를 실행하는 명령어 변경 (serve-> serve-dist) * ci: 빌드 하지 않고 production 모드를 여는 명령어 추가 * reactor: system 상수 삭제 및 이를 NODE_ENV 환경 변수로 대체 - webpack mode에 따른 sentry. msw 실행 여부 판단을, NODE_ENV 환경 변수 사용하는 것으로 변경 * fix: webpack-dev-server 패키지 복구 - 명령어만 webpack-dev-server가 필요 없어졌고, 패키지는 여전히 필요함
* fix: 이전 버튼이 눌리지 않는 현상 수정 * chore: bool형 표현 방식 수정 * refactor: 이전 버튼 컴포넌트가 받는 불필요한 props 제거
* test: 선택형 응답 검증 테스트에 픽스처 적용 * test: 픽스처에 @component 추가 * test: 리뷰 작성 테스트에 픽스쳐 적용 * test: 불필요한 import문 제거 * chore: 픽스처가 테스트 환경에서만 빈 등록되게 TestConfig 수정 * test: 픽스처 주입 방식 변경 * fix: 생성자 오류 수정 * refactor: static 메서드 적용 * test: 필요없는 데이터 삽입 제거 * test: 가독성을 위한 변수 추출 * test: 픽스처로 대체할 수 있는 부분 추가 반영
* test: ReviewServiceTest 코드 리팩터링 * test: ReviewDetailLookupServiceTest 코드 리팩터링 * test: 유효한 테스트 코드가 되도록 수정 * refactor: 테스트 코드 이름 변경
* feat: 리뷰를 검증하는 서비스 구현 * refactor: 리뷰 등록 서비스가 리뷰를 생성하고 검증 서비스를 통해서 검증하도록 변경 - 서비스명 변경 : CreateReviewService → ReviewRegisterService * refactor: 답변 검증 서비스가 요청이 아닌 객체를 검증하도록 변경 * feat: 리뷰에서 제공하는 기능 추가 * refactor: dto명 변경 및 사용하지 않는 메서드 삭제 * refactor: 예외명 및 메세지 변경 * refactor: 리뷰 목록 조회 서비스명 변경 및 응답 객체 mapper를 활용하도록 변경 * feat: 리뷰 목록 응답 객체 매핑 서비스 추가 * refactor: dto명 변경으로 인한 수정 * refactor: ReviewMapper를 통해 리뷰를 생성하도록 변경 * refactor: 리뷰 상세 응답 mapper를 활용하도록 변경 및 일부 로직 리팩터링 * refactor: 템플릿을 찾는 로직을 mapper에서 하도록 변경 * refactor: 리팩터링에 따른 컨트롤러 변경사항 * test: 변경사항 반영 * refactor: 질문이 없는 섹션은 응답 생성하지 않도록 변경 * refactor: 답변이 null인 경우에 대한 예외 추가 * refactor: CheckBoxAnswerValidator null 검증 제외 및 질문 유효 검증 추가 * refactor: TextAnswerValidator 질문 유효 검증 추가 * test: Validator 테스트 작성 * test: Mapper 테스트 작성 * test: ReviewValidator 테스트 작성 * test: ReviewRegisterService 테스트 간소화 * test: 리팩터링 사항 반영 * test: Review 테스트 작성 * chore: secret 변경사항 반영 * refactor: 답변 생성 시, null 이 들어올 수 없도록 변경 * refactor: ReviewMapper 에서 답변에 대한 검증 없이 엔티티 매핑만 하도록 변경 * refactor: ReviewValidator 에서 답변에 대한 검증도 진행하도록 변경 * refactor: 메서드명 변경 (getAllQuestionIdsFromAnswers -> getAnsweredQuestionIds) * refactor: @transactional 어노테이션을 클래스 범위에만 적용하도록 변경 * refactor: 메서드 분리 변경 * refactor: 메서드 시그니쳐 변경 * style : 개행 수정 * refactor: 존재하는 메서드 활용하도록 변경 * refactor: 리뷰에 특적 질문에 대한 답변이 있는지 확인하는 기능 추가 * style: 개행 수정 * refactor: 객체 메서드를 활용하도록 변경 * test: AnswerMapperTest 작성 * refactor: 답변 검증 시, 답변이 빈 경우에 대한 검증도 객체 생성자에서 진행 및 QuestionNotAnsweredException을 domain 하위의 exception 패키지로 변경 * refactor: 메서드명 변경 (hasQuestions -> hasAnsweredQuestion) * style: 개행 변경 * refactor: ReviewValidator 파라미터를 나눠보 보낼 수 있는 부분 적용 * test: develop 브랜치 머치 후, 미변경 사항 적용
* test: QuestionRepositoryTest 패키지를 question 하위로 변경 * test: OptionItemRepositoryTest 픽스처 적용 * test: QuestionRepositoryTest 픽스처 적용 * test: ReviewGroupServiceTest 픽스처 적용 * test: SectionRepositoryTest 픽스처 적용 * test: SectionTest 픽스처 적용 * test: TemplateMapperTest 픽스처 적용 * test: TemplateServiceTest 픽스처 적용 * test: ReviewTest 사용하지 않는 필드 제거 * test: 필요없는 항목 제거 및 필드 접근제어자 적용
* fix: 홈 페이지 캐러셀 frame drop 최소화 작업 - scrollToSlide에서 slide 애니메이션 작업하는 함수 분리 - window,requestAnimationFrame 적용 - will-change 적용 Co-Authored-By: Fe <[email protected]> * feat: Carousel 공통 컴포넌트에 cardIndex에 따라 translate 변경하는 기능 추가 Co-Authored-By: Fe <[email protected]> * chore: 불필요한 코드 삭제 Co-Authored-By: Fe <[email protected]> * feat: ReviewWritingCard에 memo 적용 - 답변이 바뀌지 않은 카드의 경우 재렌더링 되지 않도록 함 Co-Authored-By: Fe <[email protected]> * refactor: useSlideWidthAndHeight -> useSlideHeight 변경 Co-Authored-By: Fe <[email protected]> * fix: useStepList 테스트 오류 수정 Co-Authored-By: Fe <[email protected]> * refactor : Home의 Carousel 이름 변경 - Carousel ->InfinityCarousel Co-Authored-By: Fe <[email protected]> * chore: 불필요한 주석 삭제 --------- Co-authored-by: Fe <[email protected]>
* ci: svgo 및 webpack-bundle-analyzer 설치 Co-authored-by: sooyeon <[email protected]> Co-authored-by: Fe <[email protected]> Co-authored-by: badahertz52 <[email protected]> * chore: svgo로 압축한 svg 파일 업데이트 * ci: svgo를 devDependency로 설치 * ci: svgo를 devDependency로 설치 2 * chore: svg 원본 파일 복원 * ci: build시 svgo로 svg 최적화를 수행할 수 있도록 script 수정 * ci: BundleAnalyzerPlugin을 dev환경에서만 실행하도록 수정 * ci: prebuild 명령어 분리 --------- Co-authored-by: sooyeon <[email protected]> Co-authored-by: Fe <[email protected]> Co-authored-by: badahertz52 <[email protected]>
* refactor: suspense와 lazy를 이용한 js 파일 분리 Co-Authored-By: ImxYJL <[email protected]> * chore: 코드 한줄로 변경 Co-Authored-By: ImxYJL <[email protected]> --------- Co-authored-by: ImxYJL <[email protected]>
* fix: BundleAnalyzerPlugin 실행 환경 오류 수정 - dev 모드가 아닌 prod 모드에서 실행하도록 수정 * feat: 구글 폰트 preload 적용 * feat: pretendard virable font로 변경 및 preload 적용 * fix : pretendard css 불러오는 link 수정 - 변경 사유 :rel이 preload인 link, stylesheet인 link가 두 개 같이 존재해서 헷갈림 - 변경 방법 : onload="this.onload=null;this.rel='stylesheet'" 사용 * fix: useStepList 테스트 오류 수정 * chore: BundleAnalyzerPlugin 주석 처리
* refactor :validateInput 폴더 구조 수정 * refactor : usePostDataForReviewRequestCode 폴더 구조 수정 * chore: 불필요한 test 코드 삭제 * feat: js 압축을 위한 webpack 설정 추가
* chore: schema.sql 생성 * chore: flyway 의존성 추가 * chore: flyway를 위한 패키지 생성 * chore: 테스트 sql 파일 삭제 * chore: drop문 삭제 * chore: enum 타입을 varchar로 변경 * chore: 명시용 스키마의 자동 적용 방지를 위한 이름 변경
* refactor: 필수 질문이 아닌 경우 최소 글자수 제한을 받지 않도록 변경 * refactor: 필수 질문이 아닌 경우에 대한 검증 테스트 추가 * refactor: InvalidTextAnswerLengthException 에 min값 고정 생성자 추가 * refactor: InvalidTextAnswerLengthException 생성자 체이닝 적용
donghoony
assigned chysis, BadaHertz52, nayonsoso, soosoo22, skylar1220, ImxYJL and Kimprodp
Kimprodp
approved these changes
Sep 26, 2024
|
FE 실험 주욱 해보고 머지할게요 |
* fix: 섹션에 해당하는 질문만 응답하도록 수정 * build: 로컬 환경 flyway 비활성화
…해제 (#737) * refactor: 가로 스크롤을 숨기기 위해 AnswerListModalContent 추가 * refactor: ReviewList 인터페이스에 isLastPage 추가 * refactor: isLastPage가 true일 때, IntersectionObserver 해제 * refactor: 리뷰 요청 개수 size를 10으로 수정 * refactor: 목 데이터 응답에 isLastPage 값 추가 * chore: stylelint 적용
* refactor: 정렬 기준 추가 * test: 테스트명 수정
* feat: 마지막 페이지 여부 반환 구현 * test: 테스트 코드 작성 * test: 테스트 코드 작성 - 기존 테스트코드에 검증만 추가 * refactor: 쿼리 개선 * test: 변경된 쿼리 테스트 코드 작성 * refactor: 서비스 코드에 적용 * docs: API 문서 누락 부분 수정 * refactor: 쿼리 개선 * chore: 이전 API 문서를 제거한다 * refactor: 쿼리를 개선한다 * refactor: 쿼리를 개선한다 * refactor: 함수명 변경 * style: 들여쓰기 수정 --------- Co-authored-by: Donghoon Lee <[email protected]>
|
마피아는고개를들고 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
고생 많았습니다 🔥