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

[FE][Feat] #106 : add-channel 페이지 사용자 별 경로 설정 기능 구현 및 header 상수화 #237

Merged
merged 8 commits into from
Nov 21, 2024

Conversation

leedongyull
Copy link
Collaborator

📝 PR 개요

  • 바텀시트 최소 크기 변경
  • header 상수화
  • 사용자 Context로 관리
  • 사용자 별 루트 설정 기능 구현 (mock데이터)
  • 위도 및 경도 주소로 변환 (아직 전체주소)

✅ 체크리스트 (Checklist)

  • 코드가 빌드 오류 없이 잘 작동하는지 확인
  • 테스트가 통과하는지 확인
  • 스타일 가이드와 일관성을 유지했는지 확인
  • 관련 문서가 업데이트되었는지 확인 (선택 사항)
  • 리뷰어가 이해할 수 있도록 주석이나 설명을 추가했는지 확인

🔄 관련 이슈 (Linked Issues)

#106


📷 스크린샷 및 동영상 (선택 사항)

image

바텀 시트 최소 크기

Vite-React-TS-외-페이지-5개-개인-Microsoft_-Edge-2024-11-21-19-39-19

유저 추가 삭제 및 footer 상태 변환 테스트


📚 생겼던 이슈

Header 상수화 과정에서 동적 파라미터 받기

  • 처음에는 url을 파싱해서 변환해주는 함수를 만들고
export const HEADER_TITLE: { [key: string]: string | ((id: string) => string) } = {
  '/add-channel/:id/draw': id => ${id}에 따른 경로 설정,
};

이런식으로 구현해서 사용하려고 하였음. 하지만 title 만 로직이 복잡해지고 심지어 해시값으로 나와서 사용할 수 없었음
따라서 현재는 임시로 이런식으로 사용...

title={`${params.user || ''}${headerOption.title}`}

Footer 상수화

  • Footer의 경우 ACTIVE, PASSIVE 상태 등등 동적으로 변경이 자주 일어남
  • 따라서 url path 에 따른 상수화가 현재 불가능
  • 어떻게 해야할지..?

검색 기능 및 주소 매핑 (ex. 서울특별시 강남구 강남대로 지하396 -> 강남역) 은 추후 추가 예정

api만 연결하면 user쪽은 거의 끝난 것 같습니다

@github-actions github-actions bot added the 확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전) label Nov 21, 2024
effozen
effozen previously approved these changes Nov 21, 2024
Copy link
Collaborator

@effozen effozen left a comment

Choose a reason for hiding this comment

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

LGTM

@happyhyep happyhyep requested review from happyhyep and removed request for juwon5272 November 21, 2024 13:10
happyhyep
happyhyep previously approved these changes Nov 21, 2024
Copy link
Member

@happyhyep happyhyep left a comment

Choose a reason for hiding this comment

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

LGTM

@happyhyep happyhyep dismissed stale reviews from effozen and themself via 31fd1fe November 21, 2024 13:12
Copy link
Member

@happyhyep happyhyep left a comment

Choose a reason for hiding this comment

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

LGTM

@effozen effozen self-requested a review November 21, 2024 13:14
Copy link
Collaborator

@effozen effozen left a comment

Choose a reason for hiding this comment

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

LGTM

@effozen effozen merged commit aacc328 into frontend Nov 21, 2024
3 checks passed
@leedongyull leedongyull deleted the feature/fe/#106-add-channel branch December 5, 2024 07:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants