New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixes #8785: Improve Script field behavior in passwords #1154
Fixes #8785: Improve Script field behavior in passwords #1154
Conversation
|
||
$scope.$watch('action',updateResult); | ||
$scope.$watch('newPassword.password',updateResult); | ||
$scope.$watch('newPassword.hash',updateResult); | ||
|
||
function updateResult () { | ||
// Keep and delete, use current password as base | ||
var result = $scope.current | ||
var result = angular.copy($scope.current) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
missing ; there
Appart from the two missing ;, this is awesome |
I've been testing it, and when we fill a password value, then click "add another", the field is emptied (looks like all the password fields get back their original value in this case) |
Commit modified |
6d1f502
to
99b5b30
Compare
Awesome! |
This is good for me (except the missing ; ) |
Commit modified |
99b5b30
to
5ad4c2e
Compare
Commit modified |
5ad4c2e
to
41a18ac
Compare
OK, merging this PR |
https://www.rudder-project.org/redmine/issues/8785