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

From Rahul Shirsat
Subject Re: [Patch][pgAdmin] RM4203 Rename Database by another user
Date
Msg-id CAKtn9dMP3DzvRqoLO1iYTATTftjF2Bw8YXLY5_xpDkbCjtUNwg@mail.gmail.com
Whole thread Raw
In response to Re: [Patch][pgAdmin] RM4203 Rename Database by another user  (Akshay Joshi <akshay.joshi@enterprisedb.com>)
Responses Re: [Patch][pgAdmin] RM4203 Rename Database by another user  (Akshay Joshi <akshay.joshi@enterprisedb.com>)
List pgadmin-hackers
Hi Akshay,

Please find the updated patch below for your reference. I have added an extra check for verifying the database rename.
Sorry for the inconvenience caused.

On Tue, Jun 1, 2021 at 8:50 PM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
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



--
Rahul Shirsat
Senior Software Engineer | EnterpriseDB Corporation.
Attachment

pgadmin-hackers by date:

Previous
From: Dave Page
Date:
Subject: Re: [pgAdmin][RM-2341]: Add menu option for starting PSQL
Next
From: Nikhil Mohite
Date:
Subject: [pgAdmin][RM-6404]: Separate settings for query tool tab title and connection info