Skip to content

Refactor/socket#44

Merged
AndroidNetrunner merged 4 commits intodevelopfrom
refactor/socket
Mar 20, 2022
Merged

Refactor/socket#44
AndroidNetrunner merged 4 commits intodevelopfrom
refactor/socket

Conversation

@Seojunhwan
Copy link
Copy Markdown
Collaborator

✅ PR check list

  • PR 제목: [feat/fix/refactor...] 작업 내용 한 줄 요약 (브랜치 이름) #이슈번호
  • commit message가 적절한지 확인해주세요.
  • 적절한 branch로 요청했는지 확인해주세요.
  • Assignees, Label을 붙여주세요.
  • 주의 사항과 관련해 꼭 확인해야 할 사람이 있다면 Reviewer로 등록해주세요.
  • PR이 승인된 경우 해당 브랜치는 삭제해 주세요!

📝 Changes

  • socket.io 의 기본 연결 방식인 롱폴링, 웹소켓에서 웹소켓만 사용하도록 변경합니다.
  • firebase 삭제합니다.
  • firebase 삭제로 인해 환경변수 타입 변경합니다.

📌 변경 및 주의 사항

추후 파이어베이스 사용 검토시 이전 코드 사용하면 되니 너무 걱정하지 마세요~!
요거 일요일에 만나서 검토해주세요!


기존 롱폴링, 웹소켓 => 웹소켓 단일로 변경
socket.io 으로 인해 firebase 삭제합니다.

추후 다시 firebase 사용이 검토되면 이전 코드에서 가져오면 됩니다!
Copy link
Copy Markdown
Collaborator

@071yoon 071yoon left a comment

Choose a reason for hiding this comment

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

:1

Copy link
Copy Markdown
Owner

@AndroidNetrunner AndroidNetrunner left a comment

Choose a reason for hiding this comment

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

👍

@AndroidNetrunner AndroidNetrunner merged commit 1316022 into develop Mar 20, 2022
@AndroidNetrunner AndroidNetrunner deleted the refactor/socket branch March 20, 2022 02:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants