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

Setup doorkeeper #72

Closed
wants to merge 8 commits into from
Closed

Conversation

iida-hayato
Copy link
Member

@nisshiee nisshiee mentioned this pull request Jul 18, 2016
@nisshiee
Copy link
Contributor

@iida-hayato APIエンドポイントを作るまでしないと「ユーザに提供するストーリー」にならないので、そこまで作りきってからマージしたいです。一方で、そこまで作りきろうと思うと、それなりに大きなdiffになるかと思います(と思って細かいPRにしてくれたのかと)

なので、このブランチの状態を our-first-api という名前のブランチでそのまま取り込み、PRも #73 に作りなおしました。今後、のAPIエンドポイント作成や、bugfixなどは、 our-first-api ブランチへのPRとして作って頂ければ順次取り込みます。で、作りきった段階で #73 をマージしようと思います。

@nisshiee
Copy link
Contributor

というわけで、修正をお願いしたい点も #73 の方に記載していくので、こちらのPRは一旦閉じます

@nisshiee nisshiee closed this Jul 18, 2016
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