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

From: Pradip Parkale <pradip(dot)parkale(at)enterprisedb(dot)com>
To: Dave Page <dpage(at)pgadmin(dot)org>
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:02:35
Message-ID: CAJ9T6StD7gsTg3qfUDY3FWKPphbbuTGyyM+Di=Ov6QK7F10BLw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

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.

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

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Dave Page 2022-05-18 13:41:12 Re: [pgAdmin][RM7351]: Templates not displayed
Previous Message Akshay Joshi 2022-05-18 12:21:03 Re: [pgAdmin][RM7364]: Allow resizing of columns on dashboard tables