From: | Ronan Dunklau <ronan(dot)dunklau(at)dalibo(dot)com> |
---|---|
To: | Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com> |
Cc: | Dave Page <dpage(at)pgadmin(dot)org>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: PGAdmin 4 architecture (Was: [Patch] PGAdmin 4 JSON Handling) |
Date: | 2015-04-23 07:22:19 |
Message-ID: | 1925731.1Pore36p3p@ronan.dunklau.fr |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Le jeudi 23 avril 2015 10:53:48 Ashesh Vashi a écrit :
> Hi Dave/Ronan,
>
> On Mon, Apr 20, 2015 at 4:10 PM, Dave Page <dpage(at)pgadmin(dot)org>
wrote:
> > On Mon, Apr 20, 2015 at 10:52 AM, Ronan Dunklau
> >
> > <ronan(dot)dunklau(at)dalibo(dot)com> wrote:
> > >> Ronan; can you update the test, help and about modules as well
please?
> > >
> > > Done, please find attached a new patch for that. Ashesh, once
you're
> >
> > done with
> >
> > > what you are doing now, feel free to ask me for any help needed to
> >
> > integrate
> >
> > > this after the fact.
> >
> > Thanks.
>
> Thanks - It looks good to me.
> (And, attached patch is based on that.)
Maybe I'm missing something, but it seems like a
"generate_browser_node" function is missing:
File
"/home/ro/projets/pgadmin4/web/pgadmin/browser/server_groups/__ini
t__.py", line 21, in <module>
from pgadmin.browser.utils import generate_browser_node
ImportError: cannot import name generate_browser_node
It also brought to my attention that I should have removed the only
function (register_modules) from pgadmin.browser.utils.
Thank you for integrating my work into your patch !
--
Ronan Dunklau
http://dalibo.com - http://dalibo.org
From | Date | Subject | |
---|---|---|---|
Next Message | Dave Page | 2015-04-23 08:11:59 | Re: PGAdmin 4 architecture (Was: [Patch] PGAdmin 4 JSON Handling) |
Previous Message | Ashesh Vashi | 2015-04-23 05:23:48 | Re: PGAdmin 4 architecture (Was: [Patch] PGAdmin 4 JSON Handling) |