Merge pull request #16 from zach-chai/login_provider

login with provider in personal room

app/controllers/landing_controller.rb

@@ -42,6 +42,7 @@ class LandingController < ApplicationController
     @user = User.find_by(username: @meeting_token)
     if @user.nil?
       redirect_to root_path
+      return
     end
     render :action => 'room'
   end

app/controllers/users_controller.rb

@@ -21,7 +21,7 @@ class UsersController < ApplicationController
     def set_user
       @user = User.find(params[:id])
       if @user.username
-        raise ActionController::RoutingError.new('Not Found')
+        render 'errors/error'
       end
     end
 

app/views/landing/room.html.erb

@@ -8,7 +8,7 @@
           <span class="user">Hello <%= current_user.name %></span>
           <%= link_to "Logout", user_logout_url() %>
         <% else %>
-          <%= link_to "Login", "/auth/twitter" %>
+          <%= link_to "Login", "/auth/#{@user.provider}" %>
         <% end %>
       </div>
     </div>
@@ -21,4 +21,3 @@
   </div>
 
 </div>
-

test/fixtures/users.yml

@@ -1,13 +1,13 @@
 # Read about fixtures at http://api.rubyonrails.org/classes/ActiveRecord/FixtureSet.html
 
 one:
-  provider: Twitter
+  provider: twitter
   uid: <%= SecureRandom.hex(10) %>
   name: User 1
   username: user1
 
 two:
-  provider: TWitter
+  provider: google
   uid: <%= SecureRandom.hex(10) %>
   name: User 2
   username: user2