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

[Feature] - 검색 결과 페이지 내 필터링 옵션 적용 #565

Merged
merged 5 commits into from
Oct 23, 2024

Conversation

jinyoung234
Copy link
Contributor

✅ 작업 내용

  • 여행기 검색 결과 훅(useInfiniteSearchTravelogues)과 여행기 메인 페이지 훅(useInfiniteTravelogues)를 하나로 통합한 형태로 변경(백엔드 명세 변경으로 인한 수정)
  • 검색 결과 페이지 내 필터링 옵션 적용

📸 스크린샷

X

🙈 참고 사항

  • css 엉진망창으로 구현된거 같은데 한번 크로스 체킹해주시면 좋을거 같아요,,,,

@jinyoung234 jinyoung234 added the FE label Oct 22, 2024
@jinyoung234 jinyoung234 added this to the sprint 6 milestone Oct 22, 2024
@jinyoung234 jinyoung234 self-assigned this Oct 22, 2024
Copy link

github-actions bot commented Oct 22, 2024

Test Results

11 tests   11 ✅  14s ⏱️
 2 suites   0 💤
 1 files     0 ❌

Results for commit f3ceaec.

♻️ This comment has been updated with latest results.

Copy link
Contributor

@simorimi simorimi left a comment

Choose a reason for hiding this comment

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

지니 고생하셨어요 코멘트 하나 남겨봤는데 확인 부탁드릴게요!

singleSelectionAnimationKey,
resetSingleSelectionTags,
increaseSingleSelectionAnimationKey,
} = useSingleSelectionTag(STORAGE_KEYS_MAP.mainPageSort, STORAGE_KEYS_MAP.mainPageTravelPeriod);
Copy link
Contributor

Choose a reason for hiding this comment

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

storage key를 다르게 써야하지 않을까라는 생각이 듭니다! 왜냐하면 search 페이지에서 표시한 테그들이 메인에도 남아있으면 사용자 입장에서 당황스러울 거 같아서요 지니의 생각은 어떤가요?

Copy link
Contributor

@0jenn0 0jenn0 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 지니!!!

@jinyoung234 jinyoung234 merged commit 0b3b3ee into develop/fe Oct 23, 2024
3 checks passed
@simorimi simorimi deleted the feature/fe/#562 branch November 21, 2024 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants