dialogflow / dialogflow-dotnet-client

.NET framework for Dialogflow
Apache License 2.0
69 stars 50 forks source link

QuestionMetadata.SessionId is marked as Internal #9

Closed Zhaph closed 7 years ago

Zhaph commented 7 years ago

According to the /query documentation, sessionId is required and enables us to link messages together into a single session - is there a reason why the SessionId property is marked as internal?

Shouldn't a consumer be able to set this to some conversation specific id to link queries and contexts together?

xVir commented 7 years ago

Hi @Zhaph You could set SessionId using AIConfiguration. In AIDataService and QuestionMetadata it made as read-only to prevent unintentional changing of the field.

Thanks!

Zhaph commented 7 years ago

Thanks for that, I'll give it a try and let you know.

Cheers,

Ben

Zhaph commented 7 years ago

@xVir Thanks, that does indeed appear to work as I need.

xVir commented 7 years ago

Thanks, @Zhaph!