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

[Fix] 댓글관련 수정사항들 #110

Merged
merged 3 commits into from
Nov 11, 2024
Merged

Conversation

Sonseongoh
Copy link
Collaborator

@Sonseongoh Sonseongoh commented Nov 10, 2024

🚀 풀 리퀘스트 제안

로그인했던 안했던 삭제버튼 눌렀을때 모달 뜨게하기
비로그인시 비밀번호 틀렸을 경우에 비밀번호 틀렸다는 토스트나 알림창 띄우게하기
댓글 삭제 성공,실패시 토스트창 알림
any타입 제거

📋 작업 내용

로그인했던 안했던 삭제버튼 눌렀을때 모달 뜨게하기
비로그인시 비밀번호 틀렸을 경우에 비밀번호 틀렸다는 토스트나 알림창 띄우게하기
댓글 삭제 성공,실패시 토스트창 알림
any타입 제거

🔧 변경 사항

로그인했던 안했던 삭제버튼 눌렀을때 모달 뜨게하기
비로그인시 비밀번호 틀렸을 경우에 비밀번호 틀렸다는 토스트나 알림창 띄우게하기
댓글 삭제 성공,실패시 토스트창 알림
any타입 제거

📸 스크린샷 (선택 사항)

image
image

📄 기타

추가적으로 전달하고 싶은 내용이나 특별한 요구 사항이 있으면 작성해 주세요.

@Sonseongoh Sonseongoh self-assigned this Nov 10, 2024
@Sonseongoh Sonseongoh linked an issue Nov 10, 2024 that may be closed by this pull request
3 tasks
Copy link
Member

@ssuminii ssuminii left a comment

Choose a reason for hiding this comment

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

오,, 굳굳티비 모달 버튼 왜저래 수정해놓겠습니다.

@ssuminii ssuminii merged commit fe8f910 into develop Nov 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Fix] 댓글 관련 수정할 사항들
2 participants