Skip to content

E
edx-platform
Commit a1104ceb by AlasdairSwan
Options

Merge pull request #6464 from edx/alasdair/logistration-history-query-params-fix 

Updated history.push to include any query params
parents 72bdd48f 1d928379
Showing with 3 additions and 2 deletions
lms/static/js/student_account/views/AccessView.js
...@@ -172,7 +172,8 @@ var edx = edx || {}; ...@@ -172,7 +172,8 @@ var edx = edx || {};
toggleForm: function( e ) { toggleForm: function( e ) {
var type = $(e.currentTarget).data('type'), var type = $(e.currentTarget).data('type'),
$form = $('#' + type + '-form'), $form = $('#' + type + '-form'),
$anchor = $('#' + type + '-anchor'); $anchor = $('#' + type + '-anchor'),
queryParams = '?' + url('?');
e.preventDefault(); e.preventDefault();
...@@ -190,7 +191,7 @@ var edx = edx || {}; ...@@ -190,7 +191,7 @@ var edx = edx || {};
this.element.scrollTop( $anchor ); this.element.scrollTop( $anchor );
// Update url without reloading page // Update url without reloading page
History.pushState( null, document.title, '/account/' + type + '/' ); History.pushState( null, document.title, '/account/' + type + queryParams );
analytics.page( 'login_and_registration', type ); analytics.page( 'login_and_registration', type );
}, },
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment