Skip to content

Conversation

@solprime
Copy link
Collaborator

이슈 번호

close #265

변경 사항 요약

  • 알림 기능 구현했습니다.

  • 다른 사람이 내 위키를 수정했을 때만 알림이 옵니다. (내 위키를 내가 수정하면 알림x)

  • 알림을 누르면 내 위키로 이동 후 알림은 삭제됩니다.

  • 꼼꼼하게 체크 부탁드립니다!

테스트 결과

비어있을 경우
image

알림 있을 때

image

모바일 환경
image

@solprime solprime added ✨ 신규 기능 신규 기능 작업 관련 태그 📁 컴포넌트 컴포넌트 개발 관련 작업 labels Dec 28, 2024
@solprime solprime self-assigned this Dec 28, 2024
Copy link
Contributor

@junghwaYang junghwaYang 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

@junghwaYang junghwaYang left a comment

Choose a reason for hiding this comment

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

Menu.tsx에 있는 const fadeIn = 'pc:animate-pcFadeIn tamo:animate-tamoFadeIn'; 해당 애니메이션 globals.css에

.downFadein{
  @apply pc:animate-pcFadeIn tamo:animate-tamoFadeIn
}

이렇게 추가하셔서 Menu에도 downFadein 추가하시고, 알람에서도 해당 인터럭션 추가되면 좋을것같아요~

@junghwaYang
Copy link
Contributor

@solprime 지금 계속 In HTML, <button> cannot be a descendant of <button>. 에러가 떠서 확인해보니까 NotificationContents.tsx 안에도 버튼태그가 들어가있어서 에러가 발생하는것같습니다.
개발자도구 확인하셔서 button안에 button이 들어가는 일이 없도록 수정해주셔야할것같습니다

Copy link
Collaborator

@haksoo0918 haksoo0918 left a comment

Choose a reason for hiding this comment

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

이미지 보니 이뿌게 잘 나온거 같아요. 😄
수고하셨습니다. 👍

@solprime solprime merged commit b91d5ae into develop Dec 30, 2024
1 check passed
@solprime solprime deleted the feature/#265_추가기능-알림-기능 branch December 30, 2024 07:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ 신규 기능 신규 기능 작업 관련 태그 📁 컴포넌트 컴포넌트 개발 관련 작업

Projects

Status: 완료

Development

Successfully merging this pull request may close these issues.

[추가기능] 알림 기능

4 participants