Skip to content
New issue

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

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

Already on GitHub? # to your account

Fix update dialogue #14

Open
wants to merge 44 commits into
base: dev
Choose a base branch
from

Conversation

NeonClary
Copy link
Member

Dialogue changes. One with a simple code insertion.

@NeonDaniel
Copy link
Member

NeonDaniel commented Nov 16, 2022

Some dialog bugs I found in testing:

 >> Starbucks is closed now. It opens at 30.
 >> I found Starbucks with hours 10:from 30 A M to 8 P M. You can find this store on Street Level one, in the Ewa Wing.
 >> What store you are looking for?

:param message: Message associated with user utterance
"""
answer = self.get_response('how_can_i_help_you')
user_request, mall_link = self.user_request_handling(message, answer)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rather than answer, we could just check `message.data.get('utterances')[0]

__init__.py Outdated
Handle any input from a user interacting with the kiosk.
:param message: Message associated with user utterance
"""
answer = message.data['utterances']
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this should be message.data['utterances'][0] so you have a string instead of a list of strings

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants