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

계좌 생성 화면 구성 #58

Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

계좌 생성 화면 구성 #58

wants to merge 10 commits into from

Conversation

kd02109
Copy link
Contributor

@kd02109 kd02109 commented Mar 20, 2024

What is this PR?

  • close: 계좌 개설 페이지 #53
  • 다이나믹 라우트를 활용해서 선택한 계좌 id를 path에서 받아오도록 설정을 했습니다. 이를 통해 서버 fetch를 최대한 활용하고자 합니다.
  • user-password는 user 비밀번호를 입력하는 페이지로 작성했습니다. 해당 페이지에서 비밀번호 일치후 경로 설정을 위해서 param으로 ?to ="다음 경로" 방식으로 비밀번호 일치 후 전환될 페이지 경로를 입력하도록 했습니다.
  • account-password는 계좌 비밀번호 설정 페이지입니다. 해당 페이지는 signup/password/_component/KeyPadLayout을 그대로 복사해서 일부 수정했습니다. 우선 31일 까지 작업 마무리가 우선이기에, 이러한 중복 사용에 대한 로직 고민은 1차 구현 이후에,,, ㅜㅜㅜ

What has changed?

  • account 폴더 내부에 화면 작업을 진행했습니다.
  • account/page.tsx -> 계좌 선택 페이지
  • account/[bank] -> 환 정보 확인 페이지
  • account/[bank]/foreign-exchange -> 환율 선택 페이지
  • account/[bank]/loading -> 로딩 페이지
  • account/[bank]/fail -> 실패 페이지
  • account/[bank]/done -> 계좌 개설 성공 페이지

A notice to reviewers...


Screen

Create.Turborepo.mp4

@kd02109 kd02109 added 💬 리뷰 요청 리뷰 부탁드립니다! 🔨 기능 새로운 기능에 관한 토픽 labels Mar 20, 2024
@kd02109 kd02109 self-assigned this Mar 20, 2024
Copy link

vercel bot commented Mar 20, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
swifty-frontend-swifty-bank ❌ Failed (Inspect) Mar 23, 2024 6:46am

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💬 리뷰 요청 리뷰 부탁드립니다! 🔨 기능 새로운 기능에 관한 토픽
Projects
None yet
Development

Successfully merging this pull request may close these issues.

계좌 개설 페이지
1 participant