Postgres Pro
Downloads
Home   >   mailing lists

Re: Re : [BUG #4943] - Mailing list pgadmin-hackers

From Aditya Toshniwal
Subject Re: Re : [BUG #4943]
Date
Msg-id CAM9w-_=KyZkcHFg9g1esMq7mSiAEYqaC1hWcFiqnj4W8MyPk0w@mail.gmail.com
Whole thread Raw
List pgadmin-hackers
Tree view
++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"

pgadmin-hackers by date:

Previous
From: Akshay Joshi
Date:
Subject: Re: RM4939 - Running in the container as non-root user
Next
From: Aditya Toshniwal
Date:
Subject: [pgAdmin][RM4506] "can't execute an empty query" message displayed ifuser remove fill factor of any existing table