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
Try Vue.js #14
Try Vue.js #14
Conversation
No change the behavior from the outside
Because only property is changed
@kamontia please review and merge. |
app.rb
Outdated
result = request.env['omniauth.auth'] | ||
@twitter.access_token = result.credentials.token | ||
@twitter.access_token_secret = result.credentials.secret | ||
get '/favorite' do |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think that the pass should be renamed /api/v1.0/favorite
because this URL is not used with view rendering.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
addressed 58c6ebc
app.rb
Outdated
session[:uid] = env['omniauth.auth']['uid'] | ||
session[:twitter_oauth] = env['omniauth.auth'][:credentials] | ||
|
||
redirect to('/') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since authentication has been completed at this point, i suppose that it should be redirected to /top
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
addressed 844f9eb
@kamontia |
Hello @chaspy , Thank you for your effort. |
Please review and merge first #13