Re: Newer server version warning (was: SVN Commit by dpage: r7544...) - Mailing list pgadmin-hackers

From Dave Page
Subject Re: Newer server version warning (was: SVN Commit by dpage: r7544...)
Date
Msg-id 937d27e10901120532p3eb8b06o1db716c0b80b3428@mail.gmail.com
Whole thread Raw
Responses Re: Newer server version warning (was: SVN Commit by dpage: r7544...)  (Guillaume Lelarge <guillaume@lelarge.info>)
List pgadmin-hackers
On Mon, Jan 12, 2009 at 1:18 PM, Guillaume Lelarge
<guillaume@lelarge.info> wrote:
> 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.

This is to prevent the user seeing errors when they run an old pgAdmin
against a newer server - it's not really about backup/restore (the
thread kinda changed subject).

--
Dave Page
EnterpriseDB UK:   http://www.enterprisedb.com

pgadmin-hackers by date:

Previous
From: "Dave Page"
Date:
Subject: Re: SVN Commit by dpage: r7544 - in trunk/pgadmin3: . pgadmin/frm pgadmin/include/utils pgadmin/ui
Next
From: Guillaume Lelarge
Date:
Subject: Re: PATCH: Limit 100 default in frmEditGrid.cpp