Skip to content

Conversation

@NamgungJongMin
Copy link
Member

작업 개요

1차 리팩토링 코드 병합 PR입니다.

작업 상세

  • 기존 mvp 개발 과정에서 작성 된 타입 및 api 엔드포인트 수정

    • 기존 api 반환값의 구조 및 형식에 대해 일관성, 가독성을 고려하여 백엔드 측에 수정 요청
      1. 수정된 api 반환값 기준으로 타입 수정
      2. 중복된 타입 관련 로직을 중앙 집중화 할 수 있는 api response type 추가 및 적용
      3. 수정된 api와 관련된 컴포넌트 및 테스트 파일 수정
    • rest 원칙에 부합하는 parameter를 적용한 엔드포인트로 백엔드 측에 수정 요청
      1. 리소스를 식별하는 필수적인 값은 path parameter로 변경
      2. 필터링, 정렬 등 선택적인 값은 query parameter로 변경
  • 웹접근성을 고려한 코드 수정

    • 시멘틱 태그를 고려하여 마크업 수정. list 관련 기존의 div / section 태그를 ul/ol 로 교체
    • 키보드를 이용해 무한스크롤을 탐색하는 유저가 중간에 리스트에서 빠져나올 수 있도록 SkipList 컴포넌트 적용

NamgungJongMin and others added 30 commits January 21, 2025 13:34
- FormTravelData를 FormData로 변환하는 createTravelFormData 유틸 함수로 분리
- createTravel API 호출 시 BaseResponse 타입을 적용하여 타입 안전성 강화
- ChatRoom, Chat, ChatOverview 등 타입 이름 변경 반영
- getChatRooms, leaveChat, getChat, getChatOverview, setIsJoined, uploadChatImages 함수에 BaseResponse 타입 적용
- 타입 안전성을 강화하고 일관된 API 응답 구조로 리팩토링
- participant에 user 반영
- 엔드포인트 확인 완료
- 반환타입 확인 후 적용 완료
- 엔드포인트 확인 완료
- 반환타입 확인 후 적용
- 타입 및 api 변경에 의한 사이드 이펙트 해결
NamgungJongMin and others added 24 commits February 5, 2025 09:27
- api 공통 타입 적용
- 백엔드 엔드포인트 변경 최신화
- 관련 컴포넌트 타입 수정
- 관련 테스트 타입 수정
- initialPage 0 부터 시작하도록 변경
- Store, type 등 filter 관련 설정에 size, pageParam 추가
- 테스트 오류 해결
- 리뷰 좋아요 버튼 비로그인 시  401 => 400 수정 적용
- 리스트 ul, il 사용
- 여행 후기 이미지 등록하기 aria-label에 설명 추가
- 무한스크롤 첫 페이지 1에서 0으로 변경
- any 타입 대신 unknown으로 변경
- 불필요한 타입 삭제
- page 1 시작에서 page 0 시작으로 변경
refactor: review 관련 타입 및 api 수정 및 컴포넌트 적용.
…l-list

refactor: 여행 api 수정사항 적용, 리스트 ul, li 사용, 그 외 타입 정리
refactor:  chat, 여행만들기 api 수정
refactor: 접근성 관련 시멘틱 태그 수정 및 리스트 스킵 컴포넌트 적용
- content가 api 필수 반환값으로 수정됨에 따라 조건부 렌더링 로직 삭제
refactor: 여행 리스트 탈출 버튼 적용, 이미지 onload 수정, 테스트 코드 에러사항 수정 반영
@NamgungJongMin NamgungJongMin self-assigned this Feb 6, 2025
@github-actions
Copy link

github-actions bot commented Feb 6, 2025

💄 Storybook: https://674fa5b608628f124a188752-vbfgwtxqlo.chromatic.com/
🕖 Update:

@NamgungJongMin NamgungJongMin merged commit e6906a0 into main Feb 6, 2025
5 checks passed
@NamgungJongMin NamgungJongMin deleted the dev branch February 6, 2025 17:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants