Skip to content
This repository has been archived by the owner on Apr 13, 2023. It is now read-only.

Missing an hour in the hourly report #209

Open
modelm opened this issue Jun 12, 2019 · 8 comments
Open

Missing an hour in the hourly report #209

modelm opened this issue Jun 12, 2019 · 8 comments

Comments

@modelm
Copy link
Collaborator

modelm commented Jun 12, 2019

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"

image

@JulianNorton
Copy link
Owner

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

@JulianNorton
Copy link
Owner

fixed in bc1a5c0

@JulianNorton
Copy link
Owner

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

bc1a5c0

@JulianNorton
Copy link
Owner

JulianNorton commented Jun 12, 2019

Screen Shot 2019-06-12 at 9 35 43 AM

it's pulling from the same partial

@modelm
Copy link
Collaborator Author

modelm commented Jun 12, 2019

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

@ivanfrias
Copy link

@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
Copy link
Owner

JulianNorton commented Oct 3, 2019 via email

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants