Re: SVN Commit by dpage: r7544 - in trunk/pgadmin3: . pgadmin/frm pgadmin/include/utils pgadmin/ui

From: Guillaume Lelarge <guillaume(at)lelarge(dot)info>
To: Quan Zongliang <quanzongliang(at)gmail(dot)com>
Cc: Dave Page <dpage(at)pgadmin(dot)org>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: SVN Commit by dpage: r7544 - in trunk/pgadmin3: . pgadmin/frm pgadmin/include/utils pgadmin/ui
Date: 2009-01-12 13:18:06
Message-ID: 496B430E.8070202@lelarge.info
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Quan Zongliang a écrit :
>>> Now, const float SERVER_MIN_VERSION = 7.3f; is defined in pgAdmin3.h.
>>> Add a new variable SERVER_MAX_VERSION here.
>>> When end-users try to connect to db, check the version.
>>> If it is newer, display a warning dialog.
>>> Then the end-users click <OK> button to continue.
>
> I think again. It's really not a good idea.
> Another option is needed. Otherwise, when end users connect to db
> with an older version pgAdmin. They will get message box always,
> but the pgAdmin maybe work fine. This must make them very angry.
>
> Add a new option like "Don't check database version."?
>

I don't think so. pgAdmin should display the warning box only if they
try to backup/restore, and not at startup time.

--
Guillaume.
http://www.postgresqlfr.org
http://dalibo.com

In response to

Browse pgadmin-hackers by date

  From Date Subject
Next Message Dave Page 2009-01-12 13:30:18 Re: SVN Commit by dpage: r7544 - in trunk/pgadmin3: . pgadmin/frm pgadmin/include/utils pgadmin/ui
Previous Message Quan Zongliang 2009-01-12 12:53:30 Re: SVN Commit by dpage: r7544 - in trunk/pgadmin3: . pgadmin/frm pgadmin/include/utils pgadmin/ui