Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[FE] 그간 작업 사항 머지 한 번~ #660

Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

[FE] 그간 작업 사항 머지 한 번~ #660

wants to merge 10 commits into from

Conversation

semnil5202
Copy link
Collaborator

@semnil5202 semnil5202 commented Jul 12, 2024 •
edited
Loading

작업 대상

와 작년 12월 29일부터 main 머지를 안했네요?? ㅋㅋㅋㅋㅋ 주말동안 리팩토링 가능하니까 한 번 머지 고 할까요?

머지하면 반영되는 것

  • 웹 접근성 보완
  • Swiper 반응형 버그 해결
  • 메타 태그 open graph 적용

semnil5202 and others added 10 commits December 29, 2023 19:41
@semnil5202
[FE] Refactor/#628 시맨틱 태그, 웹 표준, 웹 접근성 검토 (#629)
4e3a419 
* refactor: home 및 layout 시맨틱 태그 수정

* refactor: topicCard 및 Container 시맨틱 태그 수정

* chore: switch default  off eslint 설정

* refactor: 불필요한 profile img 제거

* refactor: banner section 시맨틱 태그 적용

* refactor: 즐겨찾기, 내토픽리스트 시맨틱 태그 적용

* refactor: topicInfo 시맨틱 태그 수정

* refactor: Map 시맨틱 태그 수정

* refactor: topicCard, Container li 및 article 시맨틱 태그 수정

* refactor: 페이지 컴포넌트 시맨틱 태그 일괄 수정

* refactor: Search 페이지 layout width 지정 및 nav highlight 수정

* refactor: PinDetail h2, h3 시맨틱 태그 적용

* refactor: Logo aria-label 수정

* refactor: Banner aria-label 수정

* refactor: TopicCard 및 Image 공통 컴포넌트 등 aria-label 수정

TopicCard 컨테이너 및 리스트도 수정한다. Image 공통 컴포넌트에 aria-hidden 속성을 추가하여 필요한 상황에만 사용할 수 있도록 한다.

* refactor: topicInfo 접근성 로직 리팩토링

* test: cypress 테스트 data-cy 태그 의존성으로 변경

* refactor: PinPreview 및 뽑아오기 접근성 로직 수정

aria-live 를 사용하는 로직을 useAriaLive 훅으로 분리하고 aria-label을 추가하여 사용성을 높인다.

* refactor: 개발서버 부재에 따른 api 위치 변경

* refactor: URL 주소 공유하기 버튼 접근성 로직 수정

* refactor: pinDetail 및 상세보기 버튼 접근성 로직 추가

* refactor: 댓글 및 답글 접근성 로직 추가

* refactor: 불필요한 console.log 제거

* refactor: 지도 추가하기 접근성 로직 수정

* refactor: 핀 생성 접근성 로직 수정

* refactor: 각 영역 전체보기 페이지 접근성 로직 수정

* refactor: 즐겨찾기 및 나의 지도 접근성 로직 수정

* refactor: 내 정보 컨테이너 접근성 로직 수정

* refactor: 지도, 핀, 닉네임 수정하기 접근성 로직 추가

* design: 크로스 브라우징 UI 일관성을 위해 체크 박스 스타일 조정
@semnil5202
chore: reactQuery, Axios 추가 (#641)
a8aa2de
@GC-Park
[FE] Refactor/#643: Profile 페이지 API 로직 수정 및 React-Query 적용 (#647)
1d5a01c 
* feat: http 모듈 추가

* feat: Profile get api 추가

* feat: useProfileList hook 추가

* refactor: fetchGet을 useProfileList로 변경

* refactor: getProfile api에 url 직접 입력으로 변경

* refactor: 사용하지 않는 매개 변수 삭제 및 queryFn 수정

* refactor: 필요없는 코드 삭제

* refactor: query refetch 추가

* refactor: tanstack query 기본 설정 추가

* refactor: profile리스트 가져오는 부분 query 추가 및 필요없는 코드 삭제

* refactor: useQuery를 useSuspenseQuery로 변경

* refactor: Profile에 Query suspense 적용
@semnil5202 @jiwonh423
[FE] Refactor/#642: 북마크 및 전체보기 페이지 API 로직 수정 및 React-Query 적용 (#644)
f7d8897 
* feat: Axios http 정의 및 get 메서드 구현

* refactor: Bookmarks 페이지 TopicCardList 분리 코드 합병

TopicCardList는 재사용을 위해 만들어진 코드이나 Axios, React Query 를 적용하면서 사용하기 애매해졌다.
이에 따라 TopicCardList를 제거하고 각 페이지에서 그 책임을 이어받는다.

* design: Skeleton 컴포넌트 스타일 변경 및 적용

* refactor: SkeletonBox 공통 컴포넌트 구현 및 convertCSS 유틸 함수 구현

* refactor: Skeleton 컴포넌트 Bookmark 페이지에 적용

* refactor: http api 수정 및 useGetBookmark isLoading 상태 사용

* refactor: SeeAllNearTopics 페이지 TopicCardList 분리 및 코드 합병

* refactor: API 로직 반환값 타입 지정

* refactor: Bookmark 스켈레톤 수정

* refactor: SeeAllLatestTopics 페이지 TopicCardList 분리 및 코드 합병

* refactor: SeeAllPopularTopics 페이지 TopicCardList 분리 및 코드 합병

* refactor: AllTopics Query key 수정 및 시맨틱 태그 수정

* rename: API 명세와 페이지 이름 통일화

* refactor: 전체보기 페이지 명칭 수정 router 적용

* feat: 리프레쉬 토큰 요청 기능 추가

운영서버에 머지해야 확인가능할 것 같습니다.
기존 로직을 사용하는 운영서버와 request 값은 동일한데 cookie와 도메인 설정 문제로 실패하는 것으로 확인됩니다.

* feat: query default option 설정

* 마운트 시 리페칭 해제
* 윈도우 포커스 시 리페칭 해제
* 일정 주기로 리페칭 해제
* 받아온 데이터 stale 시 리페칭 해제

* refactor: useSuspenseQuery 를 통한 선언적으로 로딩상태 처리

* fix: token 없을 때 Authorization 빈 객체로 세팅하여 비로그인 오류 해결

* refactor: withCredentials 옵션 잠시 보류

* refactor: 01.17 회의를 통한 변경

* TopicCardList 컴포넌트를 이전처럼 활용하도록 한다. 전체보기 및 즐겨찾기는 거의 동일한 형태이며 중복코드가 다량 발생하여 위와 같이 수정한다.
* url을 넘겨받음에 따라서 리액트 쿼리 훅, API 요청 로직 또한 하나의 훅으로 재사용한다.

---------

Co-authored-by: afds4567 <[email protected]>
@jiwonh423
[FE] refactor after merge #643, #642 (#649)
d77d381 
* Refactor : Remove unused code and update import path

* Refactor : Update default topic and profile images

* Refactor :  Profile component and update routingHandlers
@jiwonh423
[FE] Refactor/#645: Home 페이지 API 로직 수정 및 React-Query 적용 (#646)
f67141a 
* feat : Add HTTP client module

* feat : Add QueryClient and QueryClientProvider for React Query

* feat : Add API and hook for fetching topics

* refactor : Refactor authorization header in http.ts

* feat : Add useTopicsQuery hook to TopicCardContainer

* refactor : Refactor TopicCard and TopicCardContainer components

eliminate useState,useEffect in TopicCardContainer
@semnil5202
Refactor/#652: SEO 최적화 및 head 옵션 검토 (#653)
504f52e 
* fix: queryClient 중복 오류 제거

* refactor: 사용하지 않는 daum POI 검색 제거

* refactor: favicon s3 에서 local 로 변경

* refactor: script 태그 위치 조정 DOM 파싱 중단 여부에 따른 위치조정

* refactor: keyword, description 추가

* feat: open graph meta 태그 적용

* refactor: image 태그 alt 누락 일괄 적용
@semnil5202
Refactor/#650: 웹팩 최적화 검토 (#651)
7b1c515 
* chore: setting json option 값 수정

* fix: 중첩 queryClient 오류 제거

* chore: eslint 설정 rule off 추가

* chore: bundle analyzer 오픈

* refactor: gzip 적용

* refactor: analyzer 사용 종료
@semnil5202
Refactor/#655: 토픽 단일 조회 페이지 레이어 분리 (#656)
34c985b 
* rename: 페이지 단위 코로케이션 적용

* refactor: 불필요한 변수 제거

* refactor: topicDetail 요청 로직 Query로 분리

* refactor: 클러스터링 로직 훅으로 분리 및 재사용

추후 모아보기 페이지에서 재사용할 예정이다.

* refactor: 사용자 지도 상호작용 이벤트 핸들링 및 상호작용 후 동작 로직 커스텀 훅으로 분리

* chore: prettier printWidth 100으로 설정

훅의 인자, 반환값의 네이밍 길이가 길어짐에 따라 위와 같이 변경한다.

* refactor: 더블 사이드바 및 쿼리파라미터 세팅 로직 커스텀 훅으로 분리

추후 모아보기 페이지에서 사용할 수 있도록 한다.

* refactor: useTags init 로직 useTags 훅 내부로 이동

모든 페이지마다 tags를 초기화해줘야하는 로직이 불필요하게 반복된다. 또한 휴먼에러가 발생할 가능성이 높은 부분으로 판단되어 useTags의 props로 받아 초기화가 불필요한 페이지는 명시적으로 초기화 하지 않도록 한다.

* fix: tags 초기화 기능 조건 로직 반대로 지정한 오류 수정
@semnil5202
Refactor/#657: Swiper 0.9.4 버전 적용 (#658)
be6a23c 
* chore: Swiper 0.9.1 버전 적용

* refactor: Swiper 0.9.1 버전 반영

* chore: Swiper 0.9.4 버전 적용
@semnil5202 semnil5202 requested review from GC-Park and jiwonh423 July 12, 2024 16:30
jiwonh423
jiwonh423 approved these changes Jul 12, 2024
View reviewed changes
Copy link
Collaborator

@jiwonh423 jiwonh423 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jiwonh423 jiwonh423 jiwonh423 approved these changes

@GC-Park GC-Park Awaiting requested review from GC-Park

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@semnil5202 @jiwonh423 @GC-Park