From: | Murtuza Zabuawala <murtuza(dot)zabuawala(at)enterprisedb(dot)com> |
---|---|
To: | Dave Page <dpage(at)pgadmin(dot)org> |
Cc: | Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: PATCH: Added "Named restore point" functionality (pgAdmin4) |
Date: | 2016-05-12 05:17:40 |
Message-ID: | 2E91A09E-7977-4081-8F7A-EFEE1E0CA908@enterprisedb.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi Dave,
If user provides empty name then we are already throwing an error.
Do you still want me to disable button?
Regards,
Murtuza
> On 11-May-2016, at 8:45 pm, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>
> Committed as is, but please submit a patch to disable the OK button,
> until a name has been entered, to avoid accepting an empty name.
>
> Thanks!
>
> On Wed, May 11, 2016 at 11:59 AM, Murtuza Zabuawala
> <murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:
>> Hi,
>>
>> PFA updated patch with i18n support added in message.
>>
>> Regards,
>> Murtuza
>>
>> --
>> Regards,
>> Murtuza Zabuawala
>> EnterpriseDB: http://www.enterprisedb.com
>> The Enterprise PostgreSQL Company
>>
>> On Wed, May 11, 2016 at 3:50 PM, Murtuza Zabuawala
>> <murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:
>>>
>>> Hi,
>>>
>>> PFA updated patch for named restore point and I have also updated reload
>>> configuration menu enable/disable condition as mentioned.
>>>
>>> Regards,
>>> Murtuza
>>>
>>> --
>>> Regards,
>>> Murtuza Zabuawala
>>> EnterpriseDB: http://www.enterprisedb.com
>>> The Enterprise PostgreSQL Company
>>>
>>> On Wed, May 11, 2016 at 2:03 PM, Ashesh Vashi
>>> <ashesh(dot)vashi(at)enterprisedb(dot)com> wrote:
>>>>
>>>> On Wed, May 11, 2016 at 1:11 PM, Murtuza Zabuawala
>>>> <murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:
>>>>>
>>>>> Hi All,
>>>>>
>>>>>
>>>>> PFA patch which will add "Adding named restore point" functionality on
>>>>> server.
>>>>
>>>> Create named restore point is action restricted to the superuser only.
>>>> Please check that in the menu enable/disable functionality.
>>>>
>>>> The check also applicable to 'relaod configuration'.
>>>>>
>>>>>
>>>>> --
>>>>> Regards,
>>>>> Murtuza Zabuawala
>>>>> EnterpriseDB: http://www.enterprisedb.com
>>>>> The Enterprise PostgreSQL Company
>>>>>
>>>>>
>>>>> --
>>>>> Sent via pgadmin-hackers mailing list (pgadmin-hackers(at)postgresql(dot)org)
>>>>> To make changes to your subscription:
>>>>> http://www.postgresql.org/mailpref/pgadmin-hackers
>>>>>
>>>>
>>>
>>
>>
>>
>> --
>> Sent via pgadmin-hackers mailing list (pgadmin-hackers(at)postgresql(dot)org)
>> To make changes to your subscription:
>> http://www.postgresql.org/mailpref/pgadmin-hackers
>>
>
>
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Ashesh Vashi | 2016-05-12 05:27:36 | Re: PATCH: Added "Named restore point" functionality (pgAdmin4) |
Previous Message | Surinder Kumar | 2016-05-11 19:53:51 | [pgAdmin4][Patch]: Load/Save file in query tool |