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

Feature/export exposure data #931

Merged
8 commits merged into from
May 13, 2022

Conversation

keiji
Copy link
Collaborator

@keiji keiji commented Mar 23, 2022

Issue 番号 / Issue ID

目的 / Purpose

  • 接触情報(ExposureData)を出力可能にする

変更内容 / Changes

  • 接触情報をJSONファイルで出力可能にする

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

[ ] Yes
[x] 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:

確認事項 / What to check

Screen Shot 2022-05-13 at 1 08 20

Screen Shot 2022-05-13 at 1 08 42

exposure_data.json.txt

  • 「接触情報を確認」のボタンは接触情報があるときにだけ表示した方が良いのではないか

その他 / Other information


Internal IDs:

@keiji keiji self-assigned this Mar 23, 2022
@keiji keiji marked this pull request as ready for review April 1, 2022 01:18
@keiji
Copy link
Collaborator Author

keiji commented Apr 13, 2022

これ接触の可能性(14日間の接触)画面にもボタン追加した方が良い。

設定画面からエクスポートできるようにすることも考えたけど、エクスポートするために画面遷移があるのはよくない。

@keiji keiji changed the base branch from feature/closed_beta-v2.1.0 to develop April 15, 2022 07:02
@keiji
Copy link
Collaborator Author

keiji commented Apr 15, 2022

うーん。

(あらためて見て)「接触情報を確認」だとちょっとわかりにくい気がします。
特に「陽性登録者との接触一覧」のボタンを押した次の画面で見ていると、ホーム画面の「陽性登録者との接触結果を確認」との違いがわからなくて利用者が混乱しそう。

「詳細なデータを見る」とかでもいいかなと思うのですが、なにかいいアイデアがあれば。

@kvaluation
Copy link
Contributor

「詳細なデータを見る」とかでもいいかなと思うのですが、なにかいいアイデアがあれば。

過去14日間のCOCOA動作情報を確認する

など、色々、配慮して通りやすくしてみるとか、いかがでしょう。とくにこだわりありませんが。

濃厚接触の通知があるときも、他の広範な接触(低リスク接触,検出できたすれ違い)の情報もあると振り返りに役立つと思います。

例1 濃厚接触の日と同日に多数の広範な接触がある→イベント・会食など
例2 濃厚接触の日と違う日で、多数の日に広範な接触がある→通勤通学など。(特定曜日で広範な接触がでるため都心への習い事をオンラインにした方などもいらっしゃいました)

@yoshitomo-g
Copy link

「詳細なデータを見る」とかでもいいかなと思うのですが、なにかいいアイデアがあれば。

「取得する」とか「ファイルに保存する」とかではどうでしょうか。
ボタンのタップで何が起きるかが予測できる単語にしたいですね。
私の場合ですが、「確認する」や「見る」は専用ページへの遷移をイメージします。

@keiji
Copy link
Collaborator Author

keiji commented Apr 15, 2022

濃厚接触の通知があるときも、他の広範な接触(低リスク接触,検出できたすれ違い)の情報もあると振り返りに役立つと思います。

エクスポート機能はDailySummaryとExposureWindow全部出すので、広範な接触も含まれます。
COCOA 1系の動作情報にMatchedCountとかの情報が記録されていたのと同様です。

@keiji
Copy link
Collaborator Author

keiji commented Apr 15, 2022

「取得する」とか「ファイルに保存する」とかではどうでしょうか。

「ファイルに保存する」いいですね! 保存されたのがJSONファイルだと読めない! って言うことになるかと思いますが、変に加工してしまうと今度は加工のロジックの妥当性を検討しなければならなくなったりしそうなのでなるべく避けたいところです。

@yoshitomo-g
Copy link

保存されたのがJSONファイルだと読めない! って言うことになるかと思いますが、

今までもJSONでしたし、「今までは読めたのに!」というような人が出る心配はないと思います。


ちなみに、iOSでのログの保存では「接触チェックの記録を書き出す」です。
v2.0.0で動作情報を取り出すときにアクションの選択を促されて気が付いたのですが、そこに「”ファイル"に保存」があります。

@keiji keiji requested a review from a user May 12, 2022 16:11
@ghost ghost merged commit 672a873 into cocoa-mhlw:develop May 13, 2022
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.

接触データのエクスポート機能を追加する
4 participants