[BUG] displaying Program next run date

Follow

Makoto Schoppert

May 08, 2017 21:56

Looks like the value for "Next run on ..." is incorrect.  The graph properly shows it as "May 10", however, the text displays Tue May 09.

There is a JavaScript function called "Util.dateStringToLocalDate()" that takes in a string value and converts it to a date.  The string value that is being passed in is "2017-05-10".  

The returned Date object is "Tue May 09 2017 20:00:00 GMT-0400 (Eastern Daylight Time)"

Perhaps the function can be modified as this:

dateStringToLocalDate(dateString){
   if(null === dateString){
       return null;
   } else{
       var dt = new Date(dateString);
       dt.setTime(dt.getTime() + dt.getTimezoneOffset()*60*1000);
       return dt;
   }
}

1

• 

  RainMachine Nicholas May 09, 2017 17:44

  Thanks Makoto, we're in the process of moving all date stuff in web ui to device date/time so we no longer depend on browser timezone.

  1

  Comment actions Permalink
• 

  Makoto Schoppert May 09, 2017 20:46

  Sounds great Nicholas! Looking forward to the next update.  I have been quite satisfied with the product, and as a developer myself, I appreciate the extensiblity of the device through the API, as well as the developer documentation that's available.

  0

  Comment actions Permalink

Please sign in to leave a comment.