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.

Sign up for GitHub

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

Jump to bottom

[REFACTOR] Polling에서 사용하지 않는 바디 정보 제거 #394

Closed
leegwichan opened this issue Nov 7, 2024 · 0 comments · Fixed by #396
Closed

[REFACTOR] Polling에서 사용하지 않는 바디 정보 제거 #394

leegwichan opened this issue Nov 7, 2024 · 0 comments · Fixed by #396
Assignees
@leegwichan
Labels
♻️ refactor 리팩토링

Comments

@leegwichan
Copy link
Contributor

Description 💭

  • 현재 모든 폴링 요청에서 "현재 방의 마스터 정보"를 제공해 주고 있었음
    • 이전 버전에서 "방장 뷰"와 "일반 참여자 뷰"를 구분하기 위한 정보로 사용
  • 더 이상 해당 정보가 (방 정보 조회를 제외한) Polling 요청에서 필요하지 않음
    • 현 버전에서는 "내 정보 조회 API"의 캐싱을 사용하여 자신의 정보를 조회함
  • 해당 정보는 코드 및 쿼리 등에서 리소스를 잡아 먹기 때문에 제거 필요

TODO ✅

  • (방 정보 조회를 제외한) Polling 요청에서 마스터 정보 제거

Reference 🔎
@leegwichan leegwichan added the ♻️ refactor 리팩토링 label Nov 7, 2024
@leegwichan leegwichan self-assigned this Nov 7, 2024
leegwichan added a commit that referenced this issue Nov 7, 2024
@leegwichan
feat: 라운드 종료 여부 API에서 마스터 정보 제거 #394
331186e
leegwichan added a commit that referenced this issue Nov 7, 2024
@leegwichan
feat: 투표 종료 여부 조회 시, 마스터 정보 제거 #394
bd870a7
leegwichan added a commit that referenced this issue Nov 7, 2024
@leegwichan
feat: 방 초기화 확인 API에서 마스터 정보 제거 #394
db1bc15
@leegwichan leegwichan mentioned this issue Nov 7, 2024
Merged
5 tasks
leegwichan added a commit that referenced this issue Nov 12, 2024
@leegwichan
merge: Polling에서 사용하지 않는 바디 정보 제거 #394
13dbd7c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@leegwichan leegwichan

Labels
♻️ refactor 리팩토링
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[REFACTOR] Polling에서 사용하지 않는 바디 정보 제거 woowacourse-teams/2024-ddangkong
1 participant
@leegwichan