Conversation
…wn UI, and add new shop features including purchase modals and a shop view.
…dicated dropdown, toast, store, API, and WebSocket service. #90
…list, and pagination. #90
…pping capabilities, along with new shop and user services. #90
[Feat/90] 상점 페이지 구현 및 기타 버그 수정
…d routing for friend management and chat.
…ing IME composition events
…ew services for multiplayer game room management.
…d state management.
…ws, friend system, and development routes. #82
…ation bar, user search modal, and game lobby. #82
…game views and components.
…w development test routes.
…iend service/store. #82
…nd initial game/friend features. #82
…luding new UI components and an updated navigation bar.
…tionToast and NotificationDropdown components. #82
[Feat/82] 친구 및 알림 컴포넌트 구현
…and add new single-player game views and components. #97
[Feat/94] 게임 시간 측정 표시 구현, 기타 버그 수정
[Feat/88] Implement Daily Mvp Componenet
[Feat/96] 멀티 플레이 컴포넌트 통합, 결과 대기 상태 표시 및 기타 버그 수정
[Feat/93] Colors 통일 작업 진행
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (97)
📝 WalkthroughWalkthroughThis PR introduces a comprehensive overhaul of the frontend with significant new features and restructuring: updates API/WebSocket URLs to include Changes
Sequence Diagram(s)sequenceDiagram
participant App
participant AppWS as appWebSocket
participant NotifWS as notificationWebSocket
participant NotifService as notificationService
participant NotifStore as notificationStore
participant UI as UI Components
App->>AppWS: connectAll() on mount
AppWS->>NotifWS: connectNotificationSocket(callback)
NotifWS->>NotifWS: STOMP connection established
NotifWS->>NotifWS: Subscribe to channels
Note over NotifWS: GLOBAL & PERSONAL<br/>notification channels
Note over NotifWS: Incoming notification
NotifWS->>NotifService: onNotification(message)
NotifService->>NotifStore: addToast(notification)
NotifStore->>UI: Toast appears
UI->>NotifService: markAsRead(notificationId)
NotifService->>NotifStore: Update state
NotifStore->>UI: Toast updated
sequenceDiagram
participant NavigationBar
participant FriendPanel
participant FriendService
participant FriendWS as friendWebSocket
participant ChatWindow
participant FriendStore
NavigationBar->>FriendStore: togglePanel()
FriendStore->>FriendPanel: isOpen = true
NavigationBar->>FriendService: getAllFriends()
FriendService->>FriendStore: Load friends
FriendPanel->>FriendPanel: User clicks friend
FriendPanel->>FriendStore: openChatRoom(friendId)
FriendStore->>FriendService: getOrCreateChatRoom()
FriendService->>FriendStore: roomId retrieved
FriendStore->>FriendWS: subscribeToChatRoom(roomId)
FriendWS->>ChatWindow: Messages streamed
ChatWindow->>ChatWindow: User sends message
ChatWindow->>FriendWS: sendChatMessage(roomId, content)
FriendWS->>FriendWS: Publish to /app/chat-rooms
Note over FriendWS: Message received<br/>by other client
sequenceDiagram
participant Game as BaseGameLayout
participant GameWS as WebSocket
participant RoadView as RoadView Component
participant PlayerList as SoloPlayerList
participant Results as FinalResults
participant GameStore
Game->>GameWS: setupGameSubscriptions()
GameWS->>GameWS: Subscribe to game/round/submission channels
Note over GameWS: Game starts
GameWS->>Game: handleGameStart()
Game->>Game: startNextRound()
Note over GameWS: Player submits guess
GameWS->>Game: handlePlayerSubmission()
Game->>GameStore: applySubmissionStatus()
GameStore->>PlayerList: Update player state
Note over GameWS: Round ends
GameWS->>Game: handleRoundEnd()
Game->>GameStore: calculatePlayerScores()
Note over GameWS: Game finishes
GameWS->>Game: handleGameEnd()
Game->>Results: Show final results
Results->>Results: Display podium + opponent states
Game->>Game: Start auto-exit countdown
Estimated code review effort🎯 4 (Complex) | ⏱️ ~75 minutes Possibly related PRs
Poem
✨ Finishing Touches
🧪 Generate unit tests (beta)
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary by CodeRabbit
New Features
Bug Fixes
Refactoring
Documentation
Chores
/apiroute prefix