Re: Re : [BUG #4943]

I have updated the success message. 

Please check the screenshots attached.

Should i make a patch?

Regards,

Karan

On Mon, 9 Dec, 2019, 3:25 PM Aditya Toshniwal, <aditya.toshniwal@enterprisedb.com> wrote:

++pgadmin-hackers

Hi Karan,

It is good to know that you're contributing.

I would suggest {server name}/{db name} as name instead of did is better for UX. You can get the db name from conn object and server name using the sid (refer - web/pgadmin/browser/server_groups/servers/__init__.py)

On Mon, Dec 9, 2019 at 3:11 PM Karan Takalkar <karan1533os@gmail.com> wrote:

Hi,

I have been working on #4943 ("Database connected" success message itself is not enough) added by you on pgAdmin4 issues.

I want to know what details of database should be included in success message.(i have added 'did')

The success message can be modified by altering json response in the connect method in file PGADMIN_SRC/web/pgadmin/browser/server_groups/servers/databases/__init__.py.

please have a look at the screenshots attached.

original:

info=_( "Database connected.")

new:

info=_("Postgres version/{0} Database connected.".format(did))

Regards,

Karan

--

Thanks and Regards,

Aditya Toshniwal

Sr. Software Engineer | EnterpriseDB India | Pune

"Don't Complain about Heat, Plant a TREE"