tourist-project / yuwaku_proto_flutter

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

Androidアルバム許可 #197

Closed keigo194547 closed 2 years ago

keigo194547 commented 2 years ago

Androidでもアルバム許可がなされていない場合の許可を促す処理.

元々出来ていなかった原因

keigo194547 commented 2 years ago

@yamanetaisei 時間ある時確認お願いします.

yamanetaisei commented 2 years ago

今回の変更差分と直接関係なくて恐縮だけど、ひょっとしたらpermissionHandlerがもう入らないかもしれないから、ざっとコードを確認して使われてなかったらついでに消しといてもらえると助かります 🙇

https://github.com/tourist-project/yuwaku_proto_flutter/blob/75617e0bc66368f272987be67c7cf2a46ef627f9/pubspec.yaml#L64

yamanetaisei commented 2 years ago

@keigo194547 レビュー1点だけ。細かいとこですがお願いします :bow:

2点質問で、 僕の方で実機動作確認した方がいいですか?? app_generalを見た感じだけど北川くんが動作確認する予定ぽいですね。それプラス藤田くんが持ってる実機で確認したならもういいかなと思っています。

2点目は、 PRの概要に

Android11以上のデバイスでAPI30以上である時の処理のみであったため利用していた実機はそれ以下

ってあるけど、これについて参考にした記事とかがあればPRの概要にURLを追記しといてもらえると助かります

keigo194547 commented 2 years ago

@yamanetaisei 1点目:了解です.手元のものと北川君で確認します. 2点目:ごめん参考記事貼るの忘れてた.アプリ権限, sdk設定 この2つを参考

keigo194547 commented 2 years ago

手持ちのiOS動作確認Ok @KitagawaFuuga の方でも動作確認おなしゃす

keigo194547 commented 2 years ago

ToDo 初回起動時のポップアップでいいえを押し,2回目以降ポップアップが出ないか調査

keigo194547 commented 2 years ago

問題なし.許可を拒絶した場合は再度写真を撮った後許可を求められるが,今後表示しないを選択したら表示されなくなる

yamanetaisei commented 2 years ago

確認ありがとうございます :bow: 難しいタスクお疲れ様でした!!

LGTM