Re: [Patch][pgAdmin] RM4203 Rename Database by another user - Mailing list pgadmin-hackers

From Akshay Joshi
Subject Re: [Patch][pgAdmin] RM4203 Rename Database by another user
Date
Msg-id CANxoLDeUFmDmZwap03H+UT+F6JAPkjZdNA8j43Qp0Dz7U04bSg@mail.gmail.com
Whole thread Raw
In response to Re: [Patch][pgAdmin] RM4203 Rename Database by another user  (Rahul Shirsat <rahul.shirsat@enterprisedb.com>)
Responses Re: [Patch][pgAdmin] RM4203 Rename Database by another user
List pgadmin-hackers
Thanks, the patch applied.

On Sun, May 30, 2021 at 12:03 AM Rahul Shirsat <rahul.shirsat@enterprisedb.com> wrote:
Hi Akshay,

On Thu, May 27, 2021 at 12:19 PM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi Rahul

Following are the review comments:
  • Why you have changed the order of the toolbar button on the browser tree? It should not.
               Changes were related to the wcDocker issue showing reverse order buttons, so had to change in the pgAdmin app to reflect it correctly.
  • sqleditor.js (check_db_name_change function)
    • var declaration should be comma-separated instead of writing var, again and again.
                          Done. 
    • Add proper comments.
                          This too is taken care of. 
    • Remove dots from the string "Database moved/renamed.."
                          Done. 
    • "Please note that the database you are working on has been moved or renamed to ${data.actual_db_name}. Refresh databases in browser panel or click OK to continue.." should be changed to "Current database has been moved or renamed to ${data.actual_db_name}. Click on the OK button to refresh the database name." 
                          Done. 
                           Also have done some code refactoring.


On Wed, May 26, 2021 at 2:07 PM Rahul Shirsat <rahul.shirsat@enterprisedb.com> wrote:
Hi Hackers,

Please find the attached patch which resolves the issue of renaming the database by another user.

A minor issue of navigation button alignment is also fixed in this patch.

--
Rahul Shirsat
Senior Software Engineer | EnterpriseDB Corporation.


--
Thanks & Regards
Akshay Joshi
pgAdmin Hacker | Principal Software Architect
EDB Postgres
Mobile: +91 976-788-8246



--
Rahul Shirsat
Senior Software Engineer | EnterpriseDB Corporation.


--
Thanks & Regards
Akshay Joshi
pgAdmin Hacker | Principal Software Architect
EDB Postgres
Mobile: +91 976-788-8246

pgadmin-hackers by date:

Previous
From: Akshay Joshi
Date:
Subject: Re: [Patch] RM6478 Duplicates on SQL tab for table with more than one partition
Next
From: Rahul Shirsat
Date:
Subject: [patch][pgAdmin] RM1591 [Web Based] Grant Wizard should also add under package node