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

refactor(fe): refactor settings page using context and hooks #2274

Merged
merged 8 commits into from
Jan 10, 2025

Conversation

jihorobert
Copy link
Contributor

@jihorobert jihorobert commented Jan 2, 2025

Description

  • Settings 페이지 이탈시 확인 모달 구현했습니다
  • useContext 이용
  • useCheckPassword 훅 이용

Additional context

closes TAS-1115


Before submitting the PR, please make sure you do the following

Copy link

settings 리팩토링

Copy link
Contributor

@Clover229 Clover229 left a comment

Choose a reason for hiding this comment

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

LGTM
수고하셨습니다 ㅎㅎ

@jihorobert jihorobert merged commit a0b215b into main Jan 10, 2025
9 checks passed
@jihorobert jihorobert deleted the t1115-settings-refactor branch January 10, 2025 11:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done ✔️
Development

Successfully merging this pull request may close these issues.

3 participants