Missing an hour in the hourly report

[modelm]

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"

[JulianNorton]

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

[JulianNorton]

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

[JulianNorton]

fixed in https://github.com/JulianNorton/weather-10kb-wxkb/commit/bc1a5c0b953393583c19f8a068693f93f6d2bf23

[JulianNorton]

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

https://github.com/JulianNorton/weather-10kb-wxkb/commit/bc1a5c0b953393583c19f8a068693f93f6d2bf23

[JulianNorton]

it's pulling from the same partial

[modelm]

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

[ivanfrias]

@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.

[JulianNorton]

Please do Ivan!

On Oct 3, 2019, at 6:29 AM, Ivan [email protected] 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.