-
Notifications
You must be signed in to change notification settings - Fork 288
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
Add: 0.19のマイグレーションを追加 #2016
Add: 0.19のマイグレーションを追加 #2016
Conversation
src/backend/common/ConfigManager.ts
Outdated
(config) => { | ||
if ( | ||
Object.prototype.hasOwnProperty.call(config, "showPitchInSongEditor") | ||
) { | ||
config.enablePitchEditInSongEditor = config.showPitchInSongEditor; | ||
delete config.showPitchInSongEditor; | ||
} | ||
}, |
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.
試してみたらうまくいかなかった・・・! 多分experimentalSetting
内ですね!
src/store/project.ts
Outdated
// pitchEditDataの追加 | ||
for (const track of projectData.song.tracks) { | ||
track.pitchEditData = []; | ||
} |
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.
自明なので、何を追加したか日本語を書くのがいいかも
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.
(この上のコードで同じこと書いてました。。 🙇 )
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!!! ありがとうございました! |
* Add: 0.19のマイグレーションを追加 * Fix: showPitchInSongEditorがあるときだけdeleteするようにする * バグ修正とコメント変更 * typescriptのバージョンが古くてエラーになった --------- Co-authored-by: Hiroshiba <hihokaruta@gmail.com>
内容
コンフィグのshowPitchInSongEditor -> enablePitchEditInSongEditor、
トラックのpitchEditDataの追加をマイグレーションするコードを追加します。
関連 Issue
スクリーンショット・動画など
(なし)
その他
(なし)