On Thu, Jan 6, 2011 at 7:30 PM, Jasmin Dizdarevic <jasmin.dizdare...@gmail.com> wrote: > Hi, > here is the patch, that enables UI for pgAgent 4.
Cool! > It would be great if somebody could try to build in Release mode. > The UI should work with pgAgent schema version 3 - if not, please tell me. It doesn't. See the attached screenshot for the error seen upon connection. Further errors when checking opening the new job dialogue, then a crash. It shouldn't take much effort to figure out I imagine :-) > Please tell me also, if I have violated coding policies or something else. Actually, it looks pretty clean to me. A good first-ever patch, style wise and architecturally. > To test the new features, you can apply attached pgagent_upgrade.sql. You're missing a ; in there on line 19, and have terminated lines 27 - 29 with , instead of ;. Fixing that, and I can see the new UI without any errors, however, I cannot test it without an agent. Actually, that's pretty annoying - currently I can configure my jobs before I setup my agent, but with this change, I can no longer do that, because the agents need to have registered their email capability in the database already. I'm not sure that's behaviour that is desirable... but it becomes irrelevant if we fully integrate the SMTP code as discussed. -- Dave Page Blog: http://pgsnake.blogspot.com Twitter: @pgsnake EnterpriseDB UK: http://www.enterprisedb.com The Enterprise PostgreSQL Company
<<attachment: Screen shot 2011-01-07 at 14.23.14.png>>
-- Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgadmin-hackers
