Skip to content

Fix: 경로 수정#8

Merged
stoneTiger0912 merged 3 commits intomainfrom
feat/add-save-cardset
Mar 17, 2026
Merged

Fix: 경로 수정#8
stoneTiger0912 merged 3 commits intomainfrom
feat/add-save-cardset

Conversation

@stoneTiger0912
Copy link
Member

@stoneTiger0912 stoneTiger0912 commented Mar 17, 2026

📝 변경 내용


✅ 체크리스트

  • 코드가 정상적으로 동작함
  • 테스트 코드 통과함
  • 문서(README 등)를 최신화함
  • 코드 스타일 가이드 준수

💬 기타 참고 사항

Summary by CodeRabbit

릴리스 노트

  • 버그 수정
    • 입력 데이터 타입 처리 개선으로 시스템 안정성 강화
    • WebSocket 연결 구성 최적화

참고: 이번 업데이트는 주로 시스템 내부 안정성 개선에 중점을 두었습니다.

@stoneTiger0912 stoneTiger0912 self-assigned this Mar 17, 2026
@coderabbitai
Copy link

coderabbitai bot commented Mar 17, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: bb9f4f72-930c-4a2b-8fe2-5b42d3e1cb64

📥 Commits

Reviewing files that changed from the base of the PR and between 35e8a45 and d73d297.

📒 Files selected for processing (2)
  • src/cardset/infrastructure/grpc/cardset.grpc-controller.ts
  • src/collaboration/infrastructure/gateway/collaboration.gateway.ts

개요

이 PR은 gRPC 컨트롤러에서 입력값을 숫자 타입으로 변환하는 처리를 추가하고, WebSocket 게이트웨이의 데코레이터 설정을 업데이트합니다.

변경 사항

집합 / 파일(s) 요약
gRPC 컨트롤러 타입 변환
src/cardset/infrastructure/grpc/cardset.grpc-controller.ts
isCardSetViewablegetCardSetsByIds 메서드에서 cardSetId, userId, cardSetIds 입력값을 명시적으로 숫자 타입으로 변환하여 유스케이스 계층에 전달합니다.
WebSocket 설정 업데이트
src/collaboration/infrastructure/gateway/collaboration.gateway.ts
WebSocketGateway 데코레이터의 설정 옵션을 namespace에서 path로 변경합니다.

예상 코드 리뷰 소요 시간

🎯 2 (Simple) | ⏱️ ~10 minutes

관련 PR

  • Fix: 동기화 수정 #3: 동일한 collaboration.gateway.ts 파일을 수정하는 PR으로, WebSocketGateway 데코레이터 옵션 변경 관련 파일 레벨의 연결이 있습니다.

🐰 숫자로 타입을 다지고, 웹소켓 길을 정하니
데이터는 흐르고 연결은 더욱 명확하게!
작은 변화가 모여 튼튼한 기초를 이루는,
그런 아름다운 날 🌟

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/add-save-cardset
📝 Coding Plan
  • Generate coding plan for human review comments

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

Tip

CodeRabbit can generate a title for your PR based on the changes.

Add @coderabbitai placeholder anywhere in the title of your PR and CodeRabbit will replace it with a title based on the changes in the PR. You can change the placeholder by changing the reviews.auto_title_placeholder setting.

@stoneTiger0912 stoneTiger0912 merged commit b1e461d into main Mar 17, 2026
1 of 2 checks passed
@stoneTiger0912 stoneTiger0912 deleted the feat/add-save-cardset branch March 17, 2026 05:52
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.

1 participant