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 | CAKKotZS6ExcFqt06DSsHLYbBanbhPZtjzYcbYJJUUwCzBvRmiQ@mail.gmail.com Whole thread Raw |
In response to | Re: [pgAdmin4][RM#3073] Allow user to schedule without End date from UI (Joao De Almeida Pereira <jdealmeidapereira@pivotal.io>) |
List | pgadmin-hackers |
Hi Joao,
It is difficult to test scenario with unit test as html input source has proper date value in both the cases but bootstrap datetime picker itself was not displaying the value on the UI.
I'm attaching the screenshot for Start field (Both the scenario when it displays with Valid date and with invalid date when it doesn't).
Let me know if you have anything in mind.
--
Regards,
On Wed, Feb 28, 2018 at 8:40 PM, Joao De Almeida Pereira <jdealmeidapereira@pivotal.io> wrote:
Hello Murtuza,I do not have the pgAgent installed so it was a little it hard to test this.After looking into the code I think we are missing some testing coverage around the pga_job Javascript part so that we can catch these problems.Even some code extraction can be done around the pg_jobset changeNevertheless I passed the patch through our CI and it is all greenThanksJoaoOn Wed, Feb 28, 2018 at 5:36 AM Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote: Hi Dave,I have found the issue, it was in Backform control itself :)Issue: We were passing today's date value as minDate option in pgAgent schedule schema while rendering the control, So when server was sending previous datetime value it was not displaying it causing sync problem.--Regards,On Tue, Feb 27, 2018 at 8:39 PM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote: Thanks Dave, I'll look into it.On Tue, Feb 27, 2018 at 8:37 PM, Dave Page <dpage@pgadmin.org> wrote:HiOn Tue, Feb 27, 2018 at 12:03 PM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.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 database2) Connect pgAdmin3) Browse to pgAgent Jobs4) Right-click Properties, and select the Schedules tab5) Open the subnode for sched1--Regards,On Tue, Feb 27, 2018 at 5:30 PM, Dave Page <dpage@pgadmin.org> wrote:Sorry - here it is.On Tue, Feb 27, 2018 at 11:59 AM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote: Hi Dave,Could you please send screenshot?--Regards,On Tue, Feb 27, 2018 at 5:24 PM, Dave Page <dpage@pgadmin.org> wrote:HiStill 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.zabuawala@enterprisedb.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,On Mon, Feb 26, 2018 at 10:14 PM, Dave Page <dpage@pgadmin.org> wrote:HiOn 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--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
pgadmin-hackers by date:
Previous
From: Khushboo VashiDate:
Subject: Re: pgAdmin 4 commit: Ensure we pick up the messages from the current query
Next
From: Khushboo VashiDate:
Subject: [pgAdmin4][Patch]: RM #3135 - [Web based] Syntax error displayed whenuser try to insert data on table where primray key is in captial letters andtable contains OIDS