From: | Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com> |
---|---|
To: | Khushboo Vashi <khushboo(dot)vashi(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [pgAdmin4][Patch]: RM #3511 Passwords not stored for ssh tunnel connections |
Date: | 2018-08-06 10:08:27 |
Message-ID: | CANxoLDe-arrpzoA3VFX=vCTbEzxhdGX6vyC8pPrGW1tz6NxZgA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Thanks Khushboo,
I'll fix those and commit the code.
On Mon, Aug 6, 2018 at 3:33 PM, Khushboo Vashi <
khushboo(dot)vashi(at)enterprisedb(dot)com> wrote:
> Hi Akshay,
>
> The functionality works properly.
>
> Some minor comments as below:
> - Migration file should contain the purpose of the file instead of empty
> message string.
> - Please remove unwanted imports from the migration file.
> - The success message should be 'The saved password cleared successfully.'
> or 'Cleared the saved password successfully.'
>
> Thanks,
> Khushboo
>
> On Fri, Aug 3, 2018 at 7:16 PM, Akshay Joshi <
> akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
>
>> Hi Hackers,
>>
>> As per discussion I have changed the menu string from "Reset SSH Tunnel
>> Password" to "Clear SSH Tunnel Password". Apart from that I have updated
>> the documentation also fixed size of some images for the documentation.
>>
>> Attached is the latest patch. Please review it.
>>
>> On Wed, Aug 1, 2018 at 2:50 PM, Akshay Joshi <
>> akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
>>
>>> Hi Hackers,
>>>
>>> Attached is the patch to fix RM #3511 Passwords not stored for ssh
>>> tunnel connections. Please review it.
>>>
>>> --
>>> *Akshay Joshi*
>>>
>>> *Sr. Software Architect *
>>>
>>>
>>>
>>> *Phone: +91 20-3058-9517Mobile: +91 976-788-8246*
>>>
>>
>>
>>
>> --
>> *Akshay Joshi*
>>
>> *Sr. Software Architect *
>>
>>
>>
>> *Phone: +91 20-3058-9517Mobile: +91 976-788-8246*
>>
>
>
--
*Akshay Joshi*
*Sr. Software Architect *
*Phone: +91 20-3058-9517Mobile: +91 976-788-8246*
From | Date | Subject | |
---|---|---|---|
Next Message | Akshay Joshi | 2018-08-06 10:27:00 | pgAdmin 4 commit: Add support to save and clear SSH Tunnel password. Fi |
Previous Message | Akshay Joshi | 2018-08-06 10:05:13 | pgAdmin 4 commit: pgAdmin4 should work with python 3.7, Fixes #3458 |