From: | Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com> |
---|---|
To: | Dave Page <dpage(at)pgadmin(dot)org> |
Cc: | Murtuza Zabuawala <murtuza(dot)zabuawala(at)enterprisedb(dot)com>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: PATCH: Added Node Type & Catalog objects [pgAdmin4] |
Date: | 2016-03-16 13:18:27 |
Message-ID: | CAG7mmoyCMZ1nm5D0D_GUs8EB_TUzPUyZyawLfwYo-wAeLwuL2A@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
On Wed, Mar 16, 2016 at 6:32 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
> Hi
>
> On Mon, Mar 14, 2016 at 12:21 PM, Murtuza Zabuawala
> <murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:
> > Hi Dave,
> >
> > Thank you for reviewing my patch.
> >
> > I have made changes according to your feedback.
> >
> > Please review updated patch.
>
> OK, I think it's in pretty good shape now. Here's what I hope are the
> last of the review comments, mostly minor things :-)
>
> - I'm allowed to edit ENUM label names, but PostgreSQL doesn't support
> that.
>
> - Security label fields resize in Edit mode. See recent changes by
> Arun to fix that elsewhere.
>
> - When editing a type, I cannot select a Grantee on the Privileges tab.
>
Is the grantor same as the current user?
If yes - then it is a bug, otherwise it is by design.
Because - you can not change privileges of a grantee, when it was given by
some other grantor.
But - if you delete the privileges for its grantor, it will remove
privileges for the privileges given by it too. (which is though, we have
not yet take care, and we will do it in later.)
--
Thanks & Regards,
Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company
<http://www.enterprisedb.com/>
*http://www.linkedin.com/in/asheshvashi*
<http://www.linkedin.com/in/asheshvashi>
>
> - When adding a composite type, I cannot type/select directly in the
> grid, I have to expand the row.
>
> - If I click ADD to add a new member to a composite type whilst the
> previous row is expanded, the new row is added, but the expanded panel
> remains linked to the first row.
>
> - If I then click on the Edit button on the new row, I see two
> expanded panels, neither of which is attached to their row (and
> they're in the wrong order). I would only expect to see one panel at a
> time.
>
> - When creating an External type, the Input, Output, Send, Receive and
> Analyze function lists are all blank. Is this expected? I would have
> thought there would be system functions listed as there are for Typmod
> in/out.
>
> - Labels for Yes/No fields should typically end in a '?', e.g.
> "Variable?", "Preferred?"
>
> - s/Prefered/Preferred
>
> - Why does "Range" type have a frame around it?
>
> - If a field is there to select/display a function, it should have
> "function" at the end of its label, e.g. "Canonical function"
>
> - As above, but with type. E.g. "Element type"
>
> - Some of the combo boxes have hint text of "Select from the list",
> whilst others don't. This should be consistent.
>
> - The colouring of "No" on the switches should be blue (see "System
> Schema?")
>
> - s/Subtype/Sub-type/
>
> - s/SubType diff/Sub-type diff function/
>
> - Please remove the pre-9.1 support.
>
> - In def create(self, gid, sid, did, scid), ensure error messages are
> properly pluralised, e.g. 'Composite types require at least two
> members' or 'External types require both Input & Output conversion
> functions.'
>
> - additional_features.sql is poorly formatted (lack of proper indentation)
>
> Thanks!
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
>
> --
> Sent via pgadmin-hackers mailing list (pgadmin-hackers(at)postgresql(dot)org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgadmin-hackers
>
From | Date | Subject | |
---|---|---|---|
Next Message | Khushboo Vashi | 2016-03-16 14:03:18 | Re: pgAdmin4 PATCH: Domain Module |
Previous Message | Dave Page | 2016-03-16 13:02:00 | Re: PATCH: Added Node Type & Catalog objects [pgAdmin4] |