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: [pgAdmin][Patch] #4059 Query Tool button in Query Tool to open a new Query Window |
Date: | 2020-09-09 14:43:23 |
Message-ID: | CANxoLDdVD0C9q85eaaHdYT_NK74rk9dT3mig=2+Vu1X1krTqzg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Thanks, patch applied.
On Wed, Sep 9, 2020 at 6:23 PM Rahul Shirsat <rahul(dot)shirsat(at)enterprisedb(dot)com>
wrote:
> Hi Akshay,
>
> Please find the attached patch which adds / corrects following features /
> issues :
>
> 1. Added tooltip & keyboard shortcut for query tool button
> 2. Corrected the *Uncaught TypeError: Cannot read property āiā of
> undefined*
> This issue can be reproduced by :
> *Keeping the click on any node other than database, refresh the
> page, open the same server (it should navigate to the previously selected
> node), open query tool, now open query tool via sqleditor query tool
> button, issue should be reproduced* - *Resolved*
> 3. Aditya's changes for *popups are blocked* are now extended for the new
> tab query tool too.
>
>
> On Fri, Aug 28, 2020 at 6:24 PM Akshay Joshi <
> akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
>
>> Thanks, patch applied.
>>
>> On Thu, Aug 27, 2020 at 8:16 PM Rahul Shirsat <
>> rahul(dot)shirsat(at)enterprisedb(dot)com> wrote:
>>
>>>
>>> Please find the update patch attached here.
>>>
>>> On Thu, Aug 27, 2020 at 8:10 PM Rahul Shirsat <
>>> rahul(dot)shirsat(at)enterprisedb(dot)com> wrote:
>>>
>>>> Hi Hackers,
>>>>
>>>> Please find the attached patch below which adds the functionality of
>>>> the query tool button in the query tool sqleditor.
>>>>
>>>> *Acceptance criteria:*
>>>> - For sqleditor on same window & on new tab:
>>>>
>>>> - When a query tool connection is initiated, as expected it will
>>>> open the connection based on the selected database in the treeview. Now,
>>>> when the user clicks the query tool connection button on the query tool
>>>> window, irrespective of the selected database in treeview, it should open a
>>>> connection based on the query tool connected database.
>>>> - Similarly, for a new tab, clicking on the query tool connection
>>>> button, it should open a connection based on the query tool connected
>>>> database, instead of selected treeview node.
>>>>
>>>> [image: query tool button.png]
>>>>
>>>> Additionally, an error is handled in the form of message dialog
>>>> prompting the user to initiate the connection, when sqleditor is opened in
>>>> the new tab with the main application window kept closed. This can be
>>>> reviewed by refreshing the window as well as clicking on the query tool
>>>> connection button.
>>>>
>>>> A prompt dialog message box is seen as:
>>>>
>>>> [image: Screen Shot 2020-08-27 at 8.04.33 PM.png]
>>>>
>>>> --
>>>> *Rahul Shirsat*
>>>> Software Engineer | EnterpriseDB Corporation.
>>>>
>>>
>>>
>>> --
>>> *Rahul Shirsat*
>>> Software Engineer | EnterpriseDB Corporation.
>>>
>>
>>
>> --
>> *Thanks & Regards*
>> *Akshay Joshi*
>> *pgAdmin Hacker | Sr. Software Architect*
>> *EDB Postgres <http://edbpostgres.com>*
>>
>> *Mobile: +91 976-788-8246*
>>
>
>
> --
> *Rahul Shirsat*
> Software Engineer | EnterpriseDB Corporation.
>
--
*Thanks & Regards*
*Akshay Joshi*
*pgAdmin Hacker | Sr. Software Architect*
*EDB Postgres <http://edbpostgres.com>*
*Mobile: +91 976-788-8246*
From | Date | Subject | |
---|---|---|---|
Next Message | Rahul Shirsat | 2020-09-09 16:15:15 | Re: [pgAdmin][Patch] #4059 Query Tool button in Query Tool to open a new Query Window |
Previous Message | Akshay Joshi | 2020-09-09 14:43:13 | Re: [pgAdmin][RM5815] Generate script is not working in new window |