Skip to content
This repository has been archived by the owner on Apr 12, 2023. It is now read-only.

IDeviceVerifierをServiceLocatorで取り扱う #185

Closed
keiji opened this issue May 17, 2021 · 1 comment · Fixed by #189
Closed

IDeviceVerifierをServiceLocatorで取り扱う #185

keiji opened this issue May 17, 2021 · 1 comment · Fixed by #189
Assignees
Labels
enhancement 新しい機能や改善のリクエスト

Comments

@keiji
Copy link
Collaborator

keiji commented May 17, 2021

その機能リクエストは何らかの問題に関連しますか / Is your feature request related to a problem?

#179 (comment) より

現状、IDeviceVerifierはDependencyServiceで依存性解決をしているが、DependencyServiceはAndroidでバックグラウンド動作時に解決できない特性がある。

[assembly: Dependency(typeof(DeviceCheckService))]
namespace Covid19Radar.Droid.Services
{
public class DeviceCheckService : IDeviceVerifier

また、デバッグモード導入( #179 )の際にifディレクティブを使用しなくてはならず、メンテナンス性を落とす一因になっていると考える。

解決策についてお書きください / Describe the solution you'd like

ServiceLocatorで取り扱うようにする。

あなたが考える代替案についてご説明ください / Describe alternatives you've considered

IDeviceVerifierはUI側(陽性登録時)から使用される処理に限定されているので現状問題が発生するとは考えにくい。
このままにしていても機能上問題はなさそう。

その他 / Additional context

#45 に関連。

全部一度にやると大きな変更になるので、このIssueではIDeviceVerifierに限定して対応したい。


Internal Tracking ID: NFR 2466

@keiji keiji added the enhancement 新しい機能や改善のリクエスト label May 17, 2021
@keiji keiji self-assigned this May 17, 2021
@i-maruyama
Copy link
Contributor

PR #191 で確認できました。詳しくは #191 のコードの入手&コードの検証 をご覧ください。

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement 新しい機能や改善のリクエスト
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants