-
Notifications
You must be signed in to change notification settings - Fork 311
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
Dialog周りと初期化周りを移動 #1868
Dialog周りと初期化周りを移動 #1868
Conversation
これ、エンジン起動が完了する前に利用規約とか出てきますね... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ややこしい起動シーケンスなのに挑戦してくださってすごい助かります!!!
これ、エンジン起動が完了する前に利用規約とか出てきますね...
確認なんですけどこれってプルリク前からですよね・・・?
じゃあまあいいかなと! 😇
プルリク後からですね。 |
レビューを反映しました。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ほぼLGTMです!! 変更ありがとうございます!
onetimeWatch良いですね!思いつきませんでした。
あ、テスト落ちてそうでした。
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!!!
良いリファクタリング&バグ解消だと思います!!
conflict解消したのでマージできるはず? |
ありがとうございます、マージします!! |
@sevenc-nanashi あっ! すみません、見逃してたんですが、このプルリクエストでcloseになっているissueはまだ解決してないかもです! 認識あってそうだったらissueを再オープンしていただければ 🙇 |
おっと、再オープンし解きます |
内容
タイトル通りです。
関連 Issue
スクリーンショット・動画など
(なし)
その他
(なし)