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

Improve documentation around SessionStart event #5675

Closed
1 task
indam23 opened this issue Apr 20, 2020 · 2 comments · Fixed by #5701
Closed
1 task

Improve documentation around SessionStart event #5675

indam23 opened this issue Apr 20, 2020 · 2 comments · Fixed by #5701
Labels
type:enhancement ✨ Additions of new features or changes to existing ones, should be doable in a single PR

Comments

@indam23
Copy link
Contributor

indam23 commented Apr 20, 2020

Description of Problem:

To make sure a new session is started each time rasa shell is run, /session_start must be sent as the first message manually.
The docs say

a manual session start is triggered with the /session_start intent message.

But usually this is sent automatically by the channel, so on starting rasa shell the user will not see the same initial greeting/welcome message that they might in another channel.

This was pointed out in
#5641

Overview of the Solution:
Add a note in the docs that /session_start needs to be entered manually in rasa shell to see the same behaviour as in another channel configured to send that payload. *Edit: Also note that any first user message sent will trigger action_session_start in shell.

Definition of Done:

  • Documentation updated
@indam23 indam23 added the type:enhancement ✨ Additions of new features or changes to existing ones, should be doable in a single PR label Apr 20, 2020
@kevinthenet
Copy link
Contributor

kevinthenet commented Apr 20, 2020

I'd also reiterate that any user message will trigger a new session to start. It's irrelevant what Rasa gets in the shell because action_session_start will be triggered each time.

@indam23
Copy link
Contributor Author

indam23 commented Apr 20, 2020

That's true, will update it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:enhancement ✨ Additions of new features or changes to existing ones, should be doable in a single PR
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants