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

Update: voicevox/voicevoxに追従 #34

Merged
merged 102 commits into from
Oct 8, 2023
Merged

Update: voicevox/voicevoxに追従 #34

merged 102 commits into from
Oct 8, 2023

Conversation

sevenc-nanashi
Copy link
Member

内容

タイトル通り。
また、いつの間にか起動時にクラッシュするようになってたので後で直します。

(Java APIで書いたら解決しそう感はある)

関連 Issue

(なし)

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

(なし)

その他

(なし)

Hiroshiba and others added 30 commits July 23, 2023 04:47
* デフォルト出力ファイル名の"..."省略条件の変更

* 0じゃなくて1だった
thiramisu and others added 18 commits September 24, 2023 15:29
* ドラッグでAudioCellが入れ替えるテストを追加しました

* 内容のかぶったテストを削除する

ファイル名を変更したときに削除されるべきspecファイルがステージングされてなかったので削除しました

* 処理の共通化、コメントの追加をしました
デザイン変更のプルリクエストは小さめだと嬉しい旨をドキュメントに追記
@sevenc-nanashi sevenc-nanashi requested a review from a team as a code owner October 7, 2023 14:46
@sevenc-nanashi sevenc-nanashi requested review from Hiroshiba and removed request for a team October 7, 2023 14:46
@Hiroshiba
Copy link
Member

Hiroshiba commented Oct 7, 2023

あ、これフォワードマージじゃなきゃですね!(スマホなのでメモだけ📝)

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!!

@sevenc-nanashi
そういえばなんですが、以前どこかで議論した(記憶があるけどしてないかも。。)スマホ向けUIをエディターリポジトリに突っ込むのどうですか。
ブラウザ版もできたので、普通のスマホブラウザからも見れるみたいになると面白そうかなと。
(まあ外部エンジンを設定する機能がまだ無いのですが)

@Hiroshiba
Copy link
Member

e2eテストいっぱい落ちてるけど・・・・・・・まあ良いか!!!

@Hiroshiba Hiroshiba merged commit 09fc91e into main Oct 8, 2023
8 of 14 checks passed
@sevenc-nanashi
Copy link
Member Author

そういえばなんですが、以前どこかで議論した(記憶があるけどしてないかも。。)スマホ向けUIをエディターリポジトリに突っ込むのどうですか。

わりとやりたいですね。追従もやりやすくなりますし。

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.