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

Improve exposure list #996

Merged
3 commits merged into from
May 17, 2022
Merged

Improve exposure list #996

3 commits merged into from
May 17, 2022

Conversation

keiji
Copy link
Collaborator

@keiji keiji commented May 2, 2022

Issue 番号 / Issue ID

目的 / Purpose

  • 「過去14日間の接触一覧」画面の表示をわかりやすくする

変更内容 / Changes

Screen Shot 2022-05-03 at 0 52 07

破壊的変更をもたらしますか / 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

  • テキストについては仮のもの。確認が必要
  • 英語・中国語の翻訳が発生します。

その他 / Other information


Internal IDs:

@keiji keiji self-assigned this May 2, 2022
@keiji keiji marked this pull request as ready for review May 2, 2022 16:10
@@ -74,6 +81,10 @@ IDateTimeUtility dateTimeUtility
_dateTimeUtility = dateTimeUtility;

ExposureCheckScores = new ObservableCollection<ExposureCheckScoreModel>();
UtcDescription = string.Format(
AppResources.LowRiskContactPageAnnotationDecription2,
TimeZoneInfo.Local.StandardName
Copy link

Choose a reason for hiding this comment

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

サマータイムがある国だと、表示期間中に複数のタイムゾーンが混在する可能性がある様に思います。
StarndardName だけ表示するのが良いかは検討の余地があるかもしれません。

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

「表示期間中にサマータイムに入る」「サマータイムから通常時間に戻る」という境界の取り扱いですね。
今回の場合、「そもそもタイムゾーンにおける時刻の補正がうまくいっていないケースがあるらしい」ことへの対応なので、対応するとして別Issueで、になるかと思います。

ちなみにAndroidだとシステムレベルで「アメリカ太平洋時間」と「アメリカ太平洋標準時」を自動で切り替えてくれるので、.NETレベルのTimeZoneInfo.Localもそこから引っ張ってくれていることを期待しています。

Screen Shot 2022-05-07 at 10 49 57

また、COCOAが日本国内での接触を日本国内で取り扱うという用途を考えると、端末の設定に依らず変換先のタイムゾーンはJST固定でもいい気もします(相談を受ける側の立場からすると別のタイムゾーンの時間帯を告げられても直感的にわからないので混乱するかも)。

Copy link

@b-wind b-wind May 7, 2022

Choose a reason for hiding this comment

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

境界での時刻表示を念頭に置いて居ましたが、よく考えるとそれに限った話では無かったですね。

.NET 的にローカルタイムゾーンへの変換処理は 太平洋標準時( Pacific Standard Time, UTC-8 )と 太平洋夏時間( Pacific Daylight Time, UTC-7 )の切替は自動で行われる事を確認しています。

今年の夏時間は3月からなので今の時期だと主にPDTで表示されている筈ですね。

これに対し TimeZoneInfo.Local は現在時刻の情報は持たないので、 StandardName は PST 、DaylightName は PDT で固定のようです。

Xamarin.Android/iOS 環境でどう動くかまでは確認していませんが、動作は変わらないのではと考えています。

Copy link

Choose a reason for hiding this comment

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

COCOAが日本国内での接触を日本国内で取り扱うという用途を考えると、端末の設定に依らず変換先のタイムゾーンはJST固定でもいい気もします

個人的にはJST固定で良いと考えています。

Copy link

Choose a reason for hiding this comment

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

端末の設定に依らず変換先のタイムゾーンはJST固定でもいい気もします

@keiji
今はまだ、システムのタイムゾーンを使用されていますが、
改修の予定はあるでしょうか?

私としては、改修範囲も広くなりますので特段理由がないようなら今の実装のままシステムタイムゾーンでもよいかと思っています。

相談を受ける側の立場からすると別のタイムゾーンの時間帯を告げられても直感的にわからないので混乱するかも

この懸念はありますが、この改修によりタイムゾーンの表示もされるのでいくらか解消されますし。

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

はい。このPull Requestではこのままでいきましょう。

タイムゾーンをJSTに固定するとしても別のPRで対応します(なので、このPRがマージされても #978 はCloseになりません。

@ghost ghost merged commit 12dab4b into cocoa-mhlw:develop May 17, 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.

接触一覧画面のタイムゾーンに関する表記を改善
2 participants