On Tue, Sep 20, 2016 at 10:22 AM, Surinder Kumar <surinder.ku...@enterprisedb.com> wrote: > On Mon, Sep 19, 2016 at 9:49 PM, Dave Page <dp...@pgadmin.org> wrote: >> >> Hi >> >> >> On Mon, Sep 19, 2016 at 4:23 PM, Surinder Kumar >> <surinder.ku...@enterprisedb.com> wrote: >> > Hi >> > >> > Please find patch with following fixes: >> > >> > 1) Newly added server-group not listing in tree view. >> > >> > 2) On creating a first node for collection node with no child. the >> > created >> > node doesn't show up under its respective parent node. >> > - to add a node to child under its parent node. its parent node >> > attribute >> > must be set 'inode: true' but it always gets false for inode that >> > doesn't >> > add the node. >> > >> > Ashesh - Can you please review it? >> >> I tested this by adding a sequence to a database and schema that >> didn't previously contain any. The node was not added, and in the >> Javascript console I get: >> >> browser.js:1108 Failed to append > > This issue is taken care in updated patch, Also properly handled the case > for 'server-group' not listing in tree view, thus removing > 'addNodeWithNoParent' method from previous patch. > I have tested it for most of the nodes, it works. > Please find updated patch and review.
Works nicely - applied, 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@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgadmin-hackers
