予算管理ページのapi定義 #900
予算管理ページのapi定義 #900
Conversation
|
リンクだけじゃなくて該当箇所のスクショもあったら凄く嬉しいです！！ |
openapi/openapi.yaml
Outdated
| application/json: | ||
| schema: | ||
| $ref: "#/components/schemas/division" | ||
| required: false |
There was a problem hiding this comment.
| required: false | |
| required: true |
更新だしtrueでも良さそう
| @@ -767,6 +831,134 @@ paths: | |||
| "200": | |||
| description: yearで指定されたexpensesを取得 | |||
| content: {} | |||
| /festival_items: | |||
There was a problem hiding this comment.
PRはfinacial_itemsと書かれてましたがfestival_itemsが正しい方ですよね.。
正直どっちでもいいとは思いますけど。
openapi/openapi.yaml
Outdated
| responses: | ||
| "200": | ||
| description: 作成されたdivisionが返ってくる | ||
| content: {} |
There was a problem hiding this comment.
| content: {} | |
| content: | |
| application/json: | |
| schema: | |
| $ref: "#/components/schemas/division" |
type: objectでもよいかも？
仮初めだけどcontentは書きたいマン
There was a problem hiding this comment.
idも含めて返したいと思い、getで配列で返しているオブジェクトを返すようにしました
response修正
| festivalItem: | ||
| required: | ||
| - name | ||
| - divisionId | ||
| - amount | ||
| type: object | ||
| properties: | ||
| name: | ||
| type: string | ||
| example: 農ポリ | ||
| divisionId: | ||
| type: integer | ||
| example: 1 | ||
| amount: | ||
| type: integer | ||
| example: 10000 | ||
| memo: | ||
| type: string | ||
| example: "" |
There was a problem hiding this comment.
festtival_itemとitem_budgetが統合されてる感じがしますけどがpostとputの際どうする予定とかあります？
普通にテーブル統合してpostするだけ？
There was a problem hiding this comment.
パラメータ受け取って適宜バックエンドでトランザクション貼って複数のテーブルにインサートするイメージです
| @@ -1927,6 +2255,18 @@ components: | |||
| remark: | |||
| type: string | |||
| example: "" | |||
| total: | |||
There was a problem hiding this comment.
このレビューに関係ないので読み飛ばし可です。
これ別個でAPI生やして管理します？それとも各APIに組み込ますか？
totalの使い方まだピンと来てないマンです。
There was a problem hiding this comment.
api内で集計して、totalに整形して返すイメージです。
局、部門、物品で合計が必要になるので、レスポンスで返す共通の型を定義した感じです。
apiを生やす？ってのがピンときていないです
There was a problem hiding this comment.
totalって何回も算出されそうだからtotal用のAPI作ってもいいのかなって思ってだったんですけど、よく考えたらべつにAPI作る必要なさそうですね
|
あと私のgo.mod以下みたいになったんですけどなります？ |
|
自分も消えました |
対応Issue
resolve #901
概要
/financial_records/divisions/festival_items画面スクリーンショット等
URLテスト項目
備考
apiは実装していないです。
figma
ER図