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

Check ExposureNotification API is enabled/activated onAppearing. #786

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

keiji
Copy link
Collaborator

@keiji keiji commented Jan 19, 2022

Issue 番号 / Issue ID

#785 に依存した変更。

目的 / Purpose

  • 一部端末で接触通知OFFで陽性登録をした時の挙動が想定と異なるの解決策の1つとしての提案

破壊的変更をもたらしますか / Does this introduce a breaking change?

[x] Yes
[ ] No

Pull Request の種類 / Pull Request type

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

検証方法 / How to test

コードの入手 / Get the code

git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
dotnet restore

コードの検証 / Test the code


確認事項 / What to check

  • 全端末の挙動を変えることになるので検討が必要。
  • 場合によってはv2.0.0から外しても良いかも。

その他 / Other information

@keiji keiji self-assigned this Jan 19, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

一部端末で接触通知OFFで陽性登録をした時の挙動が想定と異なる
1 participant