Pat Brown
Pat Brown
I agree, it shouldn't be necessary and it would be quite helpful if `@neo4j_ignore` was added to the documentation.
Until we move the cycling profile to valhalla, it's unlikely we won't be taking any feature requests. Right now that migration is targeted for Q4 by @stari4ek's team. Love the...
Where does `Locale.SIMPLIFIED_CHINESE.language` come from? `language=zh` is incorrect. It should be `language=cmn` https://api.mapbox.com/voice/v1/speak/%3Cspeak%3E%3Camazon:effect%20name=%22drc%22%3E%3Cprosody%20rate=%221.08%22%3E%E5%87%BA%E5%8F%91%E5%90%91%E5%8C%97%EF%BC%8C%E9%A9%B6%E5%85%A5Main%20Street%EF%BC%8C%E9%9A%8F%E5%90%8E%E5%B7%A6%E8%BD%AC%EF%BC%8C%E9%A9%B6%E5%85%A5West%20%3Csay-as%20interpret-as=%22address%22%3E19th%3C%2Fsay-as%3E%20Street%3C%2Fprosody%3E%3C%2Famazon:effect%3E%3C%2Fspeak%3E?textType=ssml&language=cmn&access_token=
Our documentation looks to be incorrect. `cmn` comes from AWS docs - https://docs.aws.amazon.com/polly/latest/dg/ph-table-mandarin.html
> l. Having to pass a different language value (cmn) for the speech API doesn't make sense IMO. I agree and I would expect `Locale.SIMPLIFIED_CHINESE` to work. I'll cut a...
This would be a great feature to pull!
I think the benefit of this plugin is in it's simplicity and not needing any flags like @cgross mentioned above. If you want to go the boolean route, why not...
Absolutely. I'll note that for the future.
@markgoodyear I updated the code spacing and used gulp to build the dist folder. Also merged your latest version in. Is there anything else I need to do for you...