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] 온보딩 화면 구현 #36

Merged
merged 35 commits into from
May 17, 2024
Merged

[FEAT] 온보딩 화면 구현 #36

merged 35 commits into from
May 17, 2024

Conversation

AAminha
Copy link
Member

@AAminha AAminha commented May 16, 2024

1️⃣ 요약

  • 회원가입 이후 보여지는 온보딩 화면을 구현하고 API 연동을 진행했습니다.
  • useFunnel이라는 커스텀 훅을 적용했습니다.
  • Dropdown 컴포넌트와 Input 컴포넌트를 제작했습니다.
  • 유저 상태(비로그인, 로그인 중, 로그인 완료)에 따라 접근할 수 있는 페이지를 제한하는 로직을 구현했습니다.
    • MemberPrivateRoute.tsx
    • MemberPrivateRoute.tsx
  • resolved [FEAT] 온보딩 화면 구현 #26

2️⃣ 변경사항

기존 코드에 영향을 미치는 변경사항

  • 토큰 체크하는 로직에서 잘못된 부분이 있어서 수정했습니다.
  • 레지스터 토큰은 새로고침으로 날라가면 재발급할 방법이 없어서 세션 스토리지에 저장하는 방식으로 수정했습니다.
  • 컴포넌트 이름 변경이나 파일 위치 이동된 사항이 있습니다.
  • Layout이 반응형이냐 아니냐에 따라 나누는게 이상한 것 같아 삭제하고 하나로 통일했습니다.

기존 코드에 영향을 미치지 않는 변경사항

버그 수정을 위한 변경사항

  • Dropdown 컴포넌트 변경으로 인해 홈페이지 일부 섹션에서 관련 코드를 주석처리했습니다. 이 부분은 다음 이슈에서 처리하도록 하겠습니다.

3️⃣ 참고사항

이슈에 없는 수정사항

  • 테스트 로직에서 비로그인 사용자는 noAuthClient, 로그인 사용자는 authClient를 사용하도록 수정했습니다.
  • 온보딩과 테스트 화면에서 중복으로 쓰이는 KeywordChip 컴포넌트를 제작이 완료되어 테스트 화면에도 적용했습니다.

4️⃣ 추후 작업

5️⃣ 리뷰 요구사항

AAminha added 30 commits May 13, 2024 11:59
- primary2 variant 디자인 추가
- disabled 디자인 추가
- PlainButtonProps 타입 수정
- 케이스 추가
- 파일 위치 변경
- check 아이콘 추가
- DropdownContent의 maxHeight 속성 추가
- multiple이 아닐 경우 드롭다운이 닫히도록 수정
- title이 없을 경우 드롭다운이 안열리는 문제 해결
- SetupBasicInfo 컴포넌트와 중복되는 스타일 코드 분리
- useFunnel 커스텀 훅 구현
- 삭제 아이콘 제거
- selected의 hover 디자인 추가
- selected가 string인 경우 추가
- 커스텀 Scrollbar 적용
- 레지스터 토큰 관리 방식 추가
- 로직 오류 수정
- 레지스터 토큰 저장
- 로그인 후 이동 path 수정
- LoginPage는 비로그인 사용자만 접근하도록 수정
- OnboradingPage는 로그인 중인 사용자만 접근하도록 수정
- 아이콘 여백으로 스크롤 생기는 문제 해결
- 배경 이미지 오류로 이미지 변경
@AAminha AAminha added the ✨Feature 기능 개발 이슈 label May 16, 2024
@AAminha AAminha self-assigned this May 16, 2024
@AAminha AAminha requested a review from mungjin01 as a code owner May 16, 2024 16:53
@AAminha AAminha merged commit e7f6d71 into develop May 17, 2024
@AAminha AAminha deleted the feat/#26/onboarding branch May 17, 2024 02:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨Feature 기능 개발 이슈
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEAT] 온보딩 화면 구현
2 participants