-
Notifications
You must be signed in to change notification settings - Fork 113
[Android] WorkManagerライブラリのアップデート #144
Comments
@keiji さん、 修正自体は以下を変更すれば良いだけだと思いますが、どのようなテストをしてPRすれば良いのか分からず困っています。 cocoa/Covid19Radar/Xamarin.ExposureNotification/Xamarin.ExposureNotification.csproj Line 50 in b147fbb
すみませんが、実機テストで使える端末が今手元にないのでシミュレータ(Debug_Mock)での確認になります。 |
ありがとうございます! ビルドはお使いの環境でのみ確認していただければ大丈夫です。ReleaseビルドについてもPull RequestをいただいたらGitHub Actionsが自動で各Configuration、プラットフォーム別のビルドとユニットテストまで実行して、問題があればここでわかります。 動作確認については、基本的には「アップデート後もこれまで通り動いて、これまであった不具合が改善されている」というようなもので、確認が難しいというのは承知しているので、最終的には開発チーム側で行うリリース前テストで検出することを想定しています。 よろしくお願いいたします。 |
コメントありがとうございます。 土日で対応して後でPR出します。 |
@keiji さん、 #27 で対応されたSetRequiresDeviceIdle(true)を外す修正に関する質問です。 cocoa/Covid19Radar/Xamarin.ExposureNotification/ExposureNotification.android.cs Lines 126 to 132 in b147fbb
参考:Android デベロッパー>ドキュメント>ガイド>WorkRequest の定義>定期的な処理のスケジュール設定 |
はい。大丈夫です。ご指摘の箇所はXamarin.ExposureNotificationライブラリのデフォルト設定になります。 cocoa/Covid19Radar/Covid19Radar.Android/MainActivity.cs Lines 29 to 37 in 872cc86
|
ありがとうございます。理解が深まりました。 |
その機能リクエストは何らかの問題に関連しますか / Is your feature request related to a problem?
#123 の検討過程で提案をもらったもの。
WorkManagerは、Androidでバックグラウンドタスクの実行を管理するライブラリ(バックエンドはJobScheduler)。
#123 (comment)
解決策についてお書きください / Describe the solution you'd like
その他 / Additional context
個人的な理想を言えば、その他のライブラリも含めて、アップデートしたらすぐに追従するPRをして、開発チームに取り込んでもらうくらいのスピード感でいきたい(実際の取り込み段階では、stableかbetaかで判断は分かれそう)
The text was updated successfully, but these errors were encountered: