search

AreYouFreeBusy / AlexaSkillsKit.NET

.NET library that simplifies Alexa skills development; same object model as Amazon's AlexaSkillsKit for Java
MIT License
212 stars 100 forks source link

Person object #87

Open shahab-a-k opened 3 years ago

shahab-a-k commented 3 years ago

Hello Team,

Is any update on the way to include person object? in context.system property?

ElvenMonky commented 3 years ago

Hi @shahab-a-k ,

Thanks for the PRs (#88 and #89)! Both PR look good and change doesn't seem to be breaking. Let me see if there were any other fields introduced since library was last published... Ok, I don't see any https://developer.amazon.com/en-US/docs/alexa/custom-skills/request-and-response-json-reference.html#request-body-syntax

ElvenMonky commented 3 years ago

Ok, I merged both PRs and now the question is: do you want us to release new nuget version with this changes? I assume it would be minor version incremented, as it affects request body format and some skills might not expect this change.

shahab-a-k commented 3 years ago

It's upto you, currently I'm working more in depth with it. I'm not sure, but I might add a few more enhancements and apply a pull request.