Skip to content
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

feat: 書き出し先を固定を有効にしたときに書き出し先が未選択の場合は自動的にダイアログを表示する #1881

Merged
merged 5 commits into from
Mar 20, 2024

Conversation

sabonerune
Copy link
Contributor

内容

書き出し先を固定を有効にしたときに書き出し先が未選択の場合、自動的にダイアログを表示するようにします。
これにより意図せずインストールディレクトリにファイルが保存されてしまう問題を回避します。

関連 Issue

その他

既に書き出し先を固定・未選択の場合起動直後に突然ダイアログが表示されてしまうのでとりあえずマイグレーション処理で書き出し先の固定を無効に切り替えるようにしてしまいました。

@sabonerune sabonerune requested a review from a team as a code owner February 27, 2024 12:08
@sabonerune sabonerune requested review from Hiroshiba and removed request for a team February 27, 2024 12:08
Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!!

fixedExportDirは空白を初期値とせず、undefinedを初期値とした方が扱いがまだわかりやすそうですね!
またfixedExportEnabledがfalseのときは必ずfixedExportDirをundefinedにする、というリファクタリングができそうです。
実際片方が保存された状態でアプリが落ちるとバグりそうですし。

あとでリファクタリング候補一覧に書き足しておくので、もしご興味あれば!

src/backend/common/ConfigManager.ts Outdated Show resolved Hide resolved
src/backend/common/ConfigManager.ts Outdated Show resolved Hide resolved
sabonerune and others added 2 commits March 12, 2024 18:33
Co-authored-by: Hiroshiba <hihokaruta@gmail.com>
Co-authored-by: Hiroshiba <hihokaruta@gmail.com>
Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

マージします!

これ結構バグだと思う人多そうな気がしますね。
多分空白のまま保存している人は結構いて、やっぱり案内されないので「なぜか消えてた」になる気がしています。不信感がちょっと積もる感じですかね。

・・・・・・もし気が向いたらダイアログの方も実装してあげるとユーザーフレンドリーかもですね!

@Hiroshiba Hiroshiba merged commit 05926fb into VOICEVOX:main Mar 20, 2024
9 checks passed
@sabonerune sabonerune deleted the fix/static-save-dir branch March 20, 2024 06:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants