From: | Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com> |
---|---|
To: | Rahul Shirsat <rahul(dot)shirsat(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [Patch][pgAdmin] RM4203 Rename Database by another user |
Date: | 2021-05-27 06:49:31 |
Message-ID: | CANxoLDdvQGWZT7DbmMmssLxy3biY+--KeQx40AkTXzmp7kizLg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi Rahul
Following are the review comments:
- Why you have changed the order of the toolbar button on the browser
tree? It should not.
- *sqleditor.js (check_db_name_change function)*
- var declaration should be comma-separated instead of writing var,
again and again.
- Add proper comments.
- Remove dots from the string "Database moved/renamed.."
- "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."
On Wed, May 26, 2021 at 2:07 PM Rahul Shirsat <
rahul(dot)shirsat(at)enterprisedb(dot)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 <http://edbpostgres.com>*
*Mobile: +91 976-788-8246*
From | Date | Subject | |
---|---|---|---|
Next Message | Akshay Joshi | 2021-05-29 07:26:05 | pgAdmin 4 commit: Fixes SonarQube issues. |
Previous Message | Akshay Joshi | 2021-05-27 05:32:52 | Re: [pgAdmin][Patch] - Feature Test failures fix |