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

Our first api #73

Open
wants to merge 24 commits into
base: master
Choose a base branch
from
Open

Our first api #73

wants to merge 24 commits into from

Conversation

nisshiee
Copy link
Contributor

@nisshiee nisshiee commented Jul 18, 2016 •
edited
Loading

#72 からこちらに移動

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

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

@nisshiee nisshiee self-assigned this Jul 18, 2016
@nisshiee nisshiee mentioned this pull request Jul 18, 2016
Closed
nisshiee
nisshiee reviewed Jul 18, 2016
View reviewed changes
db/schema.rb
@@ -48,7 +88,7 @@
t.index ["user_id"], name: "index_templates_on_user_id", using: :btree
end

create_table "users", force: :cascade, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8mb4" do |t|
create_table "users", force: :cascade, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8mb4", comment: "ユーザー情報" do |t|
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

このdiffは恐らく手動でテーブルにコメント付けてしまったために出てしまっているので、戻して欲しいです

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#81

@nisshiee
Copy link
Contributor Author

nisshiee commented Jul 18, 2016 •
edited
Loading

OAuthまわりの画面（ /oauth/applications とか）に、ログインしなくてもアクセスできてしまうので、制御して欲しいですー

@iida-hayato
Copy link
Member

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

↑このへん了解ですー

@nisshiee
Copy link
Contributor Author

↑こんな感じで、適宜、最新のmasterを取り込んでいきます

@iida-hayato
Copy link
Member

テスト用のアプリrepository
https://github.com/iida-hayato/i-nippo

@nisshiee
Copy link
Contributor Author

@iida-hayato このブランチ、どこまでやります？

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees

@nisshiee nisshiee

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@nisshiee @iida-hayato @pataiji