Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add devise and facebook login #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

telsaiori
Copy link
Collaborator

facebook登入

Copy link
Owner

@kevinluo201 kevinluo201 left a comment

Choose a reason for hiding this comment

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

omniauth.rb 有嗎?

gem 'omniauth-facebook'
gem 'cancancan', '~> 1.10'
# Use sqlite3 as the database for Active Record
gem 'sqlite3'
Copy link
Owner

Choose a reason for hiding this comment

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

要用sqlite?

Copy link
Owner

Choose a reason for hiding this comment

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

新加的gem加到檔案後面 但在develop那個gruop之前喔

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

因為不是說只是先試一下,所以我就沒特別改要用哪一種資料庫

@@ -38,7 +43,6 @@ gem 'jbuilder', '~> 2.5'
group :development, :test do
# Call 'byebug' anywhere in the code to stop execution and get a debugger console
gem 'byebug', platform: :mri
Copy link
Owner

Choose a reason for hiding this comment

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

我的awesome_print啊~

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

因為你rails new之前我就已經先用了,所以你加的東西我那邊不會有

# Configure the e-mail address which will be shown in Devise::Mailer,
# note that it will be overwritten if you use your own mailer class
# with default "from" parameter.
config.mailer_sender = '[email protected]'
Copy link
Owner

Choose a reason for hiding this comment

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

EMAIL尚未完成喔

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

還沒~但結果我們要用什麼寄信?

@@ -0,0 +1,7 @@
# Read about fixtures at http://api.rubyonrails.org/classes/ActiveRecord/FixtureSet.html
Copy link
Owner

Choose a reason for hiding this comment

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

我忘記說...我不太用fixture
之後會說用 machinist 和 faker來做test data

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

沒關係,這他自動生的東西而已

@telsaiori
Copy link
Collaborator Author

telsaiori commented Apr 21, 2017

omniauth.rb 有嗎?
沒有
fb app_id之類的在devise.rb
然後我先用我自己的id試

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.

2 participants