-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Fix devise mapping #540
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
Fix devise mapping #540
Conversation
This fails for me:
|
@ivawzh line 13? Here is nothing at this line. |
Yesterday I couldn't install this gem with Rails 5. So I manually merged the change, and then the error occurred. However, when I try it tonight, this pull is installed without any conflicts. And the gem is working as expected. So 👍 . Thank you @merqlove. Hope this can be merged ASAP |
LGTM .. Thanks @merqlove ! |
Hi @merqlove I am still having the This issue happens when I open the url |
Hi, resource_class param still required. This exists before my fix.
|
@merqlove the value of is there anywhere I should set up the Thanks! |
@anklos It setup automatically. You can check routes here: |
@merqlove yeah i followed the setup. i put a break point above that line and restart the server. However server does not stop there before it crashes. |
@anklos It means that you never call it. Can you post output of |
rake routes:
I was calling GET I just tried to call GET I am a bit confused now, which route I should use? |
if I do GET |
@anklos, hmm, looks weird. |
yes I did this exactly |
@anklos good. https://github.com/lynndylanhurley/devise_token_auth/blob/ad8c60695bcefbf9a21ea35d8b07059380fcb9bf/test/controllers/devise_token_auth/omniauth_callbacks_controller_test.rb For example, how i test it: it 'should signup with twitter and e-mail' do
get_via_redirect '/api/v1/auth/twitter', email: '[email protected]', auth_origin_url: host
expect(response).to be_success
users = User.where(email: '[email protected]')
check_data(users)
end |
@merqlove you are a legend!!!! Thanks so much!! |
Fix for #362