Refresh lists after adding a account using New User Experience #1176
Conversation
| @@ -57,7 +57,7 @@ var display_final_nux_step = function(res) { | |||
| } | |||
| ); | |||
| } else { | |||
| Hm_Utils.redirect(); | |||
| Hm_Utils.redirect('?page=servers'); | |||
There was a problem hiding this comment.
at https://github.com/cypht-org/cypht/blob/master/modules/core/site.js#L1711 we have window.location.href that is already redirecting to the current URL of the page that is being displayed in the browser, in this case it will be www.domain.com?page= servers, even if we did not specify this parameter.
Have you had a case? I can't reproduce a one that redirects to an another page than servers.
There was a problem hiding this comment.
Hi @Shadow243,
You’re right! After updating to the latest version of the master branch, I’m no longer able to reproduce the bug. Let me test various cases to ensure it's completely fixed.
There was a problem hiding this comment.
I can not reproduce this too... I have tried several time. @henochit have you tried too so we know...
|
Once merged to master, please backport to 2.x |
Pullrequest
This fix allows the refresh of the server list after adding an account through the new user experience wizard.