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

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

Merged
merged 3 commits into from
Nov 12, 2024

Conversation

leegwichan
Copy link
Contributor

Issue Number

Closed #394

As-Is

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

To-Be

  • (방 정보 조회를 제외한) Polling 요청에서 마스터 정보 제거
    • 라운드 종료 조회 API
    • 투표 종료 조회 API
    • 방 초기화 조회 API

Check List

  • 테스트가 전부 통과되었나요?
  • 모든 commit이 push 되었나요?
  • merge할 branch를 확인했나요?
  • Assignee를 지정했나요?
  • Label을 지정했나요?

Test Screenshot

image

(Optional) Additional Description

@leegwichan leegwichan added ♻️ refactor 리팩토링 🍃 BE back end labels Nov 7, 2024
@leegwichan leegwichan self-assigned this Nov 7, 2024
Copy link
Member

@PgmJun PgmJun left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Member

@jhon3242 jhon3242 left a comment

Choose a reason for hiding this comment

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

더 이상 필요없는 정보를 제공할 필요는 없죠! LGTM~~

Copy link
Member

@GIVEN53 GIVEN53 left a comment

Choose a reason for hiding this comment

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

꽤나 귀찮았을 작업일 것 같은데 닉값을 전혀 하지 않으시군요 :)
감사합니다 👍🏻

@leegwichan leegwichan merged commit 13dbd7c into develop Nov 12, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🍃 BE back end ♻️ refactor 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

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