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

refactor: e2eテストでのトーク・ソング画面へのnavigate関数を分離 #2626

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

jdkfx
Copy link
Contributor

@jdkfx jdkfx commented Mar 21, 2025

内容

以下のPRからCherry-pickして少し修正を追加

関連 Issue

#2011

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

その他

#2409 がマージされてからマージしたほうがいい?

@jdkfx jdkfx requested a review from a team as a code owner March 21, 2025 14:42
@jdkfx jdkfx requested review from Hiroshiba and removed request for a team March 21, 2025 14:42
@jdkfx jdkfx changed the title Refactor/improve e2e test エディター選択ダイアログに関するE2Eテストを追加 Mar 21, 2025
@voicevox-preview-pages
Copy link

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

更新時点でのコミットハッシュ:50fa2ea

@Hiroshiba
Copy link
Member

Hiroshiba commented Mar 21, 2025

あ、主旨が違うかもです!

やりたいのは #2409 のプルリクエストの一部を切り分けることなので、こっちをマージしてから #2409 をマージするのが良いと思います。
#2409 がエディター選択ダイアログを作ることと、e2eテスト周りのリファクタリングを含んでいるので、リファクタリングだけ先に切り出す感じ!

なのでタイトルは「refactor: e2eテストでのトーク・ソング画面へのnavigate関数を分離」とかにして、navigateToTalknavigateToSongを今のmainブランチに合う形にするのが良いのかなと!
これで #2409 のテストに合わせるための変更が最小で済む・・・はず!

@jdkfx
Copy link
Contributor Author

jdkfx commented Mar 22, 2025

@Hiroshiba ありがとうございます!タイトルそのままいただいて変更させていただきます!

@jdkfx jdkfx changed the title エディター選択ダイアログに関するE2Eテストを追加 refactor: e2eテストでのトーク・ソング画面へのnavigate関数を分離 Mar 22, 2025
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.

None yet

2 participants