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

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

Merged
merged 1 commit into from
Sep 13, 2022

Conversation

cocoa-dev003
Copy link
Contributor

@cocoa-dev003 cocoa-dev003 commented Sep 13, 2022

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:

  • PBI 8849

@cocoa-dev003 cocoa-dev003 self-assigned this Sep 13, 2022
@cocoa-dev003 cocoa-dev003 requested review from a user and cocoa-dev009 September 13, 2022 06:02
@ghost
Copy link

ghost commented Sep 13, 2022

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

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

Copy link
Contributor Author

@cocoa-dev003 cocoa-dev003 left a comment

Choose a reason for hiding this comment

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

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

@ghost
Copy link

ghost commented Sep 13, 2022

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

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

@sonarcloud
Copy link

sonarcloud bot commented Sep 13, 2022

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

@cocoa-dev003 cocoa-dev003 merged commit 3da2430 into develop Sep 13, 2022
@cocoa-dev003 cocoa-dev003 deleted the feature/delete-terms-update-info branch September 13, 2022 06:36
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