Missing an hour in the hourly report

[Ryan Williams]

What's the current situation?

Missing an hour in the hourly report for https://wxkb.io/sacramento

What do you recommend?

Change "in 5 hours and in 7 hours" to "in 5 hours until 7 hours"

[Julian Norton]

weather-10kb-wxkb/views/partials/hourly.ejs

Line 31 in df24e95

             <%= data.hourly[i].startTime ? moment.unix(data.hourly[i].startTime).fromNow() + ' and ' : '' %><%= hourlyTime.fromNow()%><% if (params.hoursFormat === 'h') { %> 

[Julian Norton]

https://github.com/JulianNorton/weather-10kb-wxkb/blob/master/views/partials/hourly.ejs

[Julian Norton]

fixed in bc1a5c0

[Julian Norton]

@modelm do you know how to remove the second "in"?

bc1a5c0

[Julian Norton]

it's pulling from the same partial

[Ryan Williams]

That's coming from hourlyTime.fromNow() - one way would be something like hourlyTime.fromNow().replace(/in /, '')

[Ivan]

@JulianNorton @modelm If you want I can take this.I can try to normalize all the the strings that come from momentjs

Let me know your thoughts.

[Julian Norton]

Please do Ivan!

…

On Oct 3, 2019, at 6:29 AM, Ivan ***@***.***> wrote:  @JulianNorton @modelm If you want I can take this.I can try to normalize all the the strings that come from momentjs Let me know your thoughts. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.