Skip to content

Commit b704a56

Browse files
authored
Merge pull request #166 from espadonne/main
fix takes_imported never logged from TakesImportModal
2 parents fac99cd + 3de63f9 commit b704a56

2 files changed

Lines changed: 6 additions & 2 deletions

File tree

components/audio/DAW/Multitrack/MultitrackEditor.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1212,7 +1212,7 @@ export default function MultitrackEditor({ availableTakes: propTakes = [], logOp
12121212
show={showTakesModal}
12131213
onHide={() => setShowTakesModal(false)}
12141214
takes={availableTakes}
1215-
onImport={handleImportTake}
1215+
logOperation={logOperation}
12161216
/>
12171217

12181218
{/* Clip Effects Modal */}

components/audio/DAW/Multitrack/TakesImportModal.js

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ import { FaFileAudio, FaClock, FaMusic } from 'react-icons/fa';
77
import { useMultitrack } from '../../../../contexts/MultitrackContext';
88
import { getAudioProcessor } from './AudioProcessor';
99

10-
export default function TakesImportModal({ show, onHide, takes = [] }) {
10+
export default function TakesImportModal({ show, onHide, takes = [], logOperation = null }) {
1111
const { addTrack, updateTrack } = useMultitrack();
1212
const [selectedTake, setSelectedTake] = useState(null);
1313
const [trackName, setTrackName] = useState('');
@@ -127,6 +127,10 @@ export default function TakesImportModal({ show, onHide, takes = [] }) {
127127

128128
console.log('🎵 TakesImportModal: Track created immediately with ID:', newTrack.id);
129129

130+
if (logOperation) {
131+
logOperation('takes_imported', { takeName: selectedTake.name || trackName });
132+
}
133+
130134
// Close modal immediately - track is already visible
131135
setSelectedTake(null);
132136
setTrackName('');

0 commit comments

Comments
 (0)