This repository has been archived by the owner on Apr 12, 2023. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 113
NonContactPageなどに日付を表示 #198
Comments
description にも書きましたが、v1.2.4 で #121 がマージされましたので、ユーザーへの混乱なく「接触有り」にも日時が表示できるようになりました。スクショも追加しておきます。 しかし Auto Merge が煩雑でした。備忘録ですが、結局、個人リポジトリへのマージはウェブから GitHub のボタンで行い、ローカルでの編集分を stash するため一度、VisualStudio でステージして、一時退避しておきます。ネット上でマージした変更をプルリクエストして、一時退避していたものをポップすると、上手くマージされました(ContactedNotifyPageViewModel.cs では、ファイルの変更が存在したが、それも上手くマージされている)。このままビルドすると Xamarin 関連でファイルがないというエラーが出ますが、メニューのビルド関連からクリーンし、再度ビルドすると上手く行きました。(手持ちの教科書が SourceTree & Bitbucket なので・・・) |
i-maruyama
added a commit
to i-maruyama/cocoa
that referenced
this issue
Jun 9, 2021
今日の流れを見ると、コレも #194 の結果を待てと言うことになりそうな。 |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
その機能リクエストは何らかの問題に関連しますか / Is your feature request related to a problem?
上記3に関連し、電車を乗り換えるたびに「陽性者との接触を確認する」ボタンを押すユーザーを失望させるのも問題と考えます。
解決策についてお書きください / Describe the solution you'd like
日付文字列だけを表示するので、翻訳に手を入れる必要はなく、デザイン変更も極小です。また時刻を表示しないことで、一日一度であることを暗にほのめかす効果を狙っています。
あなたが考える代替案についてご説明ください / Describe alternatives you've considered
これは、 #121 がマージされてからでないと、逆に混乱させるので、別 issue でお願いします。v1.2.4 で #121 がマージされましたので、混乱なく「接触有り」にも日時が表示できるようになりました(スクショも追加)。
追加情報表示は #128 (& #160 )で議論されてきましたが、翻訳文字列やデザインの問題になります。マージまで遅い案件になりそうです。
下のスクリーンショットを見ると、M月N日「に」陽性者との接触は確認されませんでしたという意味合いが出ていると、個人的には思います。機能的には、M月N日に GetExposureCount = 0 だったという意味です。
次のリリースでは、少しでも表記が変わっていると、変更点に気づいた人が twitter に乗せてくれる人も出て、普及の一助になるのではないかと期待します。
その他 / Additional context
英語、日本語でのスクリーンショットです。
Internal IDs:
The text was updated successfully, but these errors were encountered: