Part3-Team15 / taskify

[중급 프로젝트 15팀] taskify: 스마트한 일정 관리
https://taskify-15.vercel.app/
1 stars 3 forks source link

✨ Feat: 대시보드 초대 알림 기능 추가 #178

Closed un0211 closed 2 months ago

un0211 commented 2 months ago

연관된 이슈

작업 내용

3초마다 초대 알림을 불러오고 토스트로 알림 보냅니다. ("초대자"님이 "대시보드"에 초대했어요) 알림 지속 기간도 3초입니다! 전에 썼던 거라 react-toastify 사용했어요! (https://fkhadra.github.io/react-toastify/introduction)

스크린샷

https://github.com/Part3-Team15/taskify/assets/24778465/64fdae16-7aca-4aaa-a7c2-9d075fef2ec1

코멘트 및 논의 사항

기능 잘 동작하는지 확인 부탁드려요~

wjsdncl commented 2 months ago

image image

파일 하나 안올리신거 같아요! 🤔

un0211 commented 2 months ago

DefaultHeadr.tsx -> DefaultHeader/index.tsx 로 바꿨는데 실행 안되나요...?ㅠㅠ

un0211 commented 2 months ago
image

이렇게 되어있어요! 이게 파일 옮기거나 이름 바꿀 때 잘 안되더라구요.. 혹시 계속 그런가요??

wjsdncl commented 2 months ago

react-toastify 설치하니까 괜찮아졌습니다 🫡👍

wjsdncl commented 2 months ago

image 오 완전 잘됩니다 고생하셨습니다 👍

un0211 commented 2 months ago

3초마다 보내는 게 접속해 있는 사람 늘어날수록 서버 부하가 커질 것 같긴 한데, 개선 방법 있으시면 공유 부탁드려요! 변경된 내용 있는지 확인하고 있으면 쏴줘도 좋을 것 같은데, 서버까지 건드리지 않는 이상 방법이 있을까요..? 시간을 5초~10초 늘리는 건 어떨까요?? (어차피 해결은 안되지만 그나마 더 오래 버티게?)

wjsdncl commented 2 months ago

저는 알림은 3초~5초 사이가 적당하고 10초까지가 마지노선 일거 같네요 결국 초대 내역을 확인하려면 서버랑 통신을 해야해서 n초 간격으로 확인하는거 말고는 잘 모르겠네요 🤔

un0211 commented 2 months ago

5초로 변경할까 하는데 괜찮으면 👍 눌러주세요!! 아니면 👎

un0211 commented 2 months ago

넣는 게 오히려 안좋은 선택일까 고민중인데, 시연에는 좋을 거 같고 성능에는 안 좋을 것 같아서.. 나중에 이거 없는 버전으로 돌리고 이럴 일은 없겠죠...?

wjsdncl commented 2 months ago

나중에 문제 생기면 _app에 있는 <InvitationNotice />이거만 주석처리하면 되지않을까요?

wayandway commented 2 months ago

넣는 게 오히려 안좋은 선택일까 고민중인데, 시연에는 좋을 거 같고 성능에는 안 좋을 것 같아서.. 나중에 이거 없는 버전으로 돌리고 이럴 일은 없겠죠...?

알림 기능은 저희 팀만의 기능이니까 넣으면 좋을 것 같아요..!!! 성능은 차차 고민해보고, 멘토님께 조언을 구해봐도 좋겠네요!

un0211 commented 2 months ago

나중에 문제 생기면 _app에 있는 이거만 주석처리하면 되지않을까요?

그런 간단한 방법이!!