On Wed, Aug 17, 2016 at 4:35 PM, Dave Page <dp...@pgadmin.org> wrote:
> Hi, > > I did some quick testing, and found the following issues: > > - Package ACL is not shown in properties > > - Package header and body are not shown in properties > > - System package? is not shown in properties > > - The header for the RE SQL on a package procedure should read: > > -- Package Procedure > > (not -- Package Function) > > Can you also have someone (Khushboo?) do a code review, if that hasn't > happened already? > > Sure, will do this. > Thanks. > > On Tue, Aug 16, 2016 at 1:31 PM, Harshal Dhumal > <harshal.dhu...@enterprisedb.com> wrote: > > Hi, > > > > PFA initial patch for edb packages. > > > > Other changes: > > 1] Added 'canEdit' flag for node to enable/disabled node edit. (by > default > > it's enable) > > 2] Privileges: Do not show 'ALL' in query if object has only one > applicable > > privilege instead show that privilege. > > > > > > -- > > Harshal Dhumal > > Software Engineer > > > > EnterpriseDB India: http://www.enterprisedb.com > > The Enterprise PostgreSQL Company > > > > > > -- > > Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org) > > To make changes to your subscription: > > http://www.postgresql.org/mailpref/pgadmin-hackers > > > > > > -- > 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@postgresql.org) > To make changes to your subscription: > http://www.postgresql.org/mailpref/pgadmin-hackers >
