-
Notifications
You must be signed in to change notification settings - Fork 305
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: UtaFormatixを導入してインポートできる対応形式を増やす #2104
Add: UtaFormatixを導入してインポートできる対応形式を増やす #2104
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.
良いですね!!!!
まだstore/singing.ts
追えてないですが、一旦コメントまで・・・!
外部プロジェクトファイル
コード内で書くときは歌限定なことが伝わる名前にすると無難かもですね!
あとユーザーに露出するとこだと、MIDIファイルやMusicXMLはプロジェクトファイルに見えないかもです。
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.
良い感じだと思います!!
ちょっとコードのメンテナンス性を上げたくいくつかコメントしました!
あとUXも相談したくいくつかコメントしました。もしよければ 🙏
Co-authored-by: Hiroshiba <hihokaruta@gmail.com>
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です!!!
Co-Authored-By: Hiroshiba <Hiroshiba@users.noreply.github.com>
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!!
いくつかコメントしましたが、こちらで書き加えようと思います!
roundの部分だけどうすべきか @sevenc-nanashi さんの意見をお待ちします!
スナップショットテスト良いですね!!
こういうときにはバンバン使っていって良い気もしますね。
あとImportSongProjectDialog.vue
のStorybook興味あったらぜひチャレンジしてみてください。
めっちゃ面白い&健全になると思うのですが、手が出せてないだけなので。。
あ、utaformatixを使って"いろんなソングプロジェクトをimport可能にする"ってタイトルに変えたほうが良いかもですね。 |
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.
変えてみたけど、なんかいびつなコードになりました 😇
940826d
エラーメッセージにエラーメッセージ以外の意味を持たせてますね・・・
{error: Error, message: string}
型にするのが正しそうだけど、まあ良いかな。。。
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!!!!!
お疲れ様でした!!
エクスポート機能も期待しています・・・!!
(あとマルチトラックも・・・!!!)
ツイート文でUtaFormatixへの感謝を伝えたい気持ちがありますが、UtaFormatix公式提供みたいな見え方になっちゃうと迷惑かけちゃうかもですねぇ・・・
まあUtaFormatixを導入し、様々なソングプロジェクトファイル(.ccs/.svp/.vsqなど)のインポートに対応しました🎉
とかが無難ですかね。
そうします!
内容
utaformatixを入れます。
とりあえず「外部プロジェクトファイル」/「外部プロジェクト」と呼んでいますがもっといい名前があるかも。
関連 Issue
(なし)
スクリーンショット・動画など
(なし)
その他
(なし)