[pgAdmin][Patch] #4059 Query Tool button in Query Tool to open a new Query Window

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
7 messages Options
Reply | Threaded
Open this post in threaded view
|

[pgAdmin][Patch] #4059 Query Tool button in Query Tool to open a new Query Window

Rahul Shirsat
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.
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:

Screen Shot 2020-08-27 at 8.04.33 PM.png

--
Rahul Shirsat
Software Engineer | EnterpriseDB Corporation.

RM4059.patch (7K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [pgAdmin][Patch] #4059 Query Tool button in Query Tool to open a new Query Window

Rahul Shirsat

Please find the update patch attached here.

On Thu, Aug 27, 2020 at 8:10 PM Rahul Shirsat <[hidden email]> 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.
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:

Screen Shot 2020-08-27 at 8.04.33 PM.png

--
Rahul Shirsat
Software Engineer | EnterpriseDB Corporation.


--
Rahul Shirsat
Software Engineer | EnterpriseDB Corporation.

RM4059_v2.patch (7K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [pgAdmin][Patch] #4059 Query Tool button in Query Tool to open a new Query Window

Akshay Joshi
Thanks, patch applied.

On Thu, Aug 27, 2020 at 8:16 PM Rahul Shirsat <[hidden email]> wrote:

Please find the update patch attached here.

On Thu, Aug 27, 2020 at 8:10 PM Rahul Shirsat <[hidden email]> 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.
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:

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
Mobile: +91 976-788-8246

Reply | Threaded
Open this post in threaded view
|

Re: [pgAdmin][Patch] #4059 Query Tool button in Query Tool to open a new Query Window

Rahul Shirsat
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 <[hidden email]> wrote:
Thanks, patch applied.

On Thu, Aug 27, 2020 at 8:16 PM Rahul Shirsat <[hidden email]> wrote:

Please find the update patch attached here.

On Thu, Aug 27, 2020 at 8:10 PM Rahul Shirsat <[hidden email]> 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.
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:

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
Mobile: +91 976-788-8246



--
Rahul Shirsat
Software Engineer | EnterpriseDB Corporation.

RM4059_v3.patch (9K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [pgAdmin][Patch] #4059 Query Tool button in Query Tool to open a new Query Window

Akshay Joshi
Thanks, patch applied.

On Wed, Sep 9, 2020 at 6:23 PM Rahul Shirsat <[hidden email]> 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 <[hidden email]> wrote:
Thanks, patch applied.

On Thu, Aug 27, 2020 at 8:16 PM Rahul Shirsat <[hidden email]> wrote:

Please find the update patch attached here.

On Thu, Aug 27, 2020 at 8:10 PM Rahul Shirsat <[hidden email]> 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.
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:

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
Mobile: +91 976-788-8246



--
Rahul Shirsat
Software Engineer | EnterpriseDB Corporation.


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

Reply | Threaded
Open this post in threaded view
|

Re: [pgAdmin][Patch] #4059 Query Tool button in Query Tool to open a new Query Window

Rahul Shirsat
Hi Akshay,

Issue is fixed for alerts getting popped up on the main window instead of the current window.

On Wed, Sep 9, 2020 at 8:13 PM Akshay Joshi <[hidden email]> wrote:
Thanks, patch applied.

On Wed, Sep 9, 2020 at 6:23 PM Rahul Shirsat <[hidden email]> 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 <[hidden email]> wrote:
Thanks, patch applied.

On Thu, Aug 27, 2020 at 8:16 PM Rahul Shirsat <[hidden email]> wrote:

Please find the update patch attached here.

On Thu, Aug 27, 2020 at 8:10 PM Rahul Shirsat <[hidden email]> 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.
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:

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
Mobile: +91 976-788-8246



--
Rahul Shirsat
Software Engineer | EnterpriseDB Corporation.


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



--
Rahul Shirsat
Software Engineer | EnterpriseDB Corporation.

RM4059_v4.patch (2K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [pgAdmin][Patch] #4059 Query Tool button in Query Tool to open a new Query Window

Akshay Joshi
Thanks, patch applied.

On Wed, Sep 9, 2020 at 9:45 PM Rahul Shirsat <[hidden email]> wrote:
Hi Akshay,

Issue is fixed for alerts getting popped up on the main window instead of the current window.

On Wed, Sep 9, 2020 at 8:13 PM Akshay Joshi <[hidden email]> wrote:
Thanks, patch applied.

On Wed, Sep 9, 2020 at 6:23 PM Rahul Shirsat <[hidden email]> 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 <[hidden email]> wrote:
Thanks, patch applied.

On Thu, Aug 27, 2020 at 8:16 PM Rahul Shirsat <[hidden email]> wrote:

Please find the update patch attached here.

On Thu, Aug 27, 2020 at 8:10 PM Rahul Shirsat <[hidden email]> 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.
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:

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
Mobile: +91 976-788-8246



--
Rahul Shirsat
Software Engineer | EnterpriseDB Corporation.


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



--
Rahul Shirsat
Software Engineer | EnterpriseDB Corporation.


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