فهرست منبع

Merge pull request #5146 from cloudron-io/fix-password-change-regression

Fix password change regression
Lauri Ojansivu 1 سال پیش
والد
کامیت
16991498fb
2فایلهای تغییر یافته به همراه4 افزوده شده و 4 حذف شده
  1. 1 0
      client/components/users/userHeader.js
  2. 3 4
      config/accounts.js

+ 1 - 0
client/components/users/userHeader.js

@@ -248,6 +248,7 @@ Template.editProfilePopup.events({
 // XXX For some reason the useraccounts autofocus isnt working in this case.
 // XXX For some reason the useraccounts autofocus isnt working in this case.
 // See https://github.com/meteor-useraccounts/core/issues/384
 // See https://github.com/meteor-useraccounts/core/issues/384
 Template.changePasswordPopup.onRendered(function() {
 Template.changePasswordPopup.onRendered(function() {
+  $('.at-pwd-form').show();
   this.find('#at-field-current_password').focus();
   this.find('#at-field-current_password').focus();
 });
 });
 
 

+ 3 - 4
config/accounts.js

@@ -126,10 +126,9 @@ AccountsTemplates.configure({
 
 
 AccountsTemplates.configureRoute('changePwd', {
 AccountsTemplates.configureRoute('changePwd', {
   redirect() {
   redirect() {
-    // XXX We should emit a notification once we have a notification system.
-    // Currently the user has no indication that his modification has been
-    // applied.
-    Popup.back();
+    // We should go back with the popup but we don't since user feedback about the change is within the popup only.
+    // Once we have a global feedback popup mechanism we can use that here and close with the following:
+    // Popup.back();
   },
   },
 });
 });