Skip to content

Conversation

@tnals0924
Copy link
Member

📌 관련 이슈번호

🎟️ PR 유형

어떤 변경 사항이 있나요?

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

Check List

  • 🔀 PR 제목의 형식을 잘 작성했나요?
  • 🏗️ yarn build는 성공했나요?
  • 🧹 불필요한 코드는 제거했나요? (TODO, 주석, clg... etc.)
  • ✅ 컨벤션을 지켰나요?
  • 💭 이슈는 등록했나요?
  • 🏷️ 라벨은 등록했나요?
  • 🙇‍♂️ 리뷰어를 지정했나요?

✅ Key Changes

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

  1. PWA가 모바일 메인 페이지에서만 적용되고 있던 현상을 수정했습니다.
  2. 푸시 알림 권한이 denied라면 다시 사용자가 활성화하지 않는 이상 권한 변경이 불가능한데, denied 상태일 때도 페이지에 접속할 때마다 권한을 요청해버려서 요청을 거부했다는 모달이 반복해서 뜨고 있었습니다.
    -> 일단 denied일 때는 모달이 뜨지 않도록 수정해놓았습니다. 왜 알림 거부가 기본 상태로 되어있는 지는 좀 더 알아볼게요!

📢 To Reviewers

📸 스크린샷

🔗 참고 자료

@tnals0924 tnals0924 added the 🐛 BUG 버그를 고친 경우 label Feb 27, 2025
@tnals0924 tnals0924 self-assigned this Feb 27, 2025
@tnals0924 tnals0924 linked an issue Feb 27, 2025 that may be closed by this pull request
2 tasks
@vercel
Copy link

vercel bot commented Feb 27, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
frontend ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 27, 2025 4:12am

@github-actions github-actions bot added the ✨ FEAT 기능 구현 label Feb 27, 2025
Copy link
Member

@sinji2102 sinji2102 left a comment

Choose a reason for hiding this comment

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

LGTM~

@tnals0924 tnals0924 merged commit 80aa205 into develop Feb 27, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛 BUG 버그를 고친 경우 ✨ FEAT 기능 구현

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[ Bug ] PWA 세팅 및 푸시 알림 권한 요청 수정

3 participants