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: search/debounce 등.. #79

Merged
merged 21 commits into from
Sep 24, 2023
Merged

✨Feat: search/debounce 등.. #79

merged 21 commits into from
Sep 24, 2023

Conversation

nayeon-hub
Copy link
Collaborator

🪄 변경사항

  1. 스타일 디테일하게 적용했으나 그럼에도 아쉬운 스타일 적용이 있으면 언제든지 말해주세요!
  2. 수연님이 만들어 주신 debounce 훅을 써서 input에 디바운스 적용했습니다.
  3. filter값은 상수화 했습니다.
  4. 에러 핸들링은 주말에 적용할 예정입니다.
  5. postPreview는 이미 컴포넌트내에서 navigation 처리가 되어있더라구요 follow 유저도 Follow 컴포넌트에서 처리해놓겠습니다!

🖥 결과 화면

짜잔!

image

✏️ PR 포인트

@nayeon-hub nayeon-hub changed the title Feat/search/debounce 등.. ✨Feat: search/debounce 등.. Sep 22, 2023
@nayeon-hub nayeon-hub self-assigned this Sep 22, 2023
Copy link
Collaborator

@suyeon1218 suyeon1218 left a comment

Choose a reason for hiding this comment

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

나연님도 정말 깔끔한 코드를 짜셨군요... 👍 클린 코드 정말 보기 좋았습니다!

const getUser = async (userId: string) => {
const response = await axios.get(`${API_BASE_URL}/users/${userId}`);
const getUser = async (userId: string | User) => {
const response = await axios.get<User>(`${API_BASE_URL}/users/${userId}`);
Copy link
Collaborator

Choose a reason for hiding this comment

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

p3

const url = `${API_BASE_URL}/users/${userId}`

로 따로 빼내어 사용하는 건 어떻게 생각하실까요?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

혜성님의 코드여서 혜성님을 소환하겠습니다! @hyesung99

Copy link
Collaborator

Choose a reason for hiding this comment

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

좋은 방법인것 같아요!

src/pages/layout/components/SearchHead.tsx Outdated Show resolved Hide resolved
src/pages/layout/components/SearchHead.tsx Show resolved Hide resolved
src/pages/layout/components/Search.tsx Outdated Show resolved Hide resolved
src/pages/layout/components/SearchMain.tsx Outdated Show resolved Hide resolved
src/pages/layout/components/Search.tsx Outdated Show resolved Hide resolved
src/pages/layout/components/SearchResultPost.tsx Outdated Show resolved Hide resolved
@nayeon-hub nayeon-hub linked an issue Sep 23, 2023 that may be closed by this pull request
src/pages/layout/types/typeGuard.ts Outdated Show resolved Hide resolved
src/pages/layout/components/SearchHead.tsx Outdated Show resolved Hide resolved
src/pages/layout/components/SearchHead.tsx Outdated Show resolved Hide resolved
@nayeon-hub nayeon-hub merged commit 332545e into main Sep 24, 2023
@nayeon-hub nayeon-hub deleted the feat/search/style branch September 24, 2023 12:30
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.

Search Page 중간 점검 이슈
3 participants