Thread: Change server password [pgadmin4]
Attachment
Hi,
This patch needs a rebase. (as "Named restore point" patch is committed.)
On Wed, May 11, 2016 at 12:35 PM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:
Hi,PFA patch for changing server password.
Hi,
PFA updated patch.On Thu, May 12, 2016 at 1:27 PM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:
Hi,This patch needs a rebase. (as "Named restore point" patch is committed.)On Wed, May 11, 2016 at 12:35 PM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:Hi,PFA patch for changing server password.
Attachment
Hi Harshal
Below are my review comments :
- Facing error unexpected identifier 'change_password: function(args)' in js file.
- Please correct the spelling of password in the menu item.
- Title for alertify dialog showing "Change password for servers" it should be only "Change Password" or "Change Password for <server name>".
- Facing error "name 'decrypt' is not defined."
- Not able to change the password facing error.
On Thu, May 12, 2016 at 1:43 PM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:
Hi,PFA updated patch.On Thu, May 12, 2016 at 1:27 PM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:Hi,This patch needs a rebase. (as "Named restore point" patch is committed.)On Wed, May 11, 2016 at 12:35 PM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:Hi,PFA patch for changing server password.
--
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers
Akshay Joshi
Principal Software Engineer

Phone: +91 20-3058-9517
Mobile: +91 976-788-8246
Mobile: +91 976-788-8246
On Thu, May 12, 2016 at 4:14 PM, Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi HarshalBelow are my review comments :
- Facing error unexpected identifier 'change_password: function(args)' in js file.
- Please correct the spelling of password in the menu item.
- Title for alertify dialog showing "Change password for servers" it should be only "Change Password" or "Change Password for <server name>".
It should be "Change password for the user on the server - '<server name>'".
- Facing error "name 'decrypt' is not defined."
- Not able to change the password facing error.
On Thu, May 12, 2016 at 1:43 PM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:Hi,PFA updated patch.On Thu, May 12, 2016 at 1:27 PM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:Hi,This patch needs a rebase. (as "Named restore point" patch is committed.)On Wed, May 11, 2016 at 12:35 PM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:Hi,PFA patch for changing server password.
--
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers--Akshay JoshiPrincipal Software EngineerPhone: +91 20-3058-9517
Mobile: +91 976-788-8246
Hi,
PFA updated patch.On Thu, May 12, 2016 at 4:14 PM, Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi HarshalBelow are my review comments :
- Facing error unexpected identifier 'change_password: function(args)' in js file
Fixed
- Please correct the spelling of password in the menu item.
Fixed
- Title for alertify dialog showing "Change password for servers" it should be only "Change Password" or "Change Password for <server name>".
Fixed
- Facing error "name 'decrypt' is not defined."
Fixed
- Not able to change the password facing error.
Fixed
On Thu, May 12, 2016 at 1:43 PM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:Hi,PFA updated patch.On Thu, May 12, 2016 at 1:27 PM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:Hi,This patch needs a rebase. (as "Named restore point" patch is committed.)On Wed, May 11, 2016 at 12:35 PM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:Hi,PFA patch for changing server password.
--
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers--Akshay JoshiPrincipal Software EngineerPhone: +91 20-3058-9517
Mobile: +91 976-788-8246
Attachment
Hi,
On Thu, May 12, 2016 at 4:16 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
Fixed in latest patch. Also added user name whose password we are changing.
On Thu, May 12, 2016 at 4:14 PM, Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi HarshalBelow are my review comments :
- Facing error unexpected identifier 'change_password: function(args)' in js file.
- Please correct the spelling of password in the menu item.
- Title for alertify dialog showing "Change password for servers" it should be only "Change Password" or "Change Password for <server name>".
It should be "Change password for the user on the server - '<server name>'".
Fixed in latest patch. Also added user name whose password we are changing.
- Facing error "name 'decrypt' is not defined."
- Not able to change the password facing error.
On Thu, May 12, 2016 at 1:43 PM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:Hi,PFA updated patch.On Thu, May 12, 2016 at 1:27 PM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:Hi,This patch needs a rebase. (as "Named restore point" patch is committed.)On Wed, May 11, 2016 at 12:35 PM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:Hi,PFA patch for changing server password.
--
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers--Akshay JoshiPrincipal Software EngineerPhone: +91 20-3058-9517
Mobile: +91 976-788-8246
On Thu, May 12, 2016 at 11:45 PM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:
Hi,On Thu, May 12, 2016 at 4:16 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:On Thu, May 12, 2016 at 4:14 PM, Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi HarshalBelow are my review comments :
- Facing error unexpected identifier 'change_password: function(args)' in js file.
- Please correct the spelling of password in the menu item.
- Title for alertify dialog showing "Change password for servers" it should be only "Change Password" or "Change Password for <server name>".
It should be "Change password for the user on the server - '<server name>'".
Fixed in latest patch. Also added user name whose password we are changing.
Thanks - Committed with some GUI changes.
- Facing error "name 'decrypt' is not defined."
- Not able to change the password facing error.
On Thu, May 12, 2016 at 1:43 PM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:Hi,PFA updated patch.On Thu, May 12, 2016 at 1:27 PM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:Hi,This patch needs a rebase. (as "Named restore point" patch is committed.)On Wed, May 11, 2016 at 12:35 PM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:Hi,PFA patch for changing server password.
--
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers--Akshay JoshiPrincipal Software EngineerPhone: +91 20-3058-9517
Mobile: +91 976-788-8246
Akshay Joshi
Principal Software Engineer

Phone: +91 20-3058-9517
Mobile: +91 976-788-8246
Mobile: +91 976-788-8246