Skip to content

Comments

Feat/113/use confirm#122

Merged
cksrlcks merged 6 commits intomainfrom
feat/113/useConfirm
Feb 13, 2025
Merged

Feat/113/use confirm#122
cksrlcks merged 6 commits intomainfrom
feat/113/useConfirm

Conversation

@cksrlcks
Copy link
Collaborator

❓이슈

✍️ Description

작업내용

  • useAlert 개선
  • useConfirm 추가
  • modalStore 개선
  • DialogContainer 개선

개선한방법
dialog가 열리면 promise를 생성하고, resolve와 reject를 state에 추가하여,
dialog 확인, 취소버튼에서 각각 resolve와 reject를 실행하도록 했습니다.

resolve, reject둘다 dialog가 닫히도록 state를 조작하며, 각각 true, false를 반환하여
confirm, alert을 사용하는 쪽에서 활용이 가능합니다.

변경된 훅내용으로 수정한 페이지들

  • 로그인, 회원가입 폼
  • 비밀번호 수정 컴포넌트
  • 대시보드 정보 수정 페이지
image

✅ Checklist

PR

  • Branch Convention 확인

    feat/* 기능 구현, fix/* 버그 수정, refactor/* 개선

  • Base Branch 확인
  • 적절한 Label 지정
  • Assignee 및 Reviewer 지정

Test

  • 로컬 작동 확인

Additional Notes

  • (없음)

@cksrlcks cksrlcks requested a review from ToKyun02 February 12, 2025 13:58
@cksrlcks cksrlcks self-assigned this Feb 12, 2025
@cksrlcks cksrlcks linked an issue Feb 12, 2025 that may be closed by this pull request
1 task
@vercel
Copy link

vercel bot commented Feb 12, 2025

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

Name Status Preview Comments Updated (UTC)
taskify ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 12, 2025 2:16pm

- buttons 디폴트값을 지정
Copy link
Owner

@ToKyun02 ToKyun02 left a comment

Choose a reason for hiding this comment

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

변경하신 내용확인했습니다!
작업하시느라 고생하셨습니다!

@cksrlcks cksrlcks merged commit ac796a6 into main Feb 13, 2025
3 checks passed
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.

Feat : useConfirm 만들기

2 participants