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

Modified SDK for Quickstart Playground - Chat Style #3908

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

chenss3
Copy link
Contributor

@chenss3 chenss3 commented Sep 28, 2024

Summary of the pull request

Introduces SDK changes needed for the Chat Style integration of Quickstart Playground

References and relevant issues

Detailed description of the pull request / Additional comments

  • Does not modify the existing Quickstart playground provider and creates a new one that we will then modify
  • Adds QuickStartChatStyleResult response object

Validation steps performed

PR checklist

  • Closes #xxx
  • Tests added/passed
  • Documentation updated

// This allows to have an async operation can be started but also additional context reported back to Dev Home
// as progress is being made such as a detailed log out output through the adaptive card session.
[experimental]
[contract(Microsoft.Windows.DevHome.SDK.DevHomeContract, 6)]
Copy link
Member

Choose a reason for hiding this comment

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

Do we need to increment contract version throughout.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sarah stated in a previous thread: If the contract version is being incremented here, we may also need to change the VersionOfSDK value inside build/azure-pipelines.yml.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I believe I saw the current contract version is 7, so I updated all of the new ones I'm adding in to be contract version 8.

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

Successfully merging this pull request may close these issues.

3 participants