-
Notifications
You must be signed in to change notification settings - Fork 113
Conversation
|
||
AppCompatDelegate.DefaultNightMode = AppCompatDelegate.ModeNightNo; |
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.
MainApplecationのOnCreate()に移動しています
この位置だとMainActivityのOnCreate()が2回呼ばれる。
base.OnCreate()の前にしたらこれは解消するが、初回起動時に反映されない
ので、MainApplecationのOnCreate()で行うことで解消した
{ | ||
_sendEventLogStateRepository.SetSendEventLogState(EventType.ExposureNotified, ExposureNotifyIsChecked ? SendEventLogState.Enable : SendEventLogState.Disable); | ||
|
||
foreach (var eventType in EventType.All) |
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.
仕様の理解足りてないかもですが、この処理ってなんのためですか?
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.
現状つかっている設定値がEventType.ExposureNotifiedだけですが、
EventType.Allに仮の値が存在しているため、未使用のものはDisableにしています。
複数設定値を考慮しておきたいとのことなのでこのままにしていましたが、使わない値は消しておいたほうが良いですね。
不要な設定値は削除するようにします。
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.
修正しました
9084cf1
この処理も不要ですね。
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.
<comment>・送信は匿名で行われます。</comment> | ||
</data> | ||
<data name="EventLogCooperationPageBullets2" xml:space="preserve"> | ||
<value>- 发送内容中不包括可以识别您和与您接触的阳性登记人的信息。</value> |
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.
中国語って今まで日本語と同じように黒丸にしていませんでしたっけ。
この画面だけ違うの違和感あるので、どちらかに揃えたほうがいいかと思います。
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.
修正しました
c9afc73
|
||
try | ||
{ | ||
foreach (var eventType in EventType.All) |
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.
「あとで設定する」を選択したらNotSetのままだと思ってたんですが、私の仕様の理解違いましたかね?
ACに「アプリ起動時に、on/offどちらにも設定していない送信項目があった場合に都度表示すること」と記載あるので、ちょっとしつこいですがあとで設定するを選択した人には起動時に何回も表示するんだと思ってました。。
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.
後で設定する=設定画面であとで設定する
になるものと思っていました。
確認、調整します。
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.
...ar/Tests/Covid19Radar.UnitTests/ViewModels/Tutorial/EventLogCooperationPageViewModelTests.cs
Outdated
Show resolved
Hide resolved
SonarCloud Quality Gate failed. |
Issue 番号 / Issue ID
目的 / Purpose
変更内容 / Changes
破壊的変更をもたらしますか / Does this introduce a breaking change?
Pull Request の種類 / Pull Request type
確認事項 / What to check
その他 / Other information
Internal IDs: