Re: [pgAdmin4][RM#3073] Allow user to schedule without End date from UI

From: Dave Page <dpage(at)pgadmin(dot)org>
To: Murtuza Zabuawala <murtuza(dot)zabuawala(at)enterprisedb(dot)com>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: [pgAdmin4][RM#3073] Allow user to schedule without End date from UI
Date: 2018-02-27 15:07:49
Message-ID: CA+OCxowhpCOc8puFNRzxPArGyHg5ZB01C-Cpb0xVGy3h-WWWpQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi

On Tue, Feb 27, 2018 at 12:03 PM, Murtuza Zabuawala <
murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:

> I'm not able to re-produce the issue, Could you hard refresh and try again?
>

I still see it.

>
> If possible could you please provide exact steps?
>

Attached is a dump of my test pgagent schema.

1) Load the schema dump into the maintenance database
2) Connect pgAdmin
3) Browse to pgAgent Jobs
4) Right-click Properties, and select the Schedules tab
5) Open the subnode for sched1

>
> --
> Regards,
> Murtuza Zabuawala
> EnterpriseDB: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
>
> On Tue, Feb 27, 2018 at 5:30 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>
>> Sorry - here it is.
>>
>> On Tue, Feb 27, 2018 at 11:59 AM, Murtuza Zabuawala <
>> murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:
>>
>>> Hi Dave,
>>>
>>> Could you please send screenshot?
>>>
>>> --
>>> Regards,
>>> Murtuza Zabuawala
>>> EnterpriseDB: http://www.enterprisedb.com
>>> The Enterprise PostgreSQL Company
>>>
>>>
>>> On Tue, Feb 27, 2018 at 5:24 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>>
>>>> Hi
>>>>
>>>> Still not quite right - see the attached screenshot which is the result
>>>> of simply viewing the properties of an existing job. Note that the start
>>>> time is shown in the grid but not the subnode control.
>>>>
>>>> Thanks.
>>>>
>>>> On Tue, Feb 27, 2018 at 8:26 AM, Murtuza Zabuawala <
>>>> murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:
>>>>
>>>>> Hi Dave,
>>>>>
>>>>> As validation related patch was committed with RM#3148 [ Sorry about
>>>>> that I forgot to checkout :) ]
>>>>> PFA patch to fix the issues you mentioned, I have also removed extra
>>>>> error message from sub node collection control and made it optional via
>>>>> flag.
>>>>>
>>>>>
>>>>>
>>>>> --
>>>>> Regards,
>>>>> Murtuza Zabuawala
>>>>> EnterpriseDB: http://www.enterprisedb.com
>>>>> The Enterprise PostgreSQL Company
>>>>>
>>>>>
>>>>> On Mon, Feb 26, 2018 at 10:14 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>>>>
>>>>>> Hi
>>>>>>
>>>>>> On Mon, Feb 26, 2018 at 2:46 PM, Murtuza Zabuawala <
>>>>>> murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:
>>>>>>
>>>>>>> Hi,
>>>>>>>
>>>>>>> PFA patch to fix the issue where user was not able to create pgAgent
>>>>>>> job from UI without entering End date in schedule section.
>>>>>>>
>>>>>>
>>>>>> Whilst this does resolve the validation issue, there are still a
>>>>>> couple of other related problems, as can be seen in the attached
>>>>>> screenshots:
>>>>>>
>>>>>> - The Start date/time in the subnode control doesn't seem to be
>>>>>> properly synchronised with the value in the grid.
>>>>>>
>>>>>> - If you leave the End date/time blank (but maybe click into it
>>>>>> first), the grid will show "Invalid date".
>>>>>>
>>>>>> It's possible there are other oddities as well - please check
>>>>>> carefully for anything else.
>>>>>>
>>>>>> Thanks.
>>>>>>
>>>>>> --
>>>>>> Dave Page
>>>>>> Blog: http://pgsnake.blogspot.com
>>>>>> Twitter: @pgsnake
>>>>>>
>>>>>> EnterpriseDB UK: http://www.enterprisedb.com
>>>>>> The Enterprise PostgreSQL Company
>>>>>>
>>>>>
>>>>>
>>>>
>>>>
>>>> --
>>>> Dave Page
>>>> Blog: http://pgsnake.blogspot.com
>>>> Twitter: @pgsnake
>>>>
>>>> EnterpriseDB UK: http://www.enterprisedb.com
>>>> The Enterprise PostgreSQL Company
>>>>
>>>
>>>
>>
>>
>> --
>> Dave Page
>> Blog: http://pgsnake.blogspot.com
>> Twitter: @pgsnake
>>
>> EnterpriseDB UK: http://www.enterprisedb.com
>> The Enterprise PostgreSQL Company
>>
>
>

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

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

Attachment Content-Type Size
pgagent.sql application/octet-stream 40.4 KB

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Murtuza Zabuawala 2018-02-27 15:09:39 Re: [pgAdmin4][RM#3073] Allow user to schedule without End date from UI
Previous Message Murtuza Zabuawala 2018-02-27 15:02:46 [pgAdmin4][RM#3156] Fix PEP-8 issues