-
Notifications
You must be signed in to change notification settings - Fork 349
fix: ソングエディタでvvproj以外のファイルからのインポートが動作していないのを修正 #2693
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: ソングエディタでvvproj以外のファイルからのインポートが動作していないのを修正 #2693
Conversation
|
🚀 プレビュー用ページを作成しました 🚀 更新時点でのコミットハッシュ: |
sevenc-nanashi
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!ちょっと変えてマージします。
|
確認不足でした…修正PRありがとうございます! |
| throw new Error("Track not found."); | ||
| } | ||
| const rawTrack = toRaw(track); | ||
| // TODO: トラックの変換処理を関数化する |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
マージ後ですみません!
ここtoRawがなくなっていますが、大丈夫でしょうか…?
(確認したところ、importedTrack.pitchEditDataなどがProxyのままになってそうでした)
@sevenc-nanashi さん、コメントいただけると…!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ん〜確かにProxyになってますね...少なくとも問題になりそうな箇所はない(structuredCloneぐらいしかなくて、そしてrfdcで置き換えられてるはず)けど一応Proxyは外した方がいいかもですね。プルリク出します。
Co-authored-by: sevenc-nanashi <[email protected]>
内容
#2675 でfilteredTracksを生成する際にnoteとしてインポートファイルから生成されたtracksを元にしたtrackではなくrawTrackの方を使うようになっていたので、それ以前の様にtrackを使うように戻しました。
関連 Issue
close #2692
スクリーンショット・動画など
その他