Re: [gpAdmin4][patch] query history updates - Mailing list pgadmin-hackers

From Wenlin Zhang
Subject Re: [gpAdmin4][patch] query history updates
Date
Msg-id CAEawo3JNhSDAaTVDAvgsrzGNNjS6zNjYLt5pLnfpSdogJeY4YQ@mail.gmail.com
Whole thread Raw
In response to Re: [gpAdmin4][patch] query history updates  (Khushboo Vashi <khushboo.vashi@enterprisedb.com>)
Responses Re: [gpAdmin4][patch] query history updates  (Khushboo Vashi <khushboo.vashi@enterprisedb.com>)
List pgadmin-hackers
Hi Khushboo,

     Thanks for your review. 

     About the copy/paste function bug, we just found that this is an existing bug in master. Except for the query History tab, this bug also exist in the query data output . If you copy the data row in "Data Output" tab,  it works fine, but the copy/paste in query tool won't work either. We will create a Redmine bug later.

     And the feature test failure, we've already sent a new patch that fixed it. See this patch

Thanks,

Wenlin & Violet


On Mon, Aug 7, 2017 at 1:45 PM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote:
Hi Sarah & Hao,

The copy/paste functionality through the added copy button is working fine, however once I copy the query text from the History tab through the copy button, I can not perform copy/paste for some another text in the Query tool.

Also, the feature test (QueryToolJourneyTest) is failing, please refer the attached screen-shot.

Thanks,
Khushboo

On Thu, Aug 3, 2017 at 9:03 AM, Hao Wang <haowang@pivotal.io> wrote:
Hi Hackers,

Here is a patch for query tools history UX improvements:
  • Add copy button for query text
  • Historical queries are binned by day
Thanks,
Sarah & Hao


pgadmin-hackers by date:

Previous
From: Murtuza Zabuawala
Date:
Subject: [pgAdmin4][PATCH] To fix the add New Server through Quick links on Dashboard
Next
From: Murtuza Zabuawala
Date:
Subject: Re: Can someone tell me what this code does ?