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

From: Dave Page <dpage(at)pgadmin(dot)org>
To: Pradip Parkale <pradip(dot)parkale(at)enterprisedb(dot)com>
Cc: Aditya Toshniwal <aditya(dot)toshniwal(at)enterprisedb(dot)com>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: [pgAdmin][RM7351]: Templates not displayed
Date: 2022-05-18 13:41:12
Message-ID: CA+OCxoxF6M9HTXSPDv=XJuuG7vp2JmWSXRT5isCiX5mR9zK3cw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi

On Wed, 18 May 2022 at 14:02, Pradip Parkale <
pradip(dot)parkale(at)enterprisedb(dot)com> wrote:

> Hi Dave/Team,
>
> I have come up with a plan to implement this. Please give your suggestions
>
>
> 1. Users will be able to hide user-defined templates in the
> preferences setting. By default, the value is 'Yes', and all user-defined
> templates will be hidden.
> 2. The icon will be different for user-defined templates.
> 3. Users can connect to templete DB and properies are also same, so no
> need to change the properties dialog options/design.
> 4. We are giving options to hide the templates and by default, the
> option will be true, so the collection node for template DBs will be the
> same as for other databases.
>
> Works for me. Thanks.

>
> On Tue, May 17, 2022 at 1:50 PM Dave Page <dpage(at)pgadmin(dot)org> wrote:
>
>>
>>
>> On Tue, 17 May 2022 at 08:08, Pradip Parkale <
>> pradip(dot)parkale(at)enterprisedb(dot)com> wrote:
>>
>>> 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?
>>>
>>
>> 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(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
>>>
>>
>>
>> --
>> Dave Page
>> Blog: https://pgsnake.blogspot.com
>> Twitter: @pgsnake
>>
>> EDB: https://www.enterprisedb.com
>>
>>
>
> --
> Thanks & Regards,
> Pradip Parkale
> Software Engineer | EnterpriseDB Corporation
>

--
Dave Page
Blog: https://pgsnake.blogspot.com
Twitter: @pgsnake

EDB: https://www.enterprisedb.com

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Siva N 2022-05-19 10:29:40 [pgAdmin][RM-7347]:Improve color scheme for Authentication type in properties for server
Previous Message Pradip Parkale 2022-05-18 13:02:35 Re: [pgAdmin][RM7351]: Templates not displayed