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.

Sign up for GitHub

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

Jump to bottom

이메일 중복 오류 발생 시, displayMessage 보여주도록 수정 #386

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from
Open

이메일 중복 오류 발생 시, displayMessage 보여주도록 수정 #386

wants to merge 4 commits into from

Conversation

plgafhd
Copy link
Collaborator

@plgafhd plgafhd commented Jan 15, 2025

No description provided.

@plgafhd plgafhd requested a review from a team as a code owner January 15, 2025 11:54
@plgafhd
Copy link
Collaborator Author

plgafhd commented Jan 15, 2025 •
edited
Loading

새삼 우리 dialog가 좀 안이쁘긴 하네... ㅋㅋㅋㅋㅋ

@plgafhd
Copy link
Collaborator Author

plgafhd commented Feb 1, 2025

@eastshine2741 @JuTaK97
요거 서버는 이미 나가있기도 하고,
에러 메시지 통일이 아무래도 금방 할것 같지는 않으니 일단 요대로 머지하는거 괜찮아?

@plgafhd
Copy link
Collaborator Author

plgafhd commented Feb 7, 2025

안잘리게 줄이는 방향으로 잘 합의가 되어서...! 🤝🏻
그냥 displayMessage 토스트 띄우게 바꾸면 될 것 같아

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eastshine2741 eastshine2741 eastshine2741 left review comments

At least 1 approving review is required to merge this pull request.

Assignees

@plgafhd plgafhd

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@plgafhd @eastshine2741