Skip to content

hotfix: [release-0.22] AudioCellが0個のときにDEFAULT_PROJECT_FILE_BASE_NAMEでエラーが発生しないように修正 #2484

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Jan 14, 2025
Merged

hotfix: [release-0.22] AudioCellが0個のときにDEFAULT_PROJECT_FILE_BASE_NAMEでエラーが発生しないように修正 #2484

merged 2 commits into from
Jan 14, 2025

Conversation

sigprogramming
Copy link
Contributor

内容

AudioCellが0個のときにDEFAULT_PROJECT_FILE_BASE_NAMEでエラーが発生しないようにします。

関連 Issue

close #2483

その他

@sigprogramming sigprogramming requested a review from a team as a code owner January 10, 2025 13:11
@sigprogramming sigprogramming requested review from Hiroshiba and removed request for a team January 10, 2025 13:11
@voicevox-preview-pages
Copy link

voicevox-preview-pages bot commented Jan 10, 2025 •
edited
Loading

🚀 プレビュー用ページを作成しました 🚀

更新時点でのコミットハッシュ：5dc1894

sevenc-nanashi
sevenc-nanashi approved these changes Jan 10, 2025
View reviewed changes
Copy link
Member

@sevenc-nanashi sevenc-nanashi left a comment

Choose a reason for hiding this comment

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

LGTM！

release-0.22にBackportしてもいいかも？

Copilot
Copilot AI reviewed Jan 12, 2025
View reviewed changes
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.

Hiroshiba
Hiroshiba approved these changes Jan 13, 2025
View reviewed changes
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！！

そもそもソング用にプロジェクトファイル名がトークの状態に依存してるのが若干変でかもですね！
ソング用にプロジェクトファイル名を決める関数を作って、ソングエディターが開かれてる時はそっちを見るようにしたほうが良いかも。

でも結構考えることが多いので一旦このワークアラウンドが良さそう！

この変更をhotfixで出すか若干迷っています。
多分ソングエディターとして開いて、普通に保存しようとしてもエラーが出て保存できないし、解決方法も分からない、という認識であってそうでしょうか 👀

であればさすがにすぐにこの変更だけを適用した修正を出すべき（＝アップデートをリリースするべき）かなと思っています！
（それ次第でマージ先を変えようと思ってます）

Hiroshiba
Hiroshiba reviewed Jan 13, 2025
View reviewed changes
@sevenc-nanashi
Copy link
Member

多分ソングエディターとして開いて、普通に保存しようとしてもエラーが出て保存できないし、解決方法も分からない、

そのはずです。

@sigprogramming
Copy link
Contributor Author

@Hiroshiba
トークエディターを一度も開かずに、新規作成した（もしくはインポートで作成された）プロジェクトを保存しようとしたときにエラーが出ます。
トークエディターを一度開くとAudioCellが作成されるので保存できるようになりますが、これに気付くのは結構難しいと思います。

@Hiroshiba
Copy link
Member

ありがとうございます！
さすがにこれは緊急度が高いのと、直近でリリース想定がないのもあって、久しぶりにhotfixアップデートとしてリリースしたいと思います！

ちなみにマージの宛先をrelease-0.22ブランチにし、マージしたあとバージョンアップしてリリース、そのあとmainブランチにマージする流れです！

@Hiroshiba Hiroshiba changed the title Fix: AudioCellが0個のときにDEFAULT_PROJECT_FILE_BASE_NAMEでエラーが発生しないように修正 hotfix: [release-0.22] AudioCellが0個のときにDEFAULT_PROJECT_FILE_BASE_NAMEでエラーが発生しないように修正 Jan 14, 2025
@Hiroshiba Hiroshiba changed the base branch from main to release-0.22 January 14, 2025 16:13
@Hiroshiba Hiroshiba force-pushed the fix_problem_when_saving_project branch from 9792352 to 5dc1894 Compare January 14, 2025 16:18
@Hiroshiba
Copy link
Member

マージします！　PRありがとうございました！！

@Hiroshiba Hiroshiba merged commit 300fef1 into VOICEVOX:release-0.22 Jan 14, 2025
10 checks passed
@sigprogramming sigprogramming deleted the fix_problem_when_saving_project branch January 15, 2025 10:31
@sigprogramming sigprogramming mentioned this pull request Jan 19, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Hiroshiba Hiroshiba Hiroshiba approved these changes

Copilot code review Copilot Copilot left review comments

@sevenc-nanashi sevenc-nanashi sevenc-nanashi approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

ソング：プロジェクトを保存しようとするとエラーが発生する
3 participants
@sigprogramming @sevenc-nanashi @Hiroshiba