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

Fixed status check when exposure detection #1028

Merged
2 commits merged into from
Jun 6, 2022
Merged

Conversation

ghost
Copy link

@ghost ghost commented Jun 2, 2022

Issue 番号 / Issue ID

目的 / Purpose

  • Bluetoothオフでも接触確認を行うようにする

変更内容 / Changes

  • Active以外のStatusでも接触確認ができるものがあるためStatusチェックをしないように修正
    • 接触確認に失敗したときのエラーハンドリングがあるため、その場合は接触が確認できないというエラーが表示されるため、エラー(Status異常)を無視するよりは、このほうが望ましいと思われる。
  • ユーザが許可していない場合はIsEnabledAsync()で確認している
  • Android 10以下で位置情報がオフ(LOCATION_DISABLED)の場合も同様

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

[ ] Yes
[x] No

Pull Request の種類 / Pull Request type

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

確認事項 / What to check

  • [ ]

その他 / Other information


Internal IDs:

  • Bug 6571

@ghost ghost requested review from a user, cocoa-dev003 and cocoa-dev009 June 2, 2022 07:06
@ghost ghost self-assigned this Jun 2, 2022
@sonarcloud
Copy link

sonarcloud bot commented Jun 3, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@ghost ghost merged commit 527a16e into develop Jun 6, 2022
@ghost ghost deleted the feature/bluetooth-check branch June 6, 2022 02:49
This pull request was closed.
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.

Bluetoothをオフにしていると診断キーのダウンロードが行われない
1 participant