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

Intelligent apps - Chat app - new arch image and text #39913

Closed
wants to merge 5 commits into from
Closed

Intelligent apps - Chat app - new arch image and text #39913

wants to merge 5 commits into from

Conversation

diberry
Copy link
Contributor

@diberry diberry commented Mar 12, 2024 •
edited by github-actions bot
Loading

@dotnet-bot dotnet-bot added this to the March 2024 milestone Mar 12, 2024
@dotnet-policy-service dotnet-policy-service bot added the community-contribution Indicates PR is created by someone from the .NET community. label Mar 12, 2024
IEvangelist
IEvangelist approved these changes Mar 15, 2024
View reviewed changes
Copy link
Member

@IEvangelist IEvangelist left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Looks good! But we'll need to address the merge conflict.

IEvangelist
IEvangelist reviewed Mar 15, 2024
View reviewed changes
docs/azure/ai/get-started-app-chat-template.md
Copy link
Member

Choose a reason for hiding this comment

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

Is this article not in the TOC?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The AI folder structure changed after I created this PR.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It think it will be easier to just create a new PR. I've already had to deal with the folder structure change breaking link with my PMs enough yesterday.

@diberry diberry mentioned this pull request Mar 15, 2024
Merged
@diberry diberry closed this Mar 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@IEvangelist IEvangelist IEvangelist approved these changes

@CamSoper CamSoper CamSoper approved these changes

@alexwolfmsft alexwolfmsft Awaiting requested review from alexwolfmsft alexwolfmsft is a code owner

Assignees
No one assigned
Labels
community-contribution Indicates PR is created by someone from the .NET community. dotnet-azure/svc
Projects
None yet
Milestone
March 2024
Development

Successfully merging this pull request may close these issues.
4 participants
@diberry @CamSoper @IEvangelist @dotnet-bot