Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Possibility of adding OnLaunch breadcrumb to IntentSequence? #68

Open
BDGLunde opened this issue Jan 28, 2018 · 1 comment
Open

Possibility of adding OnLaunch breadcrumb to IntentSequence? #68

BDGLunde opened this issue Jan 28, 2018 · 1 comment

Comments

@BDGLunde
Copy link

Hello - I have been moving my Alexa skill away from my own custom .NET "plumbing" code and have found your library very easy to work with so far.

My only wish would be for OnLaunch to automatically add a breadcrumb to Session.IntentSequence. There of course other ways I am able to determine if a skill has been interacted with via OnLaunch, but I would love to have something as elegant as simply having it in IntentSequence :)

@stefann42
Copy link
Member

stefann42 commented Feb 2, 2018

Hi @BDGLunde, I'll think about this for the next major revision although I'm not sure that IntentSequence is really the best place to keep track of how the skill got invoked.

My intent with IntentSequence (pun not intended :) ) was not to build a history of actions if you will, but rather to lay down the foundation for conversation management. At the time, my idea was that conversation management can be done using a state machine in your skill and when processing each request you could replay IntentSequence to advance the state machine at the current point in the conversation.

So, given that purpose of IntentSequence, tracking the launch invocation there seems like polluting the data structure because you have to come up with a convention for IntentName for the launch invocation so skill logic knows that it's not actually an intent.

All this however, was before Dialogs were event introduced in ASK. So perhaps there's a newer better mechanism for tracking conversation state in which case IntentSequence could be re purposed to act as a history of user actions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants