Re: [pgAdmin][RM4979]: Configuration files for data sources or similar.

Hi Akshay,

Please find the updated patch.

On Wed, Sep 2, 2020 at 1:48 PM Akshay Joshi <akshay.joshi@enterprisedb.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.parkale@enterprisedb.com> wrote:

Hi Akshay,

Please find the updated patch.

On Tue, Aug 25, 2020 at 5:53 PM Akshay Joshi <akshay.joshi@enterprisedb.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.
  

            

Issues:

• When login using U2 or U3 PostgreSQL 9.2 and 10 shown twice which should not. Refer below screenshot
   

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.parkale@enterprisedb.com> wrote:

Hi Akshay,

Please find the updated patch.

On Fri, Aug 21, 2020 at 1:55 PM Akshay Joshi <akshay.joshi@enterprisedb.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.parkale@enterprisedb.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

Mobile: +91 976-788-8246

--

Thanks & Regards,

Pradip Parkale
Software Engineer | EnterpriseDB Corporation

--

Thanks & Regards

Akshay Joshi

pgAdmin Hacker | Sr. Software Architect

EDB Postgres

Mobile: +91 976-788-8246

--

Thanks & Regards,

Pradip Parkale
Software Engineer | EnterpriseDB Corporation

--

Thanks & Regards

Akshay Joshi

pgAdmin Hacker | Sr. Software Architect

EDB Postgres

Mobile: +91 976-788-8246

--

Thanks & Regards,

Pradip Parkale
Software Engineer | EnterpriseDB Corporation