search

tourist-project / yuwaku_proto_flutter

湯涌用アプリ
1 stars 0 forks source link

トップページのカメラアイコンからカメラページへの遷移(エミュレータでは動きませんでしたが、実機OK) #126

Closed KitagawaFuuga closed 2 years ago

KitagawaFuuga commented 2 years ago

トップページからカメラページへ遷移するようにしました。エミュレータでは動作しなかったのですが、実機では動きました。実機確認はAndroidのみです。

yamanetaisei commented 2 years ago

@KitagawaFuuga 今エミュレータで動作確認したら普通にできたけどどうしてだろう。。。

どんなエラーが出てるかってわかる？

yamanetaisei commented 2 years ago

一応動作確認もしてみたけど、僕の方では動きました！(Pixcel 4a) 簡単に気になるところをコメント書いといたから、確認してほしい！！ 実装はすごく良い感じだと思う ！ @KitagawaFuuga

KitagawaFuuga commented 2 years ago

確認ありがとうございます！クラスの命名など指摘されたところを直していきたいと思います。

KitagawaFuuga commented 2 years ago

更新しました。クラスの名前をscreenからpageに統一して、無駄なimportを減らしました。mainから引っ張ってきているavailableCameraは他のやり方がわからなかったので、そのままmainから引っ張ってます。エミュレータのエラーというのはカメラを撮る際にフリーズしてしまうというエラーです。エラー文とかは出ない感じです。

yamanetaisei commented 2 years ago

@KitagawaFuuga 対応ありがとう！

ひとまずapproveにするけど、importの削除は、このPRで触れたファイルだけでよかったかも。。。 多分全部のファイル消しちゃったおかげで他のPRと全部のファイルでimportのコンフリクトを起こすことになりそうだね。

あと、レビューの質問にはちゃんと答えてほしくて、レビューに返信する形で対応してもらえると確認しやすくて良いかな