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: #123 실시간 알림 api 연동 #127

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from

Conversation

kledyu
Copy link
Collaborator

@kledyu kledyu commented Apr 2, 2024

개요

  1. 실시간 알림 SSE API를 연동해두었습니다. 현재 Response에 대해 백 + 프론트리테스트가 필요해서 아직 완전한 기능을 하지 않습니다.
  2. localhost dev 서버에 대해 HTTPS로 적용해달라는 요청이 있어 설정 파일을 포함해두었습니다. 현재는 HTTP로만 운영합니다.
  3. 오히려 개발에 역효과를 주었던 airbnb 컨벤션을 제거하였습니다.

PR 유형

어떤 변경 사항이 있나요?

  • 새로운 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제
  • 배포 및 PR 관련

PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • 커밋 메시지 컨벤션에 맞게 작성했습니다. Commit message convention 참고.
  • 변경 사항에 대한 테스트를 했습니다.(버그 수정/기능에 대한 테스트).

@kledyu kledyu added ✨ Feature 새로운 기능을 추가한 경우 📱 FE 프론트엔드 ⚙️Settings 프로젝트 환경 및 실행환경 설정 labels Apr 2, 2024
@kledyu kledyu self-assigned this Apr 2, 2024
@kledyu kledyu changed the title feat: #123 실시간 알림 api 연동 Feat: #123 실시간 알림 api 연동 Apr 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📱 FE 프론트엔드 ✨ Feature 새로운 기능을 추가한 경우 ⚙️Settings 프로젝트 환경 및 실행환경 설정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants