-
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
音声をすべて繋げて書き出す機能を追加 #549
音声をすべて繋げて書き出す機能を追加 #549
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.
確認用と思われるconsole.logが残っていたので、それを消せば後は問題ないと思いました!
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!
判断します!こちらって何になっていますか👀 |
書き出しをfixする機能はありそうですが、上書き防止機能がなさそうでした。 Lines 671 to 678 in c83af58
(全部接続した状態で、上書き防止したいことある?と思わなくもないのですが、一応・・・) |
|
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!
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!!
内容
題の通り
ただし、ショートカットキーの追加は、以下Issue・PRとの整合性を保つために待った方が良さそうです(追加しちゃってるので消す必要がある?)マイグレーション処理は #545 に任せることにしました。
また、ツールバーのカスタマイズ(#489)とも少しconflictを起こしそうです。こちらで必要になっているQuasar Dialogに関するプロセスを
Vuex内dialog.ts
に移動させました。関連 Issue
close #107
その他
書き出すときのデフォルトの名前は要検討かもです。