From: | Dave Page <dpage(at)pgadmin(dot)org> |
---|---|
To: | Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com> |
Cc: | Aditya Toshniwal <aditya(dot)toshniwal(at)enterprisedb(dot)com>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: Feature #5370 User should be able to set the binary path for each database server |
Date: | 2021-05-21 07:31:04 |
Message-ID: | CA+OCxozV34Ec_+VoxNfUL0ROdsuzmyqvo_wfosBUSw6OTssUGg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi Akshay,
On Fri, May 21, 2021 at 8:03 AM Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
wrote:
> Hi Dave/Hackers
>
> As per your suggestion, I have created a new Backform control
> "BinaryPathsGridControl" and two new BackgridCell (BackgridRadioCell and
> BackgridSelectFileCell). Please refer to the screenshot below:
> [image: Binary_Path.png]
>
> Are the above changes look good to you? The radio button will only be
> enabled when there is a path. Added validate button which will validate the
> Utilities (pg_dump, pg_dumpall, ...)
>
Nice! Just a couple of comments:
- I assume the browse button is removed in server mode as discussed? Maybe
we should add a config.py option to allow that behaviour to be overridden
if the admin doesn't care about sandboxing?
- I think we need some hint text. How about something like:
Enter the directory in which the psql, pg_dump, pg_dumpall, and pg_restore
utilities can be found for the corresponding database server version. The
default path will be used for server versions that do not have a path
specified.
--
Dave Page
Blog: https://pgsnake.blogspot.com
Twitter: @pgsnake
From | Date | Subject | |
---|---|---|---|
Next Message | Pradip Parkale | 2021-05-21 08:52:55 | [pgAdmin][RM6231]- Add OS, Browser details in pgAdmin About us pop-up |
Previous Message | Akshay Joshi | 2021-05-21 07:03:34 | Re: Feature #5370 User should be able to set the binary path for each database server |