From: | Joao De Almeida Pereira <jdealmeidapereira(at)pivotal(dot)io> |
---|---|
To: | Khushboo Vashi <khushboo(dot)vashi(at)enterprisedb(dot)com> |
Cc: | Dave Page <dpage(at)pgadmin(dot)org>, vhenry(at)pivotal(dot)io, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: pgAdmin 4 commit: Refactor server dialogue validation for better unit t |
Date: | 2018-03-14 16:10:53 |
Message-ID: | CAE+jja==RKhCxxdRmrZrWtg=X90gQ7ode=bCGRid5N7j0obUqw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi Khushboo,
Good catch, attached you can find the fix for the problem.
Thanks
Victoria & Joao
On Wed, Mar 14, 2018 at 10:37 AM Khushboo Vashi <
khushboo(dot)vashi(at)enterprisedb(dot)com> wrote:
> Hi Joao/Victoria,
>
> I am getting an error while editing the server properties. Please refer
> the attached screen-shot for the same.
>
> Thanks,
> Khushboo
>
> On Wed, Mar 14, 2018 at 12:17 AM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>
>> Refactor server dialogue validation for better unit testing.
>>
>> Victoria & Joao @ Pivotal.
>>
>> Branch
>> ------
>> master
>>
>> Details
>> -------
>>
>> https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=6b03cb78af607b04c75de44d635bf201babb4c5a
>> Author: Victoria Henry <vhenry(at)pivotal(dot)io>
>>
>> Modified Files
>> --------------
>> web/package.json | 3 +-
>> .../server_groups/servers/static/js/server.js | 113
>> ++-------------------
>> web/pgadmin/static/bundle/browser.js | 2 +-
>> web/pgadmin/static/js/browser/index.js | 10 ++
>> .../static/js/browser/server_groups/index.js | 10 ++
>> .../server_groups/servers/databases/index.js | 10 ++
>> .../js/browser/server_groups/servers/index.js | 11 ++
>> .../server_groups/servers/model_validation.js | 104
>> +++++++++++++++++++
>> .../server_groups/servers/model_validation_spec.js | 101
>> ++++++++++++++++++
>> web/yarn.lock | 46 ++++++++-
>> 10 files changed, 299 insertions(+), 111 deletions(-)
>>
>>
>
Attachment | Content-Type | Size |
---|---|---|
fix-trim-function.diff | text/x-patch | 1.3 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Rahul Soshte | 2018-03-14 16:32:15 | Rahul Soshte GSOC Draft Proposal |
Previous Message | Khushboo Vashi | 2018-03-14 14:55:37 | Re: [pgadmin4][Patch]: RM #3122 - Backup not working on certificate (SSL) protected servers |