Re: Feature #3061

From: Neethu Mariya Joy <neethumariyajoy(at)gmail(dot)com>
To: Murtuza Zabuawala <murtuza(dot)zabuawala(at)enterprisedb(dot)com>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: Feature #3061
Date: 2018-03-01 22:03:55
Message-ID: CAAdJd1PsaExuwVsgQNJg38W99o6W-vNfQzqkYqx9QR0uNniMqA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

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(dot)zabuawala(at)enterprisedb(dot)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
> <https://git.postgresql.org/gitweb/?p=pgadmin4.git;a=commitdiff;h=801a2084e9337e4bf07355a074f3adfbdcb9a334;hp=54dd6aae83e4d8bff6dc4a5184722cf782977fa4>
> 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(at)gmail(dot)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 <https://github.com/Roboneet> | Linkedin
>> <https://www.linkedin.com/in/neethu-mariya-joy-653655128/>
>>
>
>

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Joao De Almeida Pereira 2018-03-01 22:05:57 Re: Bug #2309 fix
Previous Message Christoph Berg 2018-03-01 17:07:39 Re: Fwd: pgAdmin4 2.1 on apt.postgresql.org