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

基準値未満の接触画面リファインメント #889

Merged

Conversation

keiji
Copy link
Collaborator

@keiji keiji commented Feb 21, 2022

Issue 番号 / Issue ID

目的 / Purpose

  • 日時がUTC基準であることを明記した(接触一覧の画面に表示しているものと同様)
  • (デバッグ用途のみ)利用開始期間より過去に基準値未満の接触が存在する場合に表示する始期の計算が誤っていた不具合を修正した

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

[ ] Yes
[x] No

Pull Request の種類 / Pull Request type

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

Screen Shot 2022-02-22 at 0 05 05

その他 / Other information

@keiji keiji added COCOA2 beta Beta版 labels Feb 21, 2022
@keiji keiji self-assigned this Feb 21, 2022
@keiji keiji requested a review from kazuhiro4949 February 21, 2022 15:09
@keiji keiji changed the title Beta/improve low risk exposure 基準値未満の接触画面リファインメント Feb 21, 2022
@keiji keiji marked this pull request as ready for review February 21, 2022 15:20
</data>
<data name="LowRiskContactPageAnnotationDecription2" xml:space="preserve">
<value>日時については、協定世界時(UTC)を設定されているタイムゾーンに変換して、期間として表示しています。</value>
<comment>TODO:日時については、協定世界時(UTC)をを設定されているタイムゾーンに変換して、期間として表示しています。</comment>
</data>
<data name="LowRiskContactPageAnnotationDecription2" xml:space="preserve">
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

重複している?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

内容は同じですが画面が別なので2箇所に書いてあります。

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nameをキーに値を引いてきていると理解していたのですが、例えばこっちのLowRiskContactPageAnnotationDecription2が持つvalueを変更した場合にはExposureCheckPageには特に影響がないことは保証されているものなんでしょうか?

Copy link
Collaborator Author

@keiji keiji Feb 23, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

おっと、同じキーが2つあるのは良くないですね。
直します!(ビルド通るんか……

@keiji keiji requested a review from kazuhiro4949 February 23, 2022 06:30
@kazuhiro4949 kazuhiro4949 merged commit 529c1d9 into cocoa-mhlw:feature/beta Feb 23, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
beta Beta版
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants