-
Notifications
You must be signed in to change notification settings - Fork 309
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 歌wave出力時のデフォルトファイル名を変更 #1859
Conversation
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.
プルリクエストありがとうございます!!
かなり良い感じだと思います!!!
あ、フォーマットが必要なのでお願いします!方法はREADMEに書いています 🙏
それと、コードの流れを整理してこんな感じにするのはどうでしょう?
- 関数に切り出す
- プロジェクトファイル名があれば最初にそれを採用
- singerがあればsinger名+5ノートを採用
- どちらもなければsong.wavに
ファイル名と、その処理が現れる順番を一緒にして、優先度をわかりやすくする感じです!
ChatGPT君に(5回くらい)聞いてみたらそれっぽい答えを返してくれたので、参考になれば!!
https://chat.openai.com/share/b17896cb-ffc4-4c2e-9911-44ac33e43e22
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!!!
プルリクエストありがとうございました!!
もしよかったら是非またよろしくお願いします!!!
わからないことがあったら本当に何でも聞いてください・・・!!
ちょっと細かいところをこちらで変更させていただきます!
const beginningPartLyrics = notes | ||
.map((note) => note.lyric) | ||
.join(""); | ||
return sanitizeFileName( |
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.
ファイル名に使えない文字列が入っている可能性を考慮してませんでした 🙇
} | ||
} | ||
|
||
return "Untitled.wav"; |
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.
トーク側がこの名前だったのでなんとなく合わせてみました
問題ないと思うのでマージします!! |
内容
src/store/singing.tsの1981行目に以下を追加しました。
関連 Issue
ref #1833
その他
両方に対応しました。