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.

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

[정종윤] 기능구현 챌린지 #5

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

Conversation

BellYun
Copy link
Member

@BellYun BellYun commented May 12, 2024

필수 구현 사항

  • 질환 명 검색시 API를 호출하여 드롭박스를 통해 추천 검색어를 보여주는 기능을 구현합니다.
    • 검색어가 없을 시 “검색어 없음” 표출
    • [ x] 최근에 검색어를 보여줍니다.
  • 검색어를 검색 시 결과를 리스트로 보여줍니다.
    • 검색어가 없을 시 화면도 구현
    • 검색 결과 컴포넌트를 클릭 시 https://clinicaltrialskorea.com/studies/{검색어ID} 링크로 이동
  • 입력마다 API 호출하지 않도록 API 호출 횟수를 줄이는 전략 수립 및 실행
    • 검색어 입력 시 500ms마다 요청이 되도록 간단한 디바운스 적용
    • useQuery에 enabled을 사용해 조건에 맞을 경우에만 실행하도록 구현
  • API를 호출할 때 마다 console.info("calling api") 출력을 통해 콘솔창에서 API 호출 횟수 확인이 가능하도록 설정

추가 구현 사항

  • 공통 컴포넌트 구현
    • 헤더 구현
    • 검색 리스트 구현
    • 드롭박스 구현
    • 모달 구현
  • 커스텀훅 사용을 통한 로직 분리

선택 구현 사항

  • [선택 사항 1] 키보드만으로 추천 검색어들로 이동 가능하도록 구현합니다.
    • ex) 키보드 방향키, 탭을 사용하여 다음 추천 검색어로 이동
  • [선택 사항 2] React-Query를 활용하여 캐싱을 구현합니다.
  • [선택 사항 3] 검색어 결과는 페이지네이션 또는 무한스크롤 선택합니다.
    • 외부 라이브러리 없이 구현할 경우 추가 점수
  • [선택 사항 4] 스크랩 저장 기능을 구현합니다.
    • 페이지를 새로고침 해도 리스트가 남아있도록 구현.
    • 즐겨찾기 페이지에서 스크랩한 결과물 리스트 보여주도록 구현.
    • 확인 모달을 통해 스크랩을 삭제.
  • [선택 사항 5] 뷰포트 크기에 따른 반응형 UI를 구현합니다.
  • [선택 사항 6] Storybook을 사용하여 UI 인터렉션 테스팅을 구현합니다.

과제를 수행하면서 진행하셨던 고민

구현하고자 했던 설계의 방향성

@BellYun BellYun self-assigned this May 12, 2024
Comment on lines +20 to +29
export const Header = () =>{
return(
<Container>
<HeaderMainBox>
<div>한국임상정보</div>
<div>즐겨찾기</div>
</HeaderMainBox>
</Container>
)
}

Choose a reason for hiding this comment

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

안녕하세요, 종윤님!

Header 컴포넌트로 분리해서 잘 작성해 주셨네요.
다만, 시맨틱 태그를 사용하면 더 좋을 것 같다는 생각이 듭니다. 예를 들어, header 컴포넌트의 최상위 요소는 <header> 태그를 사용하고, 텍스트 부분은 <p> 태그를 사용하면 시맨틱 웹 접근성 측면에서 더욱 적합할 것 같습니다.😀

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.

2 participants