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

アプリの名称をvoicevoxとしてset #1466

Merged
merged 3 commits into from
Aug 5, 2023

Conversation

g-Ratie
Copy link
Contributor

@g-Ratie g-Ratie commented Aug 3, 2023

内容

app.setNameを使い、アプリ名をvoicevoxとしてsetしました

関連 Issue

ref #1456

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

以下のようにvoicevoxフォルダ内に各種ファイルがあることを確認できました。
Windowsでのスクリーンショット
スクリーンショット 2023-08-03 201627
Macでのスクリーンショット
スクリーンショット 2023-08-03 20 17 22

@g-Ratie g-Ratie requested a review from a team as a code owner August 3, 2023 11:50
@g-Ratie g-Ratie requested review from y-chan and removed request for a team August 3, 2023 11:50
src/background.ts Outdated Show resolved Hide resolved
@g-Ratie
Copy link
Contributor Author

g-Ratie commented Aug 4, 2023

app.setnameの位置を変更し、Win,Mac共にv0.13.4から自分でビルドしたバージョンの間で設定が引き継がれていることを確認しました!

@g-Ratie g-Ratie requested a review from Hiroshiba August 4, 2023 08:25
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!!!!!

2つのOSで丁寧に確認していただけてとても助かります!!!
コメント周りだけ後から見た人が分かりやすいようにちょっとこちらで変更させていただきます!

プルリクエストありがとうございました!
またもしよかったらお待ちしています・・・!!!

src/background.ts Outdated Show resolved Hide resolved
@Hiroshiba Hiroshiba removed the request for review from y-chan August 5, 2023 00:56
@Hiroshiba Hiroshiba merged commit 480cb2d into VOICEVOX:main Aug 5, 2023
@Hiroshiba
Copy link
Member

ちょっとこちらでテストビルドしてみてます!
https://github.com/VOICEVOX/voicevox/releases/tag/0.15.0-preview.4

@umi1299
Copy link
Contributor

umi1299 commented Aug 5, 2023

@Hiroshiba ここにコメントでよいのか分かりませんが…
0.15.0-preview.4をインストールしたところ /Users/<user>/Library/Logs に "空の voicevox-cpu ディレクトリ" と "ログファイル入りの voicevox ディレクトリ" が作成されました。

ss

@Hiroshiba
Copy link
Member

Hiroshiba commented Aug 6, 2023

@umi1299 ご報告ありがとうございます!

これelectron-logの仕様で、デフォルトのディレクトリが必ず作られてしまうんですよね・・・。
#1005 (comment)

挙動としては意図していないので、ちょっとissueを作ってみようと思います!

作ってみました!

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