Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA384:256) (Exim 4.89) (envelope-from ) id 1erWJ1-00077G-QP for pgadmin-hackers@arkaria.postgresql.org; Thu, 01 Mar 2018 22:04:04 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.89) (envelope-from ) id 1erWJ0-00073X-9N for pgadmin-hackers@arkaria.postgresql.org; Thu, 01 Mar 2018 22:04:02 +0000 Received: from makus.postgresql.org ([2001:4800:1501:1::229]) by malur.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA384:256) (Exim 4.89) (envelope-from ) id 1erWIz-00073N-PX for pgadmin-hackers@lists.postgresql.org; Thu, 01 Mar 2018 22:04:02 +0000 Received: from mail-pl0-x243.google.com ([2607:f8b0:400e:c01::243]) by makus.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.89) (envelope-from ) id 1erWIv-0005VL-DM for pgadmin-hackers@postgresql.org; Thu, 01 Mar 2018 22:04:00 +0000 Received: by mail-pl0-x243.google.com with SMTP id m22-v6so929706pls.5 for ; Thu, 01 Mar 2018 14:03:56 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc; bh=jRJDOp/Cvf9d87JhfnnRRwM6r6NER2LSpyzHnrkVU3M=; b=eOnFtEKv7UwIuE2if8b6gVbwGDTU5XyGl6lNgjXnQSunuTxQn4gLaB5CrqPrjqs0kf ATLO5Nik0bhzLVlnQ93tM+7HjZJKJbxddDB8m4CbpLs8kwbMa4pNaz+qPv7IdIGeWU7i cmyx0suMHVS2SWvUzCejkHAUiTxNJ5w58fmOd/RG566ixZM1fi+ja7RhUtqmb3TBZAIP uxgTbQrd8gqpj4XIcakgUIU3FwA17Dim0I6Gvrb3Kk6iiCPaapkHE2nyRLyWfVJKZ7Hd xUCymQqWuJHyHFxZIZ05uOHHrJHff7rj7R/WPpb4ynHu20G3afWEv/svYUmrM/iCKNUv dytg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=jRJDOp/Cvf9d87JhfnnRRwM6r6NER2LSpyzHnrkVU3M=; b=j1/xQTp/xejYGKwgQYcoR+La5YuEu4vODIGK5HdO47g1C+Acnp1GFSynoYBiNxG8Oi jgzC0rhkd/VBbrW7QlYC2jr//Yvqvxo5XdB3hXX+ytSDQq4Lo3SixleqNHWpjBS+rX+h JLN4X95joeqXvdEgJGY0u4LPRXd6ZFaI4shAM3udzBIBuZLa24LAq97kEmvOc95r60/Q 4yV9RszKedYY4eWGOwz0NttVON6kW7v4sYK9eH9CuChyLKwOUczqdonnjuAIQyTdCkVa rYWdmCc/9U94e6bG2NPsYteWzQ9eWj5yAnG7KBXGNl32dMNpayJA2fT7PWcsS4Lt6aue AqCw== X-Gm-Message-State: APf1xPDPoT+Kl+JmFnVt2Xsg2pYkpVBzBtCtExptr9/9LJLQRkYSL3kf T9KSAQxPRh/ccJx4vh3M2LHY39HhVq0B6/7DNP4= X-Google-Smtp-Source: AG47ELscDc0ZNQCFIXF+ybIeYv96zBQSuxreQEqQ2DOuBPrNAeJVz2O/o4n+wtDBMwwi7iT9q6S4DodkTlJ92CXCgVk= X-Received: by 2002:a17:902:720b:: with SMTP id ba11-v6mr3281328plb.148.1519941835806; Thu, 01 Mar 2018 14:03:55 -0800 (PST) MIME-Version: 1.0 Received: by 10.100.186.193 with HTTP; Thu, 1 Mar 2018 14:03:55 -0800 (PST) In-Reply-To: References: From: Neethu Mariya Joy Date: Fri, 2 Mar 2018 03:33:55 +0530 Message-ID: Subject: Re: Feature #3061 To: Murtuza Zabuawala Cc: pgadmin-hackers Content-Type: multipart/alternative; boundary="00000000000033dbc0056661098c" List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Precedence: bulk --00000000000033dbc0056661098c Content-Type: text/plain; charset="UTF-8" Hi, Thanks for the reply. I had a look at the commit and tried out the first part. I have a few doubts. a. Now that we are adding options for each individual graph, should we remove the previous show graphs option or keep it as an override when it is false? b. Do widgets include tables or just the graphs? Sincerely, Neethu On Thu, Mar 1, 2018 at 10:39 AM, Murtuza Zabuawala < murtuza.zabuawala@enterprisedb.com> wrote: > Hello, > > First of all I'm assuming that you have some degree of knowledge about > Python, HTML, JS & Git. > > We have had added similar feature recently. > Check: https://redmine.postgresql.org/issues/2951 for more details. > Code location of Dashboard module in git repository: > ../pgadmin4/web/pgadmin/dashboard > > There are 3 requirments in the ticket, > 1) disable/enable some widgets (widget - Server Sessions | Transactions > per second | Tuples in etc) > We have done this for all the graphs in one shot, now user can > enable/disable all the graphs via preferences dialog. > You can refer the commit > > as a reference if you want to add this functionality for each > individual graphs. > > 2) add custom/user specify widgets > - First of all, You need to provide an option to Create new dashboard > dialog, Update existing dashboard dialog option and also to delete those > existing custom dashboards on dashboard page. > - You have to create a separate table in sqlite3 database to store the > data for that user defined dashboards like Dashboard name, SQL etc. > - You also have to provide option to use to choose what type of graph user > wants (Line chart, Pie chart, Table etc) > - Size & location of the graph/table. > > 3) add user role with access only to dashboard > This needs further discussion with community members if we really need > this or not. > > You can refer README for more information regarding how to run pgAdmin4. > > Let us know if you need any help. > > > -- > Regards, > Murtuza Zabuawala > EnterpriseDB: http://www.enterprisedb.com > The Enterprise PostgreSQL Company > > > On Wed, Feb 28, 2018 at 6:29 PM, Neethu Mariya Joy < > neethumariyajoy@gmail.com> wrote: > >> Hi, >> >> I am Neethu Mariya Joy, an undergraduate sophomore pursuing BE(Hons) in >> Computer Engineering from BITS Pilani, India. >> >> I would like to work on the feature #3061, "Dashboard Customisation". >> Kindly provide suggestions for the same. >> >> Sincerely, >> Neethu Mariya Joy >> GitHub | Linkedin >> >> > > --00000000000033dbc0056661098c Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable
Hi,

Thanks for the reply. I had a look = at the commit and tried out the first part.

I have= a few doubts.
a. Now that we are adding options for each individ= ual graph, should we remove the previous show graphs option or keep it as a= n override when it is false?
b. Do widgets include tables or just= the graphs?

Sincerely,
Neethu

On Thu, Mar 1, = 2018 at 10:39 AM, Murtuza Zabuawala <murtuza.zabuawala@en= terprisedb.com> wrote:
Hello,

First of all I'm assuming that you have some degree of knowledge = about Python, HTML, JS & Git.

We have had added similar feature recentl= y.
Check:=C2=A0https://redmine.postgresql.org= /issues/2951 for more details.
Code location of Dashboard module in = git repository: ../pgadmi= n4/web/pgadmin/dashboard

There are 3 requirments in the ticket,
1) disable/enable= some widgets (widget - Server Sessions | Transactions per second | Tuples = in etc)=C2=A0
We have done this for all the graphs in on= e shot, now user can enable/disable all the graphs via preferences dialog.<= /font>
You can refer the = commit as a reference if you want to add this functionality=C2=A0for ea= ch individual=C2=A0graphs.

2) = add custom/user specify widgets=C2=A0
- First of all, Yo= u need to provide an option to Create new dashboard dialog, Update existing= dashboard dialog option and also to delete those existing custom dashboard= s on dashboard page.
= - You have to create a separate=C2=A0table in sqlite3 database to store the= data for that user defined dashboards like Dashboard name, SQL etc.=
- You also have to provide option to use to choose what type o= f graph user wants (Line chart, Pie chart, Table etc)
- = Size & location of the graph/table.

3) add user role with access only to dashboard
This needs further=C2=A0discussion with community membe= rs if we really need this or not.
=
You can= refer README for more information regarding how to run pgAdmin4.

=
Let us know if you need any help.


<= div dir=3D"ltr">
--
Regards,
<= font size=3D"2">Murtuz= a Zabuawala
EnterpriseDB:=C2=A0http://www.enterprisedb.comThe Enterprise PostgreSQL Company


On Wed, Feb 28, 2018 at 6:29 PM, Neethu Mari= ya Joy <neethumariyajoy@gmail.com> wrote:
Hi,=C2=A0

I am = Neethu Mariya Joy,=C2=A0an undergraduate sophomo= re pursuing BE(Hons) in Computer Engineering from BITS Pilani, India.=C2=A0=

I would like to work on the feature #3061, "Dashboa= rd Customisation". Kindly provide suggestions for the same.

Sincerely,
Neethu Mariy= a Joy
GitHub=C2=A0|=C2=A0Linkedin


--00000000000033dbc0056661098c--