Skip to content

Feature/add swagger#6

Closed
acio-o9 wants to merge 6 commits intomasterfrom
feature/add-swagger
Closed

Feature/add swagger#6
acio-o9 wants to merge 6 commits intomasterfrom
feature/add-swagger

Conversation

@acio-o9
Copy link
Copy Markdown
Owner

@acio-o9 acio-o9 commented Jun 27, 2020

追加したdocker image

api.yml

↓を参考にAPI定義を作成

もうちょっとuri, component足すかも

単語帳の1枚がCard。なんかしっくりこない 😢
そもそもバックエンドとフロントエンド、どっちに用意すべきかも悩ましい 😥

@acio-o9 acio-o9 linked an issue Jun 27, 2020 that may be closed by this pull request
@acio-o9 acio-o9 force-pushed the feature/add-swagger branch from 7334c74 to 10dbb0a Compare June 28, 2020 01:49
@acio-o9 acio-o9 force-pushed the feature/add-swagger branch from 10dbb0a to 471a87e Compare June 28, 2020 15:57
@acio-o9 acio-o9 marked this pull request as ready for review June 28, 2020 16:00
@oriore
Copy link
Copy Markdown
Collaborator

oriore commented Jun 29, 2020

バックエンド的にはAPIの定義、レスポンスの確認の用途ぐらい、
フロントエンド的にはAPIの定義とレスポンス確認の他にMockとしての用途があることを考えると
フロントエンドのコンテナと一緒に立ち上がってくれたほうが便利な可能性があるかもしれないですね

@acio-o9
Copy link
Copy Markdown
Owner Author

acio-o9 commented Jun 30, 2020

フロントエンドのブランチに依存しちゃうのが面倒かなと考えていたけど、
そもそもバックエンドとフロント両方dockerを立ち上げないといけないのは手間だよね。

Mockとしての利用頻度は高そうなので、フロント側に持って行きました 🚀
acio-o9/kbrowser_front#5

@acio-o9 acio-o9 closed this Jun 30, 2020
@acio-o9 acio-o9 deleted the feature/add-swagger branch June 30, 2020 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

APIの定義をつくる

2 participants