From: | Pradip Parkale <pradip(dot)parkale(at)enterprisedb(dot)com> |
---|---|
To: | Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [pgAdmin][RM4979]: Configuration files for data sources or similar. |
Date: | 2020-09-03 06:55:14 |
Message-ID: | CAJ9T6StLHKviij7ZPXC36amgYy=tsMvKqW48oTxzUbkUaFx3gQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi Akshay,
Please find the updated patch.
On Wed, Sep 2, 2020 at 1:48 PM Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
wrote:
> Hi Pradip
>
> Please fix SonarQube issues that introduce with this patch.
>
> On Tue, Sep 1, 2020 at 4:47 PM Pradip Parkale <
> pradip(dot)parkale(at)enterprisedb(dot)com> wrote:
>
>> Hi Akshay,
>>
>> Please find the updated patch.
>>
>> On Tue, Aug 25, 2020 at 5:53 PM Akshay Joshi <
>> akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
>>
>>> Hi Pradip
>>>
>>> Following are the GUI review comments:
>>> *Testing Scenario*: I have three users U1 and U2 are Admin and U3 is a
>>> normal user. Login from U1 first time and pgAdmin4 discover all the local
>>> servers. Steps that I perform:
>>>
>>> - Shared one server.
>>> - Create a group 'Test' and move one server to that group.
>>> - Share the server of the 'Test' group as well.
>>>
>>> [image: Screenshot 2020-08-25 at 4.38.32 PM.png]
>>> *Issues:*
>>>
>>> - When login using U2 or U3 PostgreSQL 9.2 and 10 shown twice which
>>> should not. Refer below screenshot
>>> [image: Screenshot 2020-08-25 at 4.30.22 PM.png]
>>>
>>> Fixed. If the server is auto-discovered and U1 shared the
>> auto-discovered server(PostgreSQL 9.2 in the above example) and then log in
>> using U2, in this case, shared server(PostgreSQL 9.2) will not visible to
>> U2 as the same server will be auto-discovered for U2.
>>
>>>
>>> -
>>> - When expanding the server or clicking on the 'Connect Server' menu
>>> of the shared server, it opens the properties dialog.
>>>
>>> This is expected. Properties dialogue will open the first time for a
>> shared server where the user has to enter server details. This will happen
>> for the user who is not the owner of the server.
>>
>>>
>>> - Try to update the IPAddress, Port it is not updating for the
>>> shared server. Check other properties too.
>>>
>>> Fixed.
>>
>>>
>>> -
>>> - When we set "Hide shared server?" setting to true/false. A
>>> complete browser tree should be refreshed. With the current implementation,
>>> each group needs to be refreshed individually and the server group still
>>> visible while refreshing the complete page Browser Tree looks perfect.
>>>
>>> Fixed.
>>
>>>
>>> - When there is only one server in a group and we expanded the group
>>> 'Connect to server' dialog appears which should not. (*This may be an old
>>> issue)
>>>
>>> This is expected and its an old behavior.
>>
>>>
>>> -
>>> - Schema Diff shared servers are not visible inside the appropriate
>>> group in the source and target server list.
>>>
>>> Fixed
>>
>>>
>>> - Documentation changes required (Create a new Redmine).
>>>
>>> Done.
>>
>>>
>>> -
>>> - SonarQube shows 1 new Bug and 4 new code smell. (Fixed the Bug,
>>> code smell can be fixed later)
>>>
>>> Fixed.
>>
>>>
>>> - 'pgAdmin4_test_non_admin_credentials' should be added in '
>>> test_config.json.in' file.
>>>
>>> Fixed.
>>
>>>
>>> - For Desktop mode shared server test cases should be skipped. It is
>>> failing at the moment.
>>>
>>> Fixed.
>>
>>> Code review still remains :)
>>>
>>> On Fri, Aug 21, 2020 at 6:00 PM Pradip Parkale <
>>> pradip(dot)parkale(at)enterprisedb(dot)com> wrote:
>>>
>>>> Hi Akshay,
>>>> Please find the updated patch.
>>>>
>>>> On Fri, Aug 21, 2020 at 1:55 PM Akshay Joshi <
>>>> akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
>>>>
>>>>> Hi Pradip
>>>>>
>>>>> The patch is not applied, can you please rebase and send it again.
>>>>>
>>>>> On Thu, Aug 20, 2020 at 3:58 PM Pradip Parkale <
>>>>> pradip(dot)parkale(at)enterprisedb(dot)com> wrote:
>>>>>
>>>>>> Hi Hackers,
>>>>>>
>>>>>> Please find the attached patch for the shared server implementation.
>>>>>>
>>>>>> Few key points:
>>>>>>
>>>>>> 1. The admin who is the owner of the server user can share the
>>>>>> server with other users.
>>>>>> 2. This option will be available only for admin users.
>>>>>> 3. If the user doesn't want to see the shared server then the
>>>>>> option to hide the shared server is available in preferences.
>>>>>> 4. The user who is not the owner of the server, can't delete the
>>>>>> shared server and server group.
>>>>>> 5. This option is only available in server mode.
>>>>>>
>>>>>>
>>>>>> --
>>>>>> Thanks & Regards,
>>>>>> Pradip Parkale
>>>>>> Software Engineer | EnterpriseDB Corporation
>>>>>>
>>>>>
>>>>>
>>>>> --
>>>>> *Thanks & Regards*
>>>>> *Akshay Joshi*
>>>>> *pgAdmin Hacker | Sr. Software Architect*
>>>>> *EDB Postgres <http://edbpostgres.com>*
>>>>>
>>>>> *Mobile: +91 976-788-8246*
>>>>>
>>>>
>>>>
>>>> --
>>>> Thanks & Regards,
>>>> Pradip Parkale
>>>> Software Engineer | EnterpriseDB Corporation
>>>>
>>>
>>>
>>> --
>>> *Thanks & Regards*
>>> *Akshay Joshi*
>>> *pgAdmin Hacker | Sr. Software Architect*
>>> *EDB Postgres <http://edbpostgres.com>*
>>>
>>> *Mobile: +91 976-788-8246*
>>>
>>
>>
>> --
>> Thanks & Regards,
>> Pradip Parkale
>> Software Engineer | EnterpriseDB Corporation
>>
>
>
> --
> *Thanks & Regards*
> *Akshay Joshi*
> *pgAdmin Hacker | Sr. Software Architect*
> *EDB Postgres <http://edbpostgres.com>*
>
> *Mobile: +91 976-788-8246*
>
--
Thanks & Regards,
Pradip Parkale
Software Engineer | EnterpriseDB Corporation
Attachment | Content-Type | Size |
---|---|---|
RM4979_v4.patch | application/octet-stream | 142.1 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Akshay Joshi | 2020-09-03 07:34:10 | pgAdmin 4 commit: Improve code coverage and API test cases for Tables. |
Previous Message | Yogesh Mahajan | 2020-09-03 06:41:39 | Housekeeping #5774 - [Code Coverage] Improve API test cases for Table node |