chore(05-03): clean up debug logs and document MIXER_CHANGES handler
- Remove debug logs from JKSessionScreen backing track display - Add comment to MIXER_CHANGES handler explaining it's not currently used - Document that media arrays are now extracted from REST API in useSessionModel Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
This commit is contained in:
parent
251788d799
commit
973d0c6ef2
|
|
@ -190,16 +190,6 @@ const JKSessionScreen = () => {
|
|||
const backingTrackData = useSelector(selectBackingTrackData);
|
||||
const showBackingTrackPlayer = Boolean(backingTrackData);
|
||||
|
||||
// Debug backing track display
|
||||
useEffect(() => {
|
||||
console.log('[SESSION_SCREEN] Backing track display check:', {
|
||||
showBackingTrackPlayer,
|
||||
backingTrackData,
|
||||
'mixerHelper.backingTracks': mixerHelper.backingTracks,
|
||||
'mixerHelper.backingTracks?.length': mixerHelper.backingTracks?.length
|
||||
});
|
||||
}, [showBackingTrackPlayer, backingTrackData, mixerHelper.backingTracks]);
|
||||
|
||||
// Stable callback for backing track popup close
|
||||
const handleBackingTrackClose = useCallback(() => {
|
||||
console.log('JKSessionScreen: Backing Track Popup closing');
|
||||
|
|
|
|||
|
|
@ -119,15 +119,14 @@ export const useSessionWebSocket = (sessionId) => {
|
|||
},
|
||||
|
||||
// Phase 5: Mixer and Media events
|
||||
// NOTE: MIXER_CHANGES message type not currently sent by server
|
||||
// Media arrays (backingTracks, jamTracks, recordedTracks) are now extracted
|
||||
// from session REST API response in useSessionModel.js
|
||||
MIXER_CHANGES: (sessionMixers) => {
|
||||
console.log('[MIXER_CHANGES] Received:', sessionMixers);
|
||||
console.log('[MIXER_CHANGES] Received (legacy handler):', sessionMixers);
|
||||
const session = sessionMixers.session;
|
||||
const mixers = sessionMixers.mixers;
|
||||
|
||||
console.log('[MIXER_CHANGES] Backing tracks in payload:', mixers.backingTracks);
|
||||
console.log('[MIXER_CHANGES] Jam tracks in payload:', mixers.jamTracks);
|
||||
console.log('[MIXER_CHANGES] Recorded tracks in payload:', mixers.recordedTracks);
|
||||
|
||||
// Update media summary
|
||||
if (mixers.mediaSummary) {
|
||||
dispatch(updateMediaSummary(mixers.mediaSummary));
|
||||
|
|
|
|||
Loading…
Reference in New Issue