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

채팅 알림서비스 구현 및 시큐리티 적용 #113

Merged
merged 7 commits into from
Jan 20, 2024

Conversation

Programmer-may
Copy link
Contributor

Resolves: #104

개요

채팅 알림서비스 구현 및 시큐리티 적용
알림 서비스 리팩토링

PR 유형

어떤 변경 사항이 있나요?

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

PR Checklist

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

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

알림서비스 리팩토링 - fcm 전송 메세지 형태 변경
ProductController 시큐리티 적용
Chatontroller 시큐리티 적용
팩토리 패턴 메소드 추가
메세지 전송시 알림 서비스 추가
@Programmer-may Programmer-may added enhancement New feature or request refactor refactoring code labels Jan 20, 2024
@Programmer-may Programmer-may self-assigned this Jan 20, 2024
@Programmer-may Programmer-may merged commit d9b6cb9 into develop Jan 20, 2024
2 checks passed
@Programmer-may Programmer-may deleted the feature/104 branch January 27, 2024 12:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request refactor refactoring code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

채팅 알림 및 시큐리티 도입 리팩토링
3 participants