public inbox for [email protected]
help / color / mirror / Atom feedFrom: Akshay Joshi <[email protected]>
To: Rahul Shirsat <[email protected]>
Cc: pgadmin-hackers <[email protected]>
Subject: Re: [Patch][pgAdmin] RM4203 Rename Database by another user
Date: Mon, 7 Jun 2021 22:10:00 +0530
Message-ID: <CANxoLDf5+9evQxBbV56SdtR7sFT_JXTbBiiKu+bmJ0m=Vni4VQ@mail.gmail.com> (raw)
In-Reply-To: <CAKtn9dMP3DzvRqoLO1iYTATTftjF2Bw8YXLY5_xpDkbCjtUNwg@mail.gmail.com>
References: <CAKtn9dPZQWV8aT+mW5DRU3dZYwKyKyanGST7-juuEdV4Dhp1AA@mail.gmail.com>
<CANxoLDdvQGWZT7DbmMmssLxy3biY+--KeQx40AkTXzmp7kizLg@mail.gmail.com>
<CAKtn9dOyM2ptVoAha3DhtbPzmeAEzZwbTF+amg_i3m2tYUs3wg@mail.gmail.com>
<CANxoLDeUFmDmZwap03H+UT+F6JAPkjZdNA8j43Qp0Dz7U04bSg@mail.gmail.com>
<CAKtn9dMP3DzvRqoLO1iYTATTftjF2Bw8YXLY5_xpDkbCjtUNwg@mail.gmail.com>
Thanks, the patch applied.
On Mon, Jun 7, 2021 at 5:08 PM Rahul Shirsat <[email protected]>
wrote:
> 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 <[email protected]>
> wrote:
>
>> Thanks, the patch applied.
>>
>> On Sun, May 30, 2021 at 12:03 AM Rahul Shirsat <
>> [email protected]> wrote:
>>
>>> Hi Akshay,
>>>
>>> On Thu, May 27, 2021 at 12:19 PM Akshay Joshi <
>>> [email protected]> 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 <
>>>> [email protected]> 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*
>>>>
>>>
>>>
>>> --
>>> *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*
>>
>
>
> --
> *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*
view thread (6+ messages)
reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: [email protected]
Cc: [email protected], [email protected]
Subject: Re: [Patch][pgAdmin] RM4203 Rename Database by another user
In-Reply-To: <CANxoLDf5+9evQxBbV56SdtR7sFT_JXTbBiiKu+bmJ0m=Vni4VQ@mail.gmail.com>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox