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

[Feat/#481] useToastHandler 훅 추가, 토스트 전부 상단으로 위치 (+ alert 제거), 핑크색으로 글자 칠해지도록 기능 구현 #482

Merged
merged 6 commits into from
Jan 3, 2025

Conversation

ocahs9
Copy link
Contributor

@ocahs9 ocahs9 commented Dec 31, 2024

📌 관련 이슈번호

🎟️ PR 유형

어떤 변경 사항이 있나요?

  • 새 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 리팩토링

✅ Key Changes

이번 PR에서 작업한 내용을 간략히 설명해주세요

  1. 작업 내용

📢 To Reviewers

  • 추후 toast를 바텀으로 옮겨달라는 요청에 대비하여 toast의 bottom 속성은 살려두었습니다.
  • 이에 따라, 메세지가 변하거나 toast의 위치가 유동적으로 변하는 경우에는 useToastHandler 훅을 사용하시면 됩니다.
  • useToastHandler 에서는 toastConfig(메세지, 위치)와 toastVisible, handlerToastVisible 등을 반환합니다.
  • useToastHandler는 필수가 아닙니다. 다만 만약 사용할 경우, useToast는 사용할 필요가 없습니다.

이외에도 다른 qa 반영

  • 필요 없는 alert 제거
  • toast 전부 top으로 이동

📸 스크린샷

image

🔗 참고 자료

@ocahs9 ocahs9 self-assigned this Dec 31, 2024
@github-actions github-actions bot added the ✨ FEAT 기능 구현 label Dec 31, 2024
@ocahs9 ocahs9 requested a review from pepperdad December 31, 2024 11:12
Copy link

PR 작성하느라 고생 많았어요!! 라벨 잘 지정되었는지 확인 한 번 해 주기 🫶

@ocahs9 ocahs9 requested review from sinji2102, pepperdad and imddoy and removed request for pepperdad, sinji2102 and imddoy December 31, 2024 11:12
Copy link
Member

@pepperdad pepperdad left a comment

Choose a reason for hiding this comment

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

확인했습니다~ 머지해주세용!

Copy link
Contributor

@imddoy imddoy left a comment

Choose a reason for hiding this comment

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

머지해조

Copy link

sonarqubecloud bot commented Jan 3, 2025

@ocahs9 ocahs9 changed the title [Feat/#481] useToastHandler 훅 추가, 토스트 전부 상단으로 위치 (+ alert 제거) [Feat/#481] useToastHandler 훅 추가, 토스트 전부 상단으로 위치 (+ alert 제거), 핑크색으로 글자 칠해지도록 기능 구현 Jan 3, 2025
@ocahs9 ocahs9 merged commit 9b4da3d into develop Jan 3, 2025
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ FEAT 기능 구현
Projects
Status: 🎉Done
Development

Successfully merging this pull request may close these issues.

[ Bug ] 토스트 전부 top으로 올리기, 필요없는 alert 지우기
3 participants