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.

Sign up for GitHub

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

Jump to bottom

Feature/accumulate response before send to server #168

Merged
merged 25 commits into from
Mar 22, 2018
Merged

Feature/accumulate response before send to server #168

merged 25 commits into from
Mar 22, 2018

Conversation

hyzhak
Copy link
Member

@hyzhak hyzhak commented Mar 22, 2018

@hyzhak
add render speech middleware
2569653
@hyzhak
middleware render speeche could render array of speeches
b13e98a
@hyzhak
render speech middleware should do nothing when we don't have speeches
846c825
@hyzhak
use render speech middleware in music query pipeline
a26ce62
@hyzhak
check that fulfil resolver middleware fulfil array of speeches
ddd7317
@hyzhak
remove useless assignment
7864e45
@hyzhak
template resolver could parse empty array
b26d8b9
@hyzhak
fulfil resolvers should parse speech array
8010177
@hyzhak
passed state though music query
8bd8b73
@hyzhak
add ask middleware
04f77f4
@hyzhak
create middleware substitute acknowledge
f1770c1
@hyzhak
split fetch suggestions and generate prompts middlewares
2649df7
@hyzhak
rename promptScheme to suggestionsScheme
ca860a1
@hyzhak
add prompt substitution middleware
fe2943e
@hyzhak
use prompt middleware for music query pipeline
76ac7a5
@hyzhak
fix template selector
cb7122c
@hyzhak
acknowledge middleware should pass collected slots
8436b36
@hyzhak
add suggestions suggestion middleware
96c1e00
@hyzhak
set music query pipeline middlewares
5586fc6
@hyzhak
add missed app for when we answer that haven't find anything
440362d
@hyzhak
start all middlewares with 'start' debug message
1186ccb
@hyzhak
warn when we have nothing to ask in response
0d7c00a
@hyzhak
use playback middleware for playing music query
e6267d0
@hyzhak
use prompts.speech instead of prompts.prompts
bb5c2e2
@hyzhak
use prompts.confirm instead prompts.requirements
017a142
@hyzhak hyzhak merged commit 404294a into master Mar 22, 2018
@hyzhak hyzhak deleted the feature/accumulate-response-before-send-to-server branch March 22, 2018 21:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@hyzhak