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

[FEAT] 쿼리 전달 및 뷰 전환 구현(#11) #19

Merged
merged 3 commits into from
Jul 23, 2023
Merged

Conversation

jh2ee
Copy link
Member

@jh2ee jh2ee commented Jul 22, 2023

관련 이슈

🔒 Closes

(#11)

작업내용

  • 작업 내용
  • useNavigation 이용해 query 전달
  • 검색 시 페이지 이동(쿼리 기반)
작업 전 작업 후
image
콘솔 통해 쿼리 전달된 것 확인 가능
주소창 변화 통해 페이지 이동 구현 확인 가능

추후 진행할 사항

태그, search 뷰와 연결

리뷰포인트

#11 (comment)
해당 이슈에 추가한 검색 로직 확인 후 PR부탁드립니다
Search 뷰는 검색바에서 넘긴 쿼리를 받아 뷰 보이면 될 것 같습니다.

태그와 연동 방향은 상의가 필요할 것 같은데 생각해본건 태그를 선택하면 검색창에 검색한 것과 같이 구현하는 방법입니다
SearchBar 코드를 약간 수정해 tag에서 prop을 받아오는 방법으로 처리 가능할 것 같습니다
function SearchBar({tag}){ const [query, setQuery] = useState(tag) ... );

Reference

Checklist

  • 필요없는 주석, 프린트문 제거했는지 확인
  • 컨벤션 지켰는지 확인

@jh2ee jh2ee self-assigned this Jul 22, 2023
Copy link
Contributor

@ju1e3718 ju1e3718 left a comment

Choose a reason for hiding this comment

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

p1; SearchBar.js에서 Icon 관련해서 오류 뜨는데 확인해주셔야 할 것 같습니다...!

@jh2ee
Copy link
Member Author

jh2ee commented Jul 23, 2023

p1; SearchBar.js에서 Icon 관련해서 오류 뜨는데 확인해주셔야 할 것 같습니다...!

해당 오류는 Image 폴더 경로 이동 과정에서 생긴 것 같습니다..! 제 브랜치에서는 이상 없어요,,,

Copy link
Member

@xlooslo xlooslo left a comment

Choose a reason for hiding this comment

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

내가 확인해도 이상 없는 것 같아요

@jh2ee jh2ee merged commit f6c2be2 into develop Jul 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants