Skip to content

Comments

[fix/42-roulette-filter-mapping] - 룰렛 필터 enum 매핑 오류 수정#43

Merged
Parkchanyoung0710 merged 1 commit intodevelopfrom
fix/42-roulette-filter-mapping
Feb 9, 2026
Merged

[fix/42-roulette-filter-mapping] - 룰렛 필터 enum 매핑 오류 수정#43
Parkchanyoung0710 merged 1 commit intodevelopfrom
fix/42-roulette-filter-mapping

Conversation

@Parkchanyoung0710
Copy link
Contributor

@Parkchanyoung0710 Parkchanyoung0710 commented Feb 9, 2026

변경 사항

  • 룰렛에서 카테고리/상황 필터가 적용되지 않고 전체 메뉴가 반환되던 문제 수정
  • enum key 기준 매핑 방식을 value 기준 비교 방식으로 변경

설계 방향

  • 룰렛에서 호스트가 선택한 카테고리/상황 필터가 무시되어 의도와 다른 추천 결과가 노출되는 UX 문제가 발생
  • 원인 분석 결과, 클라이언트는 enum value를 전달하는 반면 백엔드는 enum key 기준으로 조회하고 있어 필터 변환이 실패하고 있었음

테스트 방법

  1. 룰렛 방 생성 시 카테고리/상황 필터 선택

  2. 룰렛 실행

  3. 선택한 필터에 해당하는 메뉴만 반환되는지 확인

  4. 필터 미선택 시 기존 동작과 동일하게 전체 메뉴 반환되는지 확인

@coderabbitai
Copy link

coderabbitai bot commented Feb 9, 2026

📝 Walkthrough

Walkthrough

채팅 게이트웨이의 toEnumArr 함수 로직이 개선되었습니다. 기존의 인덱스 기반 enum 변환 방식에서 멤버십 기반 접근 방식으로 변경되었으며, Object.values()를 활용하여 enum 값 존재 여부를 확인하고 필터링합니다.

Changes

Cohort / File(s) Summary
Enum 변환 로직 개선
src/chat/gateways/chat.gateway.ts
toEnumArr 함수의 enum 변환 방식이 인덱스 기반(EnumObj[v])에서 멤버십 기반(Object.values 활용)으로 변경됨. "서버에서 필터에 맞는 메뉴 로드" 섹션과 handleUpdateRouletteFilters 흐름 모두에 적용됨.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20분

Poem

🐰 열거형 값들을 더 정확히,
멤버십 확인으로 우아하게,
Object.values 마법으로 변신하여,
필터들이 더 안전하게 흐르네요!
작은 변화, 큰 신뢰성! ✨


Note

🎁 Summarized by CodeRabbit Free

Your organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login.

Comment @coderabbitai help to get the list of available commands and usage tips.

@Parkchanyoung0710 Parkchanyoung0710 merged commit 29f2a36 into develop Feb 9, 2026
2 checks passed
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.

[fix/42-roulette-filter-mapping] - 룰렛 필터 enum 매핑 오류 수정

2 participants