Hi This doesn't apply - can you rebase it please?
Thanks. On Thu, Nov 24, 2016 at 8:57 AM, Khushboo Vashi <khushboo.va...@enterprisedb.com> wrote: > Hi, > > Please find the attached updated patch. > > On Fri, Nov 18, 2016 at 7:45 PM, Dave Page <dp...@pgadmin.org> wrote: >> >> Hi >> >> On Fri, Nov 18, 2016 at 1:03 PM, Khushboo Vashi >> <khushboo.va...@enterprisedb.com> wrote: >> > Hi, >> > >> > Please find the attached patch for RM #1801: Properly handle databases >> > with >> > datallowconn == false. >> > >> > Issue: While connecting the database, allowconn parameter was not in >> > consideration. Also, the database properties query call was on the >> > selected >> > database which was not correct. >> > >> > Fix: Put the check of allowconn, while connecting the database in JS >> > file. >> > Also, the database properties query call set to the maintenance >> > database. >> >> All good, but this is just infrastructure for what's really required; >> i.e. disabling menu options to do things like backups, run the query >> tool etc. (anything that needs a connection). Can you resolve that >> please? >> > > Fixed > >> >> Thanks. >> >> -- >> Dave Page >> Blog: http://pgsnake.blogspot.com >> Twitter: @pgsnake >> >> EnterpriseDB UK: http://www.enterprisedb.com >> The Enterprise PostgreSQL Company > > -- 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
