Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Wrong day of the week shown for next game's date. #8

Open
jason37 opened this issue Apr 19, 2012 · 2 comments
Open

Wrong day of the week shown for next game's date. #8

jason37 opened this issue Apr 19, 2012 · 2 comments

Comments

@jason37
Copy link

jason37 commented Apr 19, 2012

Take today's (19 April 2012) message for example: Giants will play the Mets on Wednesday 4:10 pm at Citi Field. Game is actually Friday, other items are correct. The day of the week from the last date in the json data is used.

[shameless plug]
BTW check out my fork http://istherearedsoxgametoday.com !

@altheadotspace
Copy link
Contributor

I'm seeing this on my Rockies fork too. (There is a game on Friday, but it outputs "Wednesday.") Currently looking for problems in the if (!nextGame && isDateLaterThan(nextGameDate, today)) loop.

@jason37 , I see your Red Sox fork is properly reporting the Yankees date as Friday. Have you implemented some kind of fix?

@jason37
Copy link
Author

jason37 commented Apr 19, 2012

@zekeweeks before the commented line:

// Formate next game date as day of the week

I added:
var nextGameDate = new Date(nextGame.date);

not sure if that is the best way to fix it but at least it does.

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

No branches or pull requests

2 participants