-
Notifications
You must be signed in to change notification settings - Fork 111
Conversation
@@ -74,6 +81,10 @@ IDateTimeUtility dateTimeUtility | |||
_dateTimeUtility = dateTimeUtility; | |||
|
|||
ExposureCheckScores = new ObservableCollection<ExposureCheckScoreModel>(); | |||
UtcDescription = string.Format( | |||
AppResources.LowRiskContactPageAnnotationDecription2, | |||
TimeZoneInfo.Local.StandardName |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
サマータイムがある国だと、表示期間中に複数のタイムゾーンが混在する可能性がある様に思います。
StarndardName だけ表示するのが良いかは検討の余地があるかもしれません。
There was a problem hiding this comment.
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
もそこから引っ張ってくれていることを期待しています。
また、COCOAが日本国内での接触を日本国内で取り扱うという用途を考えると、端末の設定に依らず変換先のタイムゾーンはJST固定でもいい気もします(相談を受ける側の立場からすると別のタイムゾーンの時間帯を告げられても直感的にわからないので混乱するかも)。
There was a problem hiding this comment.
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 環境でどう動くかまでは確認していませんが、動作は変わらないのではと考えています。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
COCOAが日本国内での接触を日本国内で取り扱うという用途を考えると、端末の設定に依らず変換先のタイムゾーンはJST固定でもいい気もします
個人的にはJST固定で良いと考えています。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
端末の設定に依らず変換先のタイムゾーンはJST固定でもいい気もします
@keiji
今はまだ、システムのタイムゾーンを使用されていますが、
改修の予定はあるでしょうか?
私としては、改修範囲も広くなりますので特段理由がないようなら今の実装のままシステムタイムゾーンでもよいかと思っています。
相談を受ける側の立場からすると別のタイムゾーンの時間帯を告げられても直感的にわからないので混乱するかも
この懸念はありますが、この改修によりタイムゾーンの表示もされるのでいくらか解消されますし。
There was a problem hiding this comment.
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になりません。
Issue 番号 / Issue ID
目的 / Purpose
変更内容 / Changes
破壊的変更をもたらしますか / Does this introduce a breaking change?
Pull Request の種類 / Pull Request type
確認事項 / What to check
その他 / Other information
Internal IDs: