Re: Feature #3061

From: Murtuza Zabuawala <murtuza(dot)zabuawala(at)enterprisedb(dot)com>
To: Neethu Mariya Joy <neethumariyajoy(at)gmail(dot)com>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: Feature #3061
Date: 2018-03-02 04:32:03
Message-ID: CAKKotZQ0nfGBrwE3o91G-++v4tekNW+umKkR-mg-ZJkac80RFw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hello,

On Fri, Mar 2, 2018 at 3:33 AM, Neethu Mariya Joy <neethumariyajoy(at)gmail(dot)com
> wrote:

> 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?
>

​I think we should keep that what if user wants to disable all the graphs
on dashboards including the user defined custom graphs​, User has go and
disable each every graphs, if we provide a single option to turn off all
the graphs then it will be more user friendly.

b. Do widgets include tables or just the graphs?
>

​Yes, It may include Table as you can see in "Server Activity" section but
in custom table we will only display it in terms of plain HTML table and we
won't use Backgrid.​

>
> 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
>> <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

Browse pgadmin-hackers by date

  From Date Subject
Next Message Khushboo Vashi 2018-03-02 04:56:28 Re: [pgAdmin4][Patch]: PEP-8 fixes in the foreign data wrapper module
Previous Message Murtuza Zabuawala 2018-03-02 04:15:20 Re: Bug #2309 fix