Re: [pgAdmin][RM7351]: Templates not displayed

Поиск
Список
Период
Сортировка
От Dave Page
Тема Re: [pgAdmin][RM7351]: Templates not displayed
Дата
Msg-id CA+OCxoz-LsVYMpXGisZKyR_WPbqRx+a8Ba07C_7_EvDZ-HwfNw@mail.gmail.com
обсуждение исходный текст
Ответ на Re: [pgAdmin][RM7351]: Templates not displayed  (Pradip Parkale <pradip.parkale@enterprisedb.com>)
Ответы Re: [pgAdmin][RM7351]: Templates not displayed  (Pradip Parkale <pradip.parkale@enterprisedb.com>)
Список pgadmin-hackers


On Tue, 17 May 2022 at 08:08, Pradip Parkale <pradip.parkale@enterprisedb.com> wrote:
Hi Dave/Team,

On Wed, May 11, 2022 at 4:37 PM Aditya Toshniwal <aditya.toshniwal@enterprisedb.com> wrote:
Hi,

On Wed, May 11, 2022 at 4:03 PM Dave Page <dpage@pgadmin.org> wrote:
Hi

On Wed, 11 May 2022 at 09:40, Aditya Toshniwal <aditya.toshniwal@enterprisedb.com> wrote:
Hi Dave,

On Wed, May 11, 2022 at 2:00 PM Dave Page <dpage@pgadmin.org> wrote:


On Wed, 11 May 2022 at 09:24, Aditya Toshniwal <aditya.toshniwal@enterprisedb.com> wrote:
Hi,

On Wed, May 11, 2022 at 1:24 PM Dave Page <dpage@pgadmin.org> wrote:
Hi

I think this change needs some thought and discussion on how it should be implemented. pgAdmin has worked this way for over 20 years with only this one suggestion to change afaicr - and I for one don't suddenly want templates showing up amongst my other databases.
User defined template DBs cannot come under "Show system objects". I mean, they're not system objects.

No they're not. But we have over 20 years of them being classed that way, and users may not suddenly want to see template1 (for example) listed amongst their databases. There are various things to think about here, for example:
No template1, template0 are system objects. They won't be visible. Only new databases created manually and marked as template are excluded from "Show system objects" .

template0 is. template1 is *by default*, but not if you drop and recreate it.
Didn't think in that way :)
In that case, we can add a new preference - "Show non-system/user defined template databases?" explicitly. By default "yes", show them.
Along with a new icon.
Has this been finalized?

No -  I haven't seen any proposals for what will be done, except for what you've written below which no one has commented on yet.
 
Could I add a new preference setting to show/hide user-defined template databases? The new icon for that may look like a 'T' on top of the current DB icon.

I think that's fine. 
 

 

- Should there be a separate option to show/hide template databases?
- If so, what should the default be (hint: I think yes, and off)
- Should template databases have a different icon?
We can have this. Good to differentiate. 
- Should they have their own collection node?
- They can't be connected to, so much of what's on the database properties dialog won't work. Should they have their own properties dialogue design?
I can connect to template DBs in pgAdmin.

Yeah, disregard that. My brain was mixing up datistemplate and datallowconn.
 

 

On Wed, 11 May 2022 at 07:12, Pradip Parkale <pradip.parkale@enterprisedb.com> wrote:
Hi Hackers,

Please find the attached patch for # 7351:Templates not displayed.

I have fixed below issues
  1. Databases which are marked as template manually by the user should be visible independent of - "Show System Objects".
  2.  DB properties dialogs should allow you to change template flag in edit mode.


--
Thanks & Regards,
Pradip Parkale
Software Engineer | EnterpriseDB Corporation


--


--
Thanks,
Aditya Toshniwal
pgAdmin Hacker | Software Architect | edbpostgres.com
"Don't Complain about Heat, Plant a TREE"


--


--
Thanks,
Aditya Toshniwal
pgAdmin Hacker | Software Architect | edbpostgres.com
"Don't Complain about Heat, Plant a TREE"


--


--
Thanks,
Aditya Toshniwal
pgAdmin Hacker | Software Architect | edbpostgres.com
"Don't Complain about Heat, Plant a TREE"


--
Thanks & Regards,
Pradip Parkale
Software Engineer | EnterpriseDB Corporation


--

В списке pgadmin-hackers по дате отправления:

Предыдущее
От: Pradip Parkale
Дата:
Сообщение: Re: [pgAdmin][RM7351]: Templates not displayed
Следующее
От: Aditya Toshniwal
Дата:
Сообщение: [pgAdmin][RM7388] Error message will fill the entire window if the query is long