public inbox for [email protected]  

help / color / mirror / Atom feed

From: Surinder Kumar <[email protected]>
To: Dave Page <[email protected]>
Cc: pgadmin-hackers <[email protected]>
Subject: Re: [pgAdmin4][Patch]: Render data grid only if Dashboard Panel is active
Date: Wed, 13 Jul 2016 12:41:38 +0530
Message-ID: <CAM5-9D_r-0DvfzYSsDgO_tYNZgPaxVWQ17SNzmvBwz773_s1ng@mail.gmail.com> (raw)
In-Reply-To: <[email protected]>
References: <CAM5-9D9qt9YZQa3Enpt9qXYk+kufqXTgENBB_DjAcvj2Y1Q50g@mail.gmail.com>
	<[email protected]>
List-Unsubscribe:  <mailto:[email protected]?body=unsub%20pgadmin-hackers>

On Wed, Jul 13, 2016 at 12:32 PM, Dave Page <[email protected]> wrote:

> Won't this result in incorrect graphs? I.e. chunks of data being missing
> (but not obviously so)?
>
 Oh I see.. It might result in incorrect graphs..
Please ignore the patch.

>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK:http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
> On 13 Jul 2016, at 07:47, Surinder Kumar <[email protected]>
> wrote:
>
> Hi
>
> In Dashboard, the ajax calls to *render data grid* should only made, if
> dashboard panel is active.
> Currently it make ajax calls even if other panels like properties, sql are
> active.
>
> Please find attached patch and review.
>
> Thanks,
> Surinder Kumar
>
> <dashboard_render_if_panel_is_opened.patch>
>
>
> --
> Sent via pgadmin-hackers mailing list ([email protected])
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgadmin-hackers
>
>

view thread (3+ 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: [pgAdmin4][Patch]: Render data grid only if Dashboard Panel is active
  In-Reply-To: <CAM5-9D_r-0DvfzYSsDgO_tYNZgPaxVWQ17SNzmvBwz773_s1ng@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