Closed jagtalon closed 7 years ago
YES!
There are many "fuzzy date" functions in the codebase that should be replaced. Some more that I found (in no particular order):
Thanks for looking into that, @mattr555
Looks like tvmaze has been moved to moment.js. Should probably update that checklist above.
Would be nice to have an updated list of pending IAs that need migration (or close this issue if there are no more ;))
From the current list, both game_info
and guidebox seem to have been deleted
@marianosimone Updated the list.
Quick search of the repository https://github.com/duckduckgo/zeroclickinfo-spice/search?utf8=%E2%9C%93&q=getDay the following IA's probably need updating.
@marianosimone thanks for pointing that out! I've removed GameInfo
@MrChrisW thanks for cleaning up as well!
I was going to take a look at the SeatGeek IAs that were highlighted in the search MrChrisW posted above but none of them seem to be working for me. Do they work for anyone else?
I've done a little debugging of the SeatGeek Nearby Concerts IA and it looks to me as though the triggers on the back-end are firing but the JavaScript callback isn't receiving any data.
Client-side I see my browser make a request for _https://duckduckgo.com/js/spice/seat_geek/events_near_me/lat/lon_. If I take this lat/lon pair and manually call the same Seat Geek API endpoint that EventsNearMe.pm does I get many results as expected.
@sekhavati thanks for reporting this! It looks like they've switched their API to always redirect to HTTPS and so we need to update the Perl to reflect that, for each of the SeatGeek IAs. Would you like to do that?
@moollaza Sure, I can take a look. What's the normal protocol, would you like to file an issue first?
@sekhavati I wouldn't say it's required but if you'd like to, that's perfectly fine.
@jagtalon is this issue still relevant?
We recently added support for http://momentjs.com/, and it would be cool if we move things like https://github.com/duckduckgo/zeroclickinfo-spice/blob/master/share/spice/hacker_news/hacker_news.js#L63.
If possible, of course. This might be a huge task that needs a bunch of testing so it's best to do one PR for each IA so that it's easier to review.
Usage of moment.js in DuckDuckGo is as follows: