Skip to content

Commit

Permalink
use appropriate login post url on mobile
Browse files Browse the repository at this point in the history
fixes CNVS-39869

test plan:
 * configure canvas auth
 * login on a mobile device
 * it should work
 * configure ldap auth only
 * login on a mobile device
 * it should work

Change-Id: I2dfe9eec22dbe2f74ead6b95ba871466ce99c6af
Reviewed-on: https://gerrit.instructure.com/129107
Reviewed-by: Brent Burgoyne <[email protected]>
Tested-by: Jenkins
QA-Review: Tucker McKnight <[email protected]>
Product-Review: Cody Cutrer <[email protected]>
  • Loading branch information
ccutrer authored and simonista committed Oct 11, 2017
1 parent 85b4cba commit de38010
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion app/views/login/canvas/mobile_login.html.erb
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@
<%= t 'Canvas by Instructure' %>
</span>
</div>
<%= form_for :pseudonym_session, :url => login_path, :html => {:id => "login_form", :class => "front face", :novalidate => 'novalidate', :autocomplete => "off" } do |f| %>
<%= form_for :pseudonym_session, :url => url_for({ action: :create }.merge(params.slice(:id).to_unsafe_h)), :html => {:id => "login_form", :class => "front face", :novalidate => 'novalidate', :autocomplete => "off" } do |f| %>

<div class="ic-Form-control">
<input
Expand Down

0 comments on commit de38010

Please sign in to comment.