refactor: 식물 사전 검색 결과 리팩터링, 데모데이 피드백 반영, 버그 수정 #130
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
- 기존: usdDictDetail의 suspense true로 인해 첫 접속 시 오류 - 수정: suspense 추가
쿼리스트링은 실험해봤는데 안되는거같아요
hozzijeong
reviewed
Jul 26, 2023
| <Button> | ||
| {nextCallback && <ArrowRight width={20} height={20} onClick={nextCallback} />} | ||
| </Button> | ||
| {nextCallback && ( |
| @@ -180,6 +180,11 @@ const PetRegisterForm = () => { | |||
| /> | |||
| </FormInputBox> | |||
| </Stack.Element> | |||
| <Stack.Element height={stackElementHeight}> | |||
There was a problem hiding this comment.
R
등록하기 버튼이 하나만 있으면 더욱 좋을 것 같습니다!
There was a problem hiding this comment.
제가 개인적으로 밑에 버튼을 둔 가장 큰 이유는 폼 미완일 때 미완임을 명시하기 위함 인데요. 이걸 해결하기 위해 진행도를 표시하는 UI를 추가하고 맨 아래 버튼을 없앴습니다
| @@ -23,7 +23,11 @@ const router = createBrowserRouter([ | |||
| }, | |||
| { | |||
| path: URL_PATH.petRegisterForm, | |||
| element: <PetRegisterForm />, | |||
| element: ( | |||
| <Suspense fallback={<div>로딩중입니다.</div>}> | |||
| <Stack.Element height={stackElementHeight}> | ||
| <Button type="submit" onClick={submit} disabled={!isValidForm}> | ||
| 등록하기 | ||
| </Button> |
There was a problem hiding this comment.
C
width: 100%;
height: 48px;
font-size: 2rem;
line-height: 2.4rem;
font-weight: 900;
letter-spacing: 1px;
color: #FFFFFF;
background: #1BCC66;
border-radius: 8px;와 같이 버튼 디자인을 바꿔보는 것은 어떤가요?
There was a problem hiding this comment.
input이랑 크기 맞추기 위해 width만 90%로 하고 나머지는 똑같이 갔어요 👍
|
|
hozzijeong
approved these changes
Jul 26, 2023
|
develop 브랜치와의 충돌이 아예 없기 때문에 pull은 의미 없다고 생각합니다. |
hozzijeong
pushed a commit
that referenced
this pull request
Jul 31, 2023
* refactor: 폼 입력 완료 시 최상단에도 버튼 등장 * refactor: 버튼 접근성 개선 * fix: 식물 등록 불가 오류 해결 - 기존: usdDictDetail의 suspense true로 인해 첫 접속 시 오류 - 수정: suspense 추가 * refactor: 검색 결과 페이지에서 검색어 표시 * refactor: generatePath 적용 쿼리스트링은 실험해봤는데 안되는거같아요 * feat: 진행률 표시 바 컴포넌트 * feat: form에 진행률 표시 * design: 버튼 스타일 변경 * chore: develop 브랜치와 github workflow 동기화
Choi-JJunho
pushed a commit
that referenced
this pull request
Aug 1, 2023
* refactor: 폼 입력 완료 시 최상단에도 버튼 등장 * refactor: 버튼 접근성 개선 * fix: 식물 등록 불가 오류 해결 - 기존: usdDictDetail의 suspense true로 인해 첫 접속 시 오류 - 수정: suspense 추가 * refactor: 검색 결과 페이지에서 검색어 표시 * refactor: generatePath 적용 쿼리스트링은 실험해봤는데 안되는거같아요 * feat: 진행률 표시 바 컴포넌트 * feat: form에 진행률 표시 * design: 버튼 스타일 변경 * chore: develop 브랜치와 github workflow 동기화
yeonkkk
added a commit
that referenced
this pull request
Aug 2, 2023
* feat: 단건 히스토리 조회 페이징 기능 개발 * refactor: isLastPage 메소드 서비스 계층으로 이동 * refactor: 피드백 내용 반영 * refactor: 피드백 내용 반영 * feat: 물주기 날짜 기준으로 내림차순 정렬 * build: RestAssured 라이브러리 추가 * test: 인수테스트 설정 클래스 추가 * test: 사전 식물 단건 조회 테스트 추가 * test: 사전 식물 이름으로 조회 테스트 추가 * refactor: 인수테스트에 트랜잭션 추가 * refactor: 인수테스트 Transactional 제거 * feat: 반려 식물 정보 수정 (#139) * refactor: 반려 식물 등록 request 네이밍 변경 * feat: MethodArgumentResolveException 핸들링 메서드 오버로딩 * feat: 반려 식물 정보 수정 컨트롤러 메서드 추가 * feat: 반려 식물 수정 도메인 로직 추가 * feat: 반려 식물 수정 서비스 로직 추가 * refactor: NotEmpty -> NotBlank 검증 어노테이션 변경 * refactor: 반려 식물 날짜 검증 메시지 수정 * refactor: 테스트 코드 중복 코드 제거 * refactor: null 검증 방법 변경 * refactor: MethodArgumentNotValid 예외 메시지 체이닝 방법 변경 * refactor: 사용자 인증방식 추가 (#146) * refactor: 사용자 인증방식에 어노테이션 추가 * fix: JPA Auditing 미적용 이슈 해결 - JPA Auditing이 적용되지 않아 테스트 수행 시 데이터가 Null 검증에 걸렸음 - Auditing 설정을 추가하여 해당 문제를 해결 * refactor: pet-plants 요청 사용자 파라미터 추가 * test: 헤더 인증방식 적용에 따라 Mocking 방식 수정 * refactor: DataJpaTest RepositoryTest로 상속 * refactor: 식물 사전 검색 결과 리팩터링, 데모데이 피드백 반영, 버그 수정 (#130) * refactor: 폼 입력 완료 시 최상단에도 버튼 등장 * refactor: 버튼 접근성 개선 * fix: 식물 등록 불가 오류 해결 - 기존: usdDictDetail의 suspense true로 인해 첫 접속 시 오류 - 수정: suspense 추가 * refactor: 검색 결과 페이지에서 검색어 표시 * refactor: generatePath 적용 쿼리스트링은 실험해봤는데 안되는거같아요 * feat: 진행률 표시 바 컴포넌트 * feat: form에 진행률 표시 * design: 버튼 스타일 변경 * chore: develop 브랜치와 github workflow 동기화 * feat: DB 초기화 구문 모듈화 * refactor: 인수테스트 사용자 초기정의 설정 * refactor: dDay -> dday로 변경 * feat: 반려 식물 생성 시 검증 추가 * test: 반려 식물 등록 및 조회 인수테스트 추가 * fix: API 서버 https 적용에 따른 BASE_URL 수정 (#153) * fix: BASE_URL 수정 - https 추가 - env에 따른 base url 변환 * refactor: BASE_URL 고정 및 msw 모킹 범위 확대 * refactor: 불필요한 주석, import 삭제 * feat: 네비게이션 바 구현 (#137) * feat: Navbar 아이콘 추가 * feat: Navbar 구현 * feat: 페이지 최소 높이 설정 및 Navbar 적용 * style: 아이콘 절대경로로 변경 * refactor: props 대신 useLocation 사용 * refactor: Navbar 변경내용 적용 * refactor: nav 태그로 변경 * design: Navbar 아이콘 크기 축소 * design: 높이 100%로 고정 * design: Navbar CSS fixed로 설정 * refactor: 페이지에서 main 태그 사용 / Navbar는 main과 분리 * refactor: RootTemplate으로 네이밍 변경 및 main태그 div로 변경 * chore: RootTemplate 적용 * chore: aria-hidden 적용 * feat: 내 반려 식물 상세 조회 페이지 (#150) * feat: 로딩 스피너 컴포넌트 - 색 설정 가능 - 전체적인 크기 설정 가능 - 스피너 원 테두리 두께 설정 가능 * refactor: Suspense fallback 스피너로 교체 * refactor: 불필요한 빈 스타일드 컴포넌트 제거 * refactor: Image common으로 이동 및 wide 타입 추가 * refactor: 이미지들 Image 컴포넌트로 통합 * fix: style 절대경로 아닌데 절대경로인 오류 수정 * feat: 반려 식물 조회 비동기 로직 * feat: 반려 식물 정보 타입 * refactor: API 명세 변경에 따른 타입 수정 * feat: 반려 식물 조회 msw 설정 * feat: 반려 식물 상세 마크업만 * refactor: devServer 전용 BASE_URL * refactor: 생일 확인을 위한 mock data 수정 * refactor: 스토리북용 msw 수정 받은 petPlantId를 mock data로 변환하는 수식 수정 * feat: 아이콘 추가 * design: 반려 식물 상세 컴포넌트 * refactor: APi 요청 헤더에 auth 추가 * feat: 반려 식물 상세 페이지 * refactor: 생일 계산에서 연도는 제외 * chore: develop 브랜치와 github action 통일 * refactor: api 서버 https 적용으로 인한 수정 * design: 식물 환경 정보 UI 변경 * refactor: 반려 식물 환경 정보 SVG 접근성 향상 * refactor: 반려 식물 상세 쿼리 refetch 제한 사용자가 직접 수정하지 않는 이상 자주 변하는 값이 아니므로 굳이 refetch on window focus 하지 않음 * design: 불필요한 min-width 제거 * feat: 두 날짜 사이의 간격을 구하는 함수 * design: 물주기 남은 날짜를 표시하도록 변경 * fix: typo * refactor: 불필요한 import 제거 * style: trailing comma * refactor: 테스트 support에서 사용하던 fixture 제거 (#162) * refactor: 테스트 support에서 사용하던 fixture 제거 * refactor: 도메인 생성 메서드 각 도메인 Fixture로 변경 * feat: 리마인더 기능 구현 (#148) * feat: 리마인더 물주기, 미루기 도메인 로직 구현 * feat: 서비스에 리마인더 물주기, 미루기, 조회 기능 구현 * feat: 컨트롤러 리마인더 물주기, 미루기, 조회 api 구현 * refactor: 테스트 방법 변경 * style: 개행 추가 * feat: 리마인더 물주기, 미루기 사용자 검증 추가 * refactor: 확정된 리마인더 기능에 맞게 검증 수정 * refactor: 리마인더 응답 값에 lastWaterDate 추가 * refactor: 확정된 리마인더 기능에 맞게 메서드명 수정 * feat: 반려식물 물주기 및 날짜 변경 검증 로직 수정 * feat: 엔티티 equals&hashCode 추가 * fix: dday 계산 테스트에서 Fixture 제거 후 새로운 객체 생성 * refactor: 메서드명 수정 * style: 코드 라인 정리 * refactor: 메서드명 수정 * refactor: 반려 식물 요청 정보 Fixture로 분리 * test: 반려 식물 전체 조회 인수테스트 추가 * test: 반려 식물 수정 인수테스트 추가 * refactor: 반려 식물 Fixture 함수명 변경 * test: 리마인더 인수테스트 추가 * style: 컨벤션 정렬 * test: 내림차순 정렬 확인 테스트 추가 * refactor: 회원 검증 실패 시 404로 변경 * refactor: 중복 메소드 변경 * style: 컨벤션 정렬 반영 * test: 파라미터 타입 명시 * refactor: 사전 식물 jsonPath로 검증하도록 수정 * refactor: 반려 식물 jsonPath로 검증하도록 수정 * refactor: 리마인더 jsonPath로 검증하도록 수정 * refactor: 히스토리 페이지 인덱스 1 변경 및 @ArgumentResolver 통한 객체 바인딩 * test: 반려 식물 히스토리 통합테스트 추가 * style: 컨벤션 적용 및 mockmvc param 메소드 적용 * style: 컨벤션 적용 및 mockmvc param 메소드 적용 * fix: 반려 식물 인수테스트 누락정보 수정 * test: 반려 식물 생성 테스트케이스 추가 * style: 사전 식물 인수테스트 공백 조절 * refactor: 불필요한 throws 구문 제거 * style: 히스토리 인수테스트 공백 조절 * refactor: 오타수정 * refactor: 변수명 변경 * style: static import 적용 * refactor: 테스트명 변경 * refactor: 검증 수정 * refactor: Fixture 중복 생성 함수로 분리 * refactor: 테스트 통일성 적용 * refactor: 예외메시지 구체화 * refactor: 숫자값 변수추출 --------- Co-authored-by: rawfishthelgh <[email protected]> Co-authored-by: gray <[email protected]> Co-authored-by: Q Kim <[email protected]> Co-authored-by: 유강현 <[email protected]> Co-authored-by: Seongyeon Kim <[email protected]>
Kim0914
pushed a commit
that referenced
this pull request
Aug 17, 2023
* refactor: 폼 입력 완료 시 최상단에도 버튼 등장 * refactor: 버튼 접근성 개선 * fix: 식물 등록 불가 오류 해결 - 기존: usdDictDetail의 suspense true로 인해 첫 접속 시 오류 - 수정: suspense 추가 * refactor: 검색 결과 페이지에서 검색어 표시 * refactor: generatePath 적용 쿼리스트링은 실험해봤는데 안되는거같아요 * feat: 진행률 표시 바 컴포넌트 * feat: form에 진행률 표시 * design: 버튼 스타일 변경 * chore: develop 브랜치와 github workflow 동기화
Kim0914
added a commit
that referenced
this pull request
Aug 17, 2023
* feat: 단건 히스토리 조회 페이징 기능 개발 * refactor: isLastPage 메소드 서비스 계층으로 이동 * refactor: 피드백 내용 반영 * refactor: 피드백 내용 반영 * feat: 물주기 날짜 기준으로 내림차순 정렬 * build: RestAssured 라이브러리 추가 * test: 인수테스트 설정 클래스 추가 * test: 사전 식물 단건 조회 테스트 추가 * test: 사전 식물 이름으로 조회 테스트 추가 * refactor: 인수테스트에 트랜잭션 추가 * refactor: 인수테스트 Transactional 제거 * feat: 반려 식물 정보 수정 (#139) * refactor: 반려 식물 등록 request 네이밍 변경 * feat: MethodArgumentResolveException 핸들링 메서드 오버로딩 * feat: 반려 식물 정보 수정 컨트롤러 메서드 추가 * feat: 반려 식물 수정 도메인 로직 추가 * feat: 반려 식물 수정 서비스 로직 추가 * refactor: NotEmpty -> NotBlank 검증 어노테이션 변경 * refactor: 반려 식물 날짜 검증 메시지 수정 * refactor: 테스트 코드 중복 코드 제거 * refactor: null 검증 방법 변경 * refactor: MethodArgumentNotValid 예외 메시지 체이닝 방법 변경 * refactor: 사용자 인증방식 추가 (#146) * refactor: 사용자 인증방식에 어노테이션 추가 * fix: JPA Auditing 미적용 이슈 해결 - JPA Auditing이 적용되지 않아 테스트 수행 시 데이터가 Null 검증에 걸렸음 - Auditing 설정을 추가하여 해당 문제를 해결 * refactor: pet-plants 요청 사용자 파라미터 추가 * test: 헤더 인증방식 적용에 따라 Mocking 방식 수정 * refactor: DataJpaTest RepositoryTest로 상속 * refactor: 식물 사전 검색 결과 리팩터링, 데모데이 피드백 반영, 버그 수정 (#130) * refactor: 폼 입력 완료 시 최상단에도 버튼 등장 * refactor: 버튼 접근성 개선 * fix: 식물 등록 불가 오류 해결 - 기존: usdDictDetail의 suspense true로 인해 첫 접속 시 오류 - 수정: suspense 추가 * refactor: 검색 결과 페이지에서 검색어 표시 * refactor: generatePath 적용 쿼리스트링은 실험해봤는데 안되는거같아요 * feat: 진행률 표시 바 컴포넌트 * feat: form에 진행률 표시 * design: 버튼 스타일 변경 * chore: develop 브랜치와 github workflow 동기화 * feat: DB 초기화 구문 모듈화 * refactor: 인수테스트 사용자 초기정의 설정 * refactor: dDay -> dday로 변경 * feat: 반려 식물 생성 시 검증 추가 * test: 반려 식물 등록 및 조회 인수테스트 추가 * fix: API 서버 https 적용에 따른 BASE_URL 수정 (#153) * fix: BASE_URL 수정 - https 추가 - env에 따른 base url 변환 * refactor: BASE_URL 고정 및 msw 모킹 범위 확대 * refactor: 불필요한 주석, import 삭제 * feat: 네비게이션 바 구현 (#137) * feat: Navbar 아이콘 추가 * feat: Navbar 구현 * feat: 페이지 최소 높이 설정 및 Navbar 적용 * style: 아이콘 절대경로로 변경 * refactor: props 대신 useLocation 사용 * refactor: Navbar 변경내용 적용 * refactor: nav 태그로 변경 * design: Navbar 아이콘 크기 축소 * design: 높이 100%로 고정 * design: Navbar CSS fixed로 설정 * refactor: 페이지에서 main 태그 사용 / Navbar는 main과 분리 * refactor: RootTemplate으로 네이밍 변경 및 main태그 div로 변경 * chore: RootTemplate 적용 * chore: aria-hidden 적용 * feat: 내 반려 식물 상세 조회 페이지 (#150) * feat: 로딩 스피너 컴포넌트 - 색 설정 가능 - 전체적인 크기 설정 가능 - 스피너 원 테두리 두께 설정 가능 * refactor: Suspense fallback 스피너로 교체 * refactor: 불필요한 빈 스타일드 컴포넌트 제거 * refactor: Image common으로 이동 및 wide 타입 추가 * refactor: 이미지들 Image 컴포넌트로 통합 * fix: style 절대경로 아닌데 절대경로인 오류 수정 * feat: 반려 식물 조회 비동기 로직 * feat: 반려 식물 정보 타입 * refactor: API 명세 변경에 따른 타입 수정 * feat: 반려 식물 조회 msw 설정 * feat: 반려 식물 상세 마크업만 * refactor: devServer 전용 BASE_URL * refactor: 생일 확인을 위한 mock data 수정 * refactor: 스토리북용 msw 수정 받은 petPlantId를 mock data로 변환하는 수식 수정 * feat: 아이콘 추가 * design: 반려 식물 상세 컴포넌트 * refactor: APi 요청 헤더에 auth 추가 * feat: 반려 식물 상세 페이지 * refactor: 생일 계산에서 연도는 제외 * chore: develop 브랜치와 github action 통일 * refactor: api 서버 https 적용으로 인한 수정 * design: 식물 환경 정보 UI 변경 * refactor: 반려 식물 환경 정보 SVG 접근성 향상 * refactor: 반려 식물 상세 쿼리 refetch 제한 사용자가 직접 수정하지 않는 이상 자주 변하는 값이 아니므로 굳이 refetch on window focus 하지 않음 * design: 불필요한 min-width 제거 * feat: 두 날짜 사이의 간격을 구하는 함수 * design: 물주기 남은 날짜를 표시하도록 변경 * fix: typo * refactor: 불필요한 import 제거 * style: trailing comma * refactor: 테스트 support에서 사용하던 fixture 제거 (#162) * refactor: 테스트 support에서 사용하던 fixture 제거 * refactor: 도메인 생성 메서드 각 도메인 Fixture로 변경 * feat: 리마인더 기능 구현 (#148) * feat: 리마인더 물주기, 미루기 도메인 로직 구현 * feat: 서비스에 리마인더 물주기, 미루기, 조회 기능 구현 * feat: 컨트롤러 리마인더 물주기, 미루기, 조회 api 구현 * refactor: 테스트 방법 변경 * style: 개행 추가 * feat: 리마인더 물주기, 미루기 사용자 검증 추가 * refactor: 확정된 리마인더 기능에 맞게 검증 수정 * refactor: 리마인더 응답 값에 lastWaterDate 추가 * refactor: 확정된 리마인더 기능에 맞게 메서드명 수정 * feat: 반려식물 물주기 및 날짜 변경 검증 로직 수정 * feat: 엔티티 equals&hashCode 추가 * fix: dday 계산 테스트에서 Fixture 제거 후 새로운 객체 생성 * refactor: 메서드명 수정 * style: 코드 라인 정리 * refactor: 메서드명 수정 * refactor: 반려 식물 요청 정보 Fixture로 분리 * test: 반려 식물 전체 조회 인수테스트 추가 * test: 반려 식물 수정 인수테스트 추가 * refactor: 반려 식물 Fixture 함수명 변경 * test: 리마인더 인수테스트 추가 * style: 컨벤션 정렬 * test: 내림차순 정렬 확인 테스트 추가 * refactor: 회원 검증 실패 시 404로 변경 * refactor: 중복 메소드 변경 * style: 컨벤션 정렬 반영 * test: 파라미터 타입 명시 * refactor: 사전 식물 jsonPath로 검증하도록 수정 * refactor: 반려 식물 jsonPath로 검증하도록 수정 * refactor: 리마인더 jsonPath로 검증하도록 수정 * refactor: 히스토리 페이지 인덱스 1 변경 및 @ArgumentResolver 통한 객체 바인딩 * test: 반려 식물 히스토리 통합테스트 추가 * style: 컨벤션 적용 및 mockmvc param 메소드 적용 * style: 컨벤션 적용 및 mockmvc param 메소드 적용 * fix: 반려 식물 인수테스트 누락정보 수정 * test: 반려 식물 생성 테스트케이스 추가 * style: 사전 식물 인수테스트 공백 조절 * refactor: 불필요한 throws 구문 제거 * style: 히스토리 인수테스트 공백 조절 * refactor: 오타수정 * refactor: 변수명 변경 * style: static import 적용 * refactor: 테스트명 변경 * refactor: 검증 수정 * refactor: Fixture 중복 생성 함수로 분리 * refactor: 테스트 통일성 적용 * refactor: 예외메시지 구체화 * refactor: 숫자값 변수추출 --------- Co-authored-by: rawfishthelgh <[email protected]> Co-authored-by: gray <[email protected]> Co-authored-by: Q Kim <[email protected]> Co-authored-by: 유강현 <[email protected]> Co-authored-by: Seongyeon Kim <[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.
🔥 연관 이슈
🚀 작업 내용
데모데이 피드백 관련
폼 입력 다 하면 Stack을 이용해서 맨 위에도 완료 버튼을 보여줍니다.
맨 밑에 버튼을 없애지 않은 이유는 1. 폼 미완일 때 미완임을 명시하기 위함 2. 사용자가 재확인을 위해 아래로 이동했을 때도 바로 등록 가능하게 하기 위함입니다.
검색 결과 페이지에서 검색어를 표시합니다.
코드 리뷰 관련
버그 수정 (중요도 높음)
제가 시간 내서 Cypress 전격 도입하겠습니다
💬 리뷰 중점사항