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

Feature/subregion #557

Draft
wants to merge 23 commits into
base: develop
Choose a base branch
from
Draft

Feature/subregion #557

wants to merge 23 commits into from

Conversation

keiji
Copy link
Collaborator

@keiji keiji commented Dec 5, 2021

Issue 番号 / Issue ID

#634 に依存した変更

目的 / Purpose

  • 陽性登録の際、アプリから regions, subregions の2つのパラメータを送信する

Screen Shot 2021-12-05 at 21 01 44

破壊的変更をもたらしますか / Does this introduce a breaking change?

[x] Yes
[ ] No

Pull Request の種類 / Pull Request type

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

検証方法 / How to test

コードの入手 / Get the code

git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
dotnet restore

コードの検証 / Test the code


確認事項 / What to check

その他 / Other information

デバッグ用にしては変更が大きすぎるので、宛先はfeature/cocoa2ではなく別途デバッグ用のブランチを作ってそちらに振り向けた方がいいかもしれない(COCOAアプリについてはSubRegionsを送ってもサーバー側が無視すればいいだけなので、feature/cocoa2に反映しても良いかも)。

{region} {subregion} のプレースホルダーを [region] [subregion] に変更している。
これはiOSで { } を入力するのがとても難しいことが分かったため。

@keiji keiji self-assigned this Dec 5, 2021
@keiji keiji added the COCOA2 label Dec 5, 2021
@keiji
Copy link
Collaborator Author

keiji commented Dec 5, 2021

サーバー側、パターンでテストして一応ちゃんと動いているけど、これはテストケース作らないと駄目なやつ。

@keiji keiji marked this pull request as ready for review December 21, 2021 23:55
@keiji
Copy link
Collaborator Author

keiji commented Dec 21, 2021

テストケースは V3DiagnosisApiTest で、定期実行系のテストについては変更が大きくなるので別途Issue( #564 #565 )で対応する。

@keiji keiji marked this pull request as draft January 4, 2022 16:38
@keiji
Copy link
Collaborator Author

keiji commented Jan 4, 2022

@kazuhiro4949 一旦、COCOA2のスコープから外します。
もしレビューを始めているようなら、終わらせて貰っても大丈夫です(そのときは教えてください)。

@keiji keiji removed the request for review from kazuhiro4949 January 5, 2022 14:30
@keiji keiji changed the base branch from feature/cocoa2 to develop January 13, 2022 03:11
@keiji keiji removed the COCOA2 label Jan 13, 2022
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.

[Server] デバッグ用にSubRegionパラメーターを導入する
1 participant