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

[Refactoring] voice room 데이터 관련 리팩토링 #116

Merged
merged 7 commits into from
Aug 18, 2024

Conversation

drbug2000
Copy link
Collaborator

📝 요약

🔖 변경 사항

  • livekit room name을 roomName+#roomId로 관리하던 방식을 roomId로 수정 했습니다. 이제 room의 이름을 수정하더라도, 같은 room으로 인식합니다.
  • 참가자 profile image를 response와 liveKit Token metadata에 저장합니다. 백엔드 API, livekit API 로도 참가자의 프로필 이미지를 받아 올 수 있습니다.
  • voiceRoom API에서 반환하던 userName을 userSpaceName으로 전부 변경했습니다. 해당 userSpaceDao 함수 또한 추가 하였습니다.

📸 확인 방법 (선택)

기존 방식으로 liveKit 참가 후, space-Profile 수정, voiceRoom 이름 수정 API를 사용해도, 정상 동작합니다.

단, 서버 API는 정보를 수정해도 일관성을 유지 하지만, LiveKit Token 정보는 중간에 수정 불가능합니다.
LiveKit API로 받아온 정보는 수정해도 바뀌지 않습니다.


📌 PR 진행 시 이러한 점들을 참고해 주세요

* P1 : 꼭 반영해 주세요 (Request Changes) - 이슈가 발생하거나 취약점이 발견되는 케이스 등
* P2 : 반영을 적극적으로 고려해 주시면 좋을 것 같아요 (Comment)
* P3 : 이런 방법도 있을 것 같아요~ 등의 사소한 의견입니다 (Chore)

@drbug2000 drbug2000 linked an issue Aug 16, 2024 that may be closed by this pull request
4 tasks
Copy link
Collaborator

@seongjunnoh seongjunnoh left a comment

Choose a reason for hiding this comment

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

LGTM

@drbug2000 drbug2000 merged commit 445ca4e into develop Aug 18, 2024
3 checks passed
@hyunn522 hyunn522 deleted the refactoring/#100/voice-room-data-save branch August 18, 2024 17:31
seongjunnoh pushed a commit that referenced this pull request Oct 30, 2024
…ata-save

[Refactoring] voice room 데이터 관련 리팩토링
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Refactoring : voice room 리팩토링 : 정보 저장 방식 변경
2 participants