From: | Surinder Kumar <surinder(dot)kumar(at)enterprisedb(dot)com> |
---|---|
To: | Dave Page <dpage(at)pgadmin(dot)org> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: Control for displaying "auto vacuum" fields into grid |
Date: | 2016-03-23 16:04:31 |
Message-ID: | CAM5-9D8=DMS75mhNCpPP3jn6=_JkT=drNkUr9gPRXBs5xLLDBg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi,
On Fri, Mar 18, 2016 at 8:58 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
> On Fri, Mar 18, 2016 at 10:37 AM, Surinder Kumar
> <surinder(dot)kumar(at)enterprisedb(dot)com> wrote:
> > Hi,
> >
> > PFA control for displaying auto vacuum fields into grid. This control is
> > common for
> > Materialized View Node and Table Node.
> >
> > Usage:
> >
> > {
> > id: 'vacuum_table', label: '{{ _("Vacuum Table") }}',
> > model: VacuumTableModel, editable: false, type: 'collection',
> > canEdit: true, group: '{{ _("Table") }}',
> > mode: ['edit', 'create'], url: 'get_vacuum_defaults',
> > control: Backform.VacuumCollectionControl.extend({
> > grid_columns :[
> > {
> > name: 'label', label: '{{ _("Label") }}',
> > cell: 'string', editable: false
> > },
> > {
> > name: 'value', label: '{{ _("Value") }}',
> > cellFunction: cellFunction, editable: function(m) {
> > if(m.handler.has('autovacuum_enabled')) {
> > return m.handler.get('autovacuum_enabled');
> > }
> > return !m.handler.isNew();
> > }
> > },
> > {
> > name: 'setting', label: '{{ _("Default value") }}',
> > cellFunction: cellFunction, editable: false
> > }
> > ]
> > }),
> >
> >
> > When using this control, provide following parameters in schema:
> > 1. model
> > 2. url - to fetch default values for auto vacuum fields.
> > 3. grid columns - Name of the columns to display in the grid.
> >
> >
> > Please review the patch.
>
> It's hard to review this without being able to test. Do you have a
> simple test case?
>
I have submitted patch for materialized view which is using this control.
so you can test it in materialized view.
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
From | Date | Subject | |
---|---|---|---|
Next Message | Ashesh Vashi | 2016-03-24 07:30:42 | pgAdmin 4 commit: The changed value of a multiline/text control was not |
Previous Message | Dave Page | 2016-03-23 16:03:25 | pgAdmin 4 commit: String improvements. |