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

Conversation

@litofish
Copy link
Contributor

@litofish litofish commented Jul 10, 2019

Purpose

Adding a new hospitality skill. Wanted to get feedback as I add more dialogs. Have finished a number of the main dialogs for the skill.

Changes

Features added so far:

  • Checking out
  • Checking out late
  • Get reservation details
  • Extending stay

Feature Plan

Features I'm working on:

  • Hotel FAQs with QnA maker
  • Requesting room amenities

Later extensions/connecting other skills:

  • Room controls (lights, temp, etc.) and playing music
  • Point of interest
  • Restaurant booking
  • Room service

@pr-triage pr-triage bot added the PR: draft label Jul 10, 2019
@ryanisgrig
Copy link
Collaborator

Thank you for raising a pull request. Please ensure you have have completed the following:

General

  • I have commented my code, particularly in hard-to-understand areas
  • I have added or updated the appropriate tests
  • I have updated related documentation

Bots

  • I have validated that new and updated responses use appropriate Speak and InputHint properties to ensure a high-quality speech-first experience
  • I have replicated language model changes across the English, French, Italian, German, Spanish, and Chinese .lu files and validated that deployment is successful

Deployment Scripts

  • I have replicated my changes in the Virtual Assistant Template and Sample projects
  • I have replicated my changes in the Skill Template and Sample projects

@ryanisgrig

This comment has been minimized.

@ryanisgrig

This comment has been minimized.

@litofish litofish changed the base branch from master to next July 31, 2019 20:14
@litofish litofish requested a review from lzc850612 August 1, 2019 22:50
@litofish litofish merged commit bc68f15 into next Aug 2, 2019
@delete-merged-branch delete-merged-branch bot deleted the luna/hospitality branch August 2, 2019 04:55
@litofish litofish restored the luna/hospitality branch August 2, 2019 04:55
@litofish litofish deleted the luna/hospitality branch August 2, 2019 16:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants