saito.k
|
d5e5e59f8c
|
Merged PR 128: API実装（ソート条件変更）
## 概要
[Task1835: API実装（ソート条件変更）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1835)
- ソート条件変更APIを実装
- トークンからB2CのIDを取得→userテーブルからユーザー情報を取得
- ユーザーIDでソート条件テーブルを検索→レコードを更新
- ソート条件テーブルにレコードを作成する
- アカウント作成時の処理にソート条件レコードを作成する処理を追加
- ユーザー追加時にも処理を追加
- テスト修正
## レビューポイント
- APIの引数をチェックする関数をControllerに配置してもよいか
- ソート条件のレコードを作成するタイミングに漏れはないか
- 実装漏れはないか
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-06-06 08:20:21 +00:00 |
|
湯本 開
|
1ced5ef66d
|
Merged PR 123: API実装（I/F実装）
## 概要
[Task1836: API実装（I/F実装）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1836)
- タスクのソート条件更新APIのI/Fを実装
- タスク一覧APIのI/Fを修正
- TODO/XXXですぐ対処可能、または単純に消し忘れているものを対処
## レビューポイント
- API I/Fの修正は妥当な内容であるか
- 特にfilterはカンマ区切りでいいか、指定したものを除外という形式でいいか等
- TODO/XXXの対処は妥当な対処であるか
## 動作確認状況
- ローカルでswagger表示されることを確認
|
2023-06-02 06:12:35 +00:00 |
|
saito.k
|
0a970e814f
|
Merged PR 76: API実装（I/F）
## 概要
[Task1576: API実装（I/F）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1576)
- sas発行APIのIFを実装
- アップロード用のSAS発行API
- GET /blob/uploadSas
- ダウンロード用のSAS発行API
- GET /blob/downloadSas
- notification関連のフォーマット修正
- notification関連でlintエラーが出ていた箇所を修正
- openapi.jsonを生成するコマンドを使用できるように修正
## レビューポイント
- レスポンスとして返却する内容に不足は無いか
- URIはこれで良さそうか
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-04-14 01:08:32 +00:00 |
|
saito.k
|
ee2e8dbd5d
|
Merged PR 64: デバイス登録API実装
## 概要
[Task1571: デバイス登録API実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1571)
- NotificationHubへのデバイス登録APIを実装
- NotificationHubへの接続
- 登録処理
## レビューポイント
- ディレクトリ構成に問題は無いか
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルでNotificationHubに接続できるところまで確認
## 補足
- 返却するエラーやサーバー側で設定する部分の登録情報については仮です。
- 「プロダクト バックログ項目 1389: デスクトップアプリ/モバイルアプリに様々な通知をしたい」で実装・設計する想定
|
2023-04-10 00:41:32 +00:00 |
|
saito.k
|
382bb4e25f
|
Merged PR 62: デバイス登録APIのIF実装
## 概要
[Task1570: デバイス登録APIのIF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1570)
- NotificationHubへのデバイス登録を行うAPIのIFを実装
- MSへの問い合わせで想定していた引数で概ね問題ないとの回答
- OpenAPI生成関数が動作しなくなっていたので削除
- 生成するためにはServiceごとにモックを作成しないといけないため修正コストが高すぎると判断
## レビューポイント
- Requestのプロパティについて認識違いは無いか
- 外部連携用のAPIも他のAPIと同様の場所に配置したが問題ないか
- 返却するステータスコードはこれで良いか？
- 登録の成功は201?
- エラー時のステータスコードの種類に過不足は無いか
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- OpenAPIを生成して確認
## 補足
|
2023-04-07 01:05:28 +00:00 |
|