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

From: Pradip Parkale <pradip(dot)parkale(at)enterprisedb(dot)com>
To: Aditya Toshniwal <aditya(dot)toshniwal(at)enterprisedb(dot)com>
Cc: Dave Page <dpage(at)pgadmin(dot)org>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: [pgAdmin][RM7351]: Templates not displayed
Date: 2022-05-17 07:08:38
Message-ID: CAJ9T6SvFL+hdibq+0XQdaLSz6VWdraTOsgL6Q_bgp2cMJq5Xmw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi Dave/Team,

On Wed, May 11, 2022 at 4:37 PM Aditya Toshniwal <
aditya(dot)toshniwal(at)enterprisedb(dot)com> wrote:

> Hi,
>
> On Wed, May 11, 2022 at 4:03 PM Dave Page <dpage(at)pgadmin(dot)org> wrote:
>
>> Hi
>>
>> On Wed, 11 May 2022 at 09:40, Aditya Toshniwal <
>> aditya(dot)toshniwal(at)enterprisedb(dot)com> wrote:
>>
>>> Hi Dave,
>>>
>>> On Wed, May 11, 2022 at 2:00 PM Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>>
>>>>
>>>>
>>>> On Wed, 11 May 2022 at 09:24, Aditya Toshniwal <
>>>> aditya(dot)toshniwal(at)enterprisedb(dot)com> wrote:
>>>>
>>>>> Hi,
>>>>>
>>>>> On Wed, May 11, 2022 at 1:24 PM Dave Page <dpage(at)pgadmin(dot)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? 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.

>
>>
>>>
>>>> - 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(dot)parkale(at)enterprisedb(dot)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
>>>>>>>
>>>>>>
>>>>>>
>>>>>> --
>>>>>> Dave Page
>>>>>> Blog: https://pgsnake.blogspot.com
>>>>>> Twitter: @pgsnake
>>>>>>
>>>>>> EDB: https://www.enterprisedb.com
>>>>>>
>>>>>>
>>>>>
>>>>> --
>>>>> Thanks,
>>>>> Aditya Toshniwal
>>>>> pgAdmin Hacker | Software Architect | *edbpostgres.com*
>>>>> <http://edbpostgres.com>
>>>>> "Don't Complain about Heat, Plant a TREE"
>>>>>
>>>>
>>>>
>>>> --
>>>> Dave Page
>>>> Blog: https://pgsnake.blogspot.com
>>>> Twitter: @pgsnake
>>>>
>>>> EDB: https://www.enterprisedb.com
>>>>
>>>>
>>>
>>> --
>>> Thanks,
>>> Aditya Toshniwal
>>> pgAdmin Hacker | Software Architect | *edbpostgres.com*
>>> <http://edbpostgres.com>
>>> "Don't Complain about Heat, Plant a TREE"
>>>
>>
>>
>> --
>> Dave Page
>> Blog: https://pgsnake.blogspot.com
>> Twitter: @pgsnake
>>
>> EDB: https://www.enterprisedb.com
>>
>>
>
> --
> Thanks,
> Aditya Toshniwal
> pgAdmin Hacker | Software Architect | *edbpostgres.com*
> <http://edbpostgres.com>
> "Don't Complain about Heat, Plant a TREE"
>

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

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Dave Page 2022-05-17 08:20:35 Re: [pgAdmin][RM7351]: Templates not displayed
Previous Message Pradip Parkale 2022-05-17 06:48:52 [pgAdmin]: Feature test cases fixes