Postgres Pro
Downloads
Home   >   mailing lists

Re: [pgAdmin4][RM#3073] Allow user to schedule without End date from UI - Mailing list pgadmin-hackers

From Murtuza Zabuawala
Subject Re: [pgAdmin4][RM#3073] Allow user to schedule without End date from UI
Date
Msg-id CAKKotZQ_QLJ+pgz0Q8Tjg3-oOezG7OTUXDSgg_SYXm2pX4itgQ@mail.gmail.com
Whole thread Raw
In response to Re: [pgAdmin4][RM#3073] Allow user to schedule without End date from UI  (Dave Page <dpage@pgadmin.org>)
Responses Re: [pgAdmin4][RM#3073] Allow user to schedule without End date from UI  (Dave Page <dpage@pgadmin.org>)
List pgadmin-hackers
Tree view
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@pgadmin.org> wrote:
Hi

On Mon, Feb 26, 2018 at 2:46 PM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.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

Attachment

pgadmin-hackers by date:

Previous
From: Harshal Dhumal
Date:
Subject: PEP-8 fixes
Next
From: Dave Page
Date:
Subject: pgAdmin 4 commit: Case sensitive paths are confusing git...