cocoa-mhlw / cocoa

Mozilla Public License 2.0
990 stars 113 forks source link

利用規約・プライバシーポリシーの更新確認を実施しないようにする #1136

Closed cocoa-dev003 closed 1 year ago

cocoa-dev003 commented 1 year ago

Issue 番号 / Issue ID

目的 / Purpose

変更内容 / Changes

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

ghost commented 1 year ago

ついでにイベントログの画面遷移も無効化される(実質呼ばれなくなるため) https://github.com/cocoa-mhlw/cocoa/pull/1136/files#diff-a69e3740eb141fcbafe68ebb47aaa13f62d244531a4d9063af1e4d01cf2592d3L85-R94

上記間違い NavigateNextは呼ばれるので、まだイベントログ関連の画面の処理は生きている。

ghost commented 1 year ago

利用規約・プライバシーポリシーの更新確認をしているjson取得部分のみ呼ばなくしたので、イベントログ関連には問題ありません。

削除という意味では、if文の方も削除するほうがきれいかと思いましたが、このServiceの処理はごっそり無くなると思うので良いと思います。

sonarcloud[bot] commented 1 year ago

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication