Alexa-skills-kit-sdk-for-nodejs: Bring up to date with 1.0.8 with Device Address Api support

Created on 5 Apr 2017  路  10Comments  路  Source: alexa/alexa-skills-kit-sdk-for-nodejs

It looks like this Github repo has the 1.0.6 version, but NPM has the 1.0.8 version with Device Address API support.

Most helpful comment

@rmtuckerphx Great feedback with timezone. I've sent this feedback internally.

All 10 comments

I am surprised that the Device Address API does not include timezone. All I need is the timezone in a format that can be used with Moment Timezone (https://momentjs.com/timezone/). I don't want the zipcode just to make a call to convert the zipcode to a timezone.

You can use timezone-js to create a timezone object from the date, state, and country. Not sure if that helps, but you still derive the fundamental value that you were seeking.

var x = new timezoneJS.Date('10/31/2008', 'America/New_York');

package: https://www.npmjs.com/package/timezone-js

(If you wanna go really hardcore there's a Google Maps Timezone API. 馃挴 )

IMO Google updates their API's on too frequent of a basis for the code to have the longevity it needs. I hate making any of my work dependent on something they've constructed.

Yeah, I kinda agree - just mentioning it for a bit of a laugh really. Timezones would be nice, Amazon! Think on it.

My bad, I don't catch sarcasm well without facial expressions and tone.

@rmtuckerphx Great feedback with timezone. I've sent this feedback internally.

Absolutely, in my case it would avoid 2 additional calls to Google APIs... the funny part is that all this latency is incurred just because we want to speak back "Today's, Yesterday's" etc, would be nice to know our clients time zone in order to construct meaningful time related phrases.

I am now realizing that this thread is for suggesting implementations, not requesting help on implementations. Sorry guys, I'm too used to StackOverflow. This is my first time participating in a development sequence of this kind on a public forum.

Closing this as we've updated to 1.0.8 with this commit

Was this page helpful?
0 / 5 - 0 ratings