Skip to content
This repository has been archived by the owner on Jun 30, 2022. It is now read-only.

SkillTemplate changes required for Build #1105

Closed
5 tasks done
darrenj opened this issue Apr 15, 2019 · 0 comments
Closed
5 tasks done

SkillTemplate changes required for Build #1105

darrenj opened this issue Apr 15, 2019 · 0 comments
Assignees
@lauren-mills

Comments

@darrenj
Copy link
Contributor

darrenj commented Apr 15, 2019 •
edited
Loading

We've made a number of key changes to how Skills work so ahead of Build we need to revisit the Skill Template

  • Refactor to MVC approach (perhaps just take VA Bot and lightly adjust
  • Ensure Controller derives from SkillController (and gets manifest information
  • Add PopulateStateFromSkillContext (or similar) method to MainDialog processing to highlight how to retrieve data from SkillContext and populate State
  • Add manifestTemplate.json file to root (tokenize name?)
  • Arm Deployment needs to enable "WebSockets" on the AppService

image
@darrenj darrenj added the Type: Bug Something isn't working label Apr 15, 2019
@darrenj darrenj removed Type: Bug Something isn't working labels Apr 19, 2019
@darrenj darrenj mentioned this issue Apr 23, 2019
Merged
@darrenj darrenj closed this as completed Apr 25, 2019
@lauren-mills lauren-mills removed Type: Suggestion Status: Committed This has been confirmed for the next release. labels Jul 15, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@lauren-mills lauren-mills

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@lauren-mills @darrenj