-
Notifications
You must be signed in to change notification settings - Fork 1
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
merge to main #236
merge to main #236
Conversation
haryung-lee
commented
Oct 4, 2022
•
edited
Loading
edited
- 친구기능 추가
- axios instance화
- lobby socket 수정
- 볼륨 조절 오류 수정
change socket name string to enum const
FIRE-667 add socket enum
Fire 689 다른 사용자 볼륨 조절 가능
FIRE-692 fix muted
…o FIRE-691-유저에게-친구-신청을-보낼-수-있다
FIRE-691 유저에게 친구 신청을 보낼 수 있다
Fire 693 사용자 입장시 채팅 UI 변경되는 것 수정
works only at room
FIRE-695-방에서-친구-수락-및-거절-api-연동
…관계를-확인할-수-있다 Fire 696 프로필 모달 혹은 프로필 자체에서 친구 관계를 확인할 수 있다
…않게-하기 Fire 694 로비 참여자 목록에서 중복 유저 보이지 않게 하기
Fire 690 axios 객체로 만들어두고 재사용
…o FIRE-697-프로필-창에서-친구-신청을-보낼-수-있다
FIRE-697 프로필 창에서 친구 신청을 보낼 수 있다
…되는-것-수정 FIRE-700 disableButton when permission is prompt
FIRE-702 fix editMyPage
…서-프로필-페이지로-이동-가능 FIRE-703-로비-프로피-모달-친구-방정보의-방장-에서-프로필-페이지로-이동-가능
Test summaryRun details
View run in Cypress Dashboard ➡️ Failures
This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
import lobbySocket from './lobbySocket'; | ||
|
||
const getSocketData = (roomId: string) => { | ||
const { connectedUsers: roomConnected } = connectedUsersStore(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이따가 파일로 분리해야될거같아용