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

複数エンジン対応:エンジン毎の操作欄を追加 #884

Merged
merged 17 commits into from
Sep 2, 2022

Conversation

sevenc-nanashi
Copy link
Member

@sevenc-nanashi sevenc-nanashi commented Aug 4, 2022

内容

エンジン毎の再起動ボタンを追加します。

関連 Issue

VOICEVOX/voicevox_project#2

スクリーンショット・動画など

image

エンジンが一つしかない場合は従来のUIのようになっています。
image

その他

image
何故かクリックできてしまうのでDraft。
→修正。

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.

PRありがとうございます!!

.env.production Show resolved Hide resolved
src/components/MenuBar.vue Outdated Show resolved Hide resolved
@Hiroshiba
Copy link
Member

Hiroshiba commented Aug 4, 2022

あ、テスト落ちていそうです。
https://github.com/VOICEVOX/voicevox/runs/7669946216?check_suite_focus=true#step:11:17
たぶんnameフィールド足したからテスト側コードも足さないといけないかも・・・?

@sevenc-nanashi
Copy link
Member Author

直しましたー。>テスト

@sevenc-nanashi
Copy link
Member Author

image
アイコン表示を追加してみました。
DEFAULT_ENGINE_INFOS のjsonにあるiconのパスから画像を読んでbase64化し、Vue側に渡しています。

@sevenc-nanashi sevenc-nanashi marked this pull request as ready for review August 5, 2022 00:42
@sevenc-nanashi
Copy link
Member Author

image
フォルダを開くボタン。EngineInfo.pathでエンジンの場所を指定する感じです。(enginesフォルダ読み込みの時にpathが埋まる想定。).env.productionは.にしてVOICEVOX.exeのフォルダが開くようになっていますが、なくしても良いかも?

@sevenc-nanashi sevenc-nanashi changed the title Add: エンジン毎の再起動を追加 複数エンジン対応:エンジン毎の再起動を追加 Aug 5, 2022
@sevenc-nanashi sevenc-nanashi changed the title 複数エンジン対応:エンジン毎の再起動を追加 複数エンジン対応:エンジン毎の操作欄を追加 Aug 5, 2022
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.

追加ありがとうございます!!素敵な機能だと思います!!

src/background.ts Outdated Show resolved Hide resolved
src/background.ts Outdated Show resolved Hide resolved
src/components/MenuBar.vue Outdated Show resolved Hide resolved
src/background.ts Outdated Show resolved Hide resolved
src/background.ts Outdated Show resolved Hide resolved
src/type/preload.ts Outdated Show resolved Hide resolved
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!!

@Segu-g さん、 @y-chan さん、もしよかったらレビューいただけると心強いです!!

@Hiroshiba
Copy link
Member

@raa0121 さん

もしよかったらレビューお願いしても良いでしょうか 👀
とりあえず動かしてみたり、コードをざっと眺めたりして、目立った問題がなければ気軽にapprove頂けると・・・!

@Hiroshiba Hiroshiba closed this Aug 31, 2022
@Hiroshiba
Copy link
Member

間違えてcloseしてしまいました・・・。

@Hiroshiba Hiroshiba reopened this Aug 31, 2022
@raa0121
Copy link
Contributor

raa0121 commented Sep 2, 2022

お、レビュー依頼が来てたのを見逃してました。
レビュー・動作確認しますね。

@raa0121
Copy link
Contributor

raa0121 commented Sep 2, 2022

手元で .env に複数エンジン設定して、動かしてみました。
問題なさそうでした。

@Hiroshiba
Copy link
Member

@raa0121 さん、レビューありがとうございます!!
複数エンジン対応のUI周りのタスクが1つ完了しました!!この調子でガシガシ実装して頂けると本当に嬉しいです!もしよければぜひ・・・・・・!!

@Hiroshiba Hiroshiba merged commit c166b90 into VOICEVOX:main Sep 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants