Skip to content

Conversation

@BaekJiyeon02
Copy link
Member

#️⃣연관된 티켓 넘버

CLAP-422
CLAP-427

📝작업 내용

정지된 계정은 이메일 전송을 제한 하도록 수정하였습니다
로그인, 비밀번호에 대한 모달을 재설정하였습니다.
내 정보 수정 페이지에 사진 변경 후 수정 버튼 클릭시 생기는 로딩 동안 로딩 모달을 추가했습니다.
공통 에러에 대해서 setError 사항을 추가하였습니다.
상태변경시 종료 사유 미입력 하면 종료 사유 입력 모달이 다시 생기도록 하였습니다.

🏞️스크린샷 (선택)

💬리뷰 요구사항(선택)

@BaekJiyeon02 BaekJiyeon02 added 🐞 BugFix Something isn't working 🔨 Refactor 코드 리팩토링 labels Feb 14, 2025
@BaekJiyeon02 BaekJiyeon02 self-assigned this Feb 14, 2025
@BaekJiyeon02 BaekJiyeon02 linked an issue Feb 14, 2025 that may be closed by this pull request
1 task
Copy link
Contributor

@Minkyu0424 Minkyu0424 left a comment

Choose a reason for hiding this comment

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

확인했습니다!

@BaekJiyeon02 Modal이 여러 개인 경우 저희 팀은 현재 다음과 같은 방식을 사용중인 거 같아요

A모달의 상태, B모달 등 여러 모달의 열리고 닫히고 상태를 각각 관리하는 방식의 경우

const isAOpen = ref(false)
const isBopen = ref(false)

하나의 객체에 모달의 상태를 정의해 관리하는 방식
const isModalVisible = ref({ reject: false, fail: false, success: false })

하나의 모달 상태로 관리하되, 모달의 상태를 boolean이 아닌 string으로 관리

const isModalOpen = ref('')
isModalOpen.value='A'

다양한 방식으로 쓰이곤 하는데 프로젝트가 끝나가지만..
다음 프로젝트에 모달을 쓰게된다면 어떤 방식이 좋을지 생각해봐도 좋을 거 같아요!

저도 아직 고민이되어서 그때 그때 다른거 같아요 😂

@Minkyu0424 Minkyu0424 merged commit 73a20a6 into develop Feb 14, 2025
@Minkyu0424 Minkyu0424 deleted the CLAP-427 branch February 18, 2025 00:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐞 BugFix Something isn't working 🔨 Refactor 코드 리팩토링

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CLAP-427 QA 사항 반영 Chloe

3 participants