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

Finish PR: SignalR with Open AI #34636

Open
Rick-Anderson opened this issue Feb 4, 2025 · 5 comments · May be fixed by #34848
Open

Finish PR: SignalR with Open AI #34636

Rick-Anderson opened this issue Feb 4, 2025 · 5 comments · May be fixed by #34848
Assignees
@wadepickett
Labels
seQUESTered Identifies that an issue has been imported into Quest. Source - Docs.ms Docs Customer feedback via GitHub Issue

Comments

@Rick-Anderson
Copy link
Contributor

Rick-Anderson commented Feb 4, 2025 •
edited by sequestor bot
Loading

Description

@wadepickett pull in the commits in #33448 to your PR and finish this off. Please schedule for Feb or March

Manual port to ADO https://dev.azure.com/msft-skilling/Content/_workitems/edit/402618 and set label to seQuestered. -wp
(reQuest was not working at the time.)

Page URL

https://review.learn.microsoft.com/en-us/aspnet/core/tutorials/ai-powered-group-chat/ai-powered-group-chat?view=aspnetcore-8.0&branch=pr-en-us-33448&tabs=open-ai

Content source URL

https://github.com/dotnet/AspNetCore.Docs/blob/main/aspnetcore/tutorials/ai-powered-group-chat/ai-powered-group-chat.md

Document ID

7bb9cdfd-b2c0-439f-3e65-fde31362724c

Article author

@kevinguo-ed

Metadata

  • ID: 7bb9cdfd-b2c0-439f-3e65-fde31362724c
  • Service: aspnet-core
  • Sub-service: tutorials

Associated WorkItem - 402815
@Rick-Anderson Rick-Anderson added ⌚ Not Triaged Source - Docs.ms Docs Customer feedback via GitHub Issue labels Feb 4, 2025
@Rick-Anderson
Copy link
Contributor Author

@kevinguo-ed I've assigned @wadepickett to get #34636 merged. He'll pull in your commits to his PR so when it's merged you'll get credit. Hopefully you can answer any questions he has.

@kevinguo-ed
Copy link

kevinguo-ed commented Feb 5, 2025 via email

@wadepickett
Copy link
Contributor

wadepickett commented Feb 5, 2025 •
edited
Loading

@Rick-Anderson,
you said: "Move it to the samples section." Where is the "samples section" you want it in?

Maybe you were thinking of the TOC item for SignalR samples...

Image

...that points directly to the sample repo here:
https://github.com/aspnet/SignalR-samples

The sample for the doc in the PR is located here:
https://github.com/microsoft/SignalR-Samples-AI/tree/main/AIStreaming

Is the intent that the content in the PR would become a README type format for the sample and reside with the sample? Or it would become a README in /aspnet/SignalR-samples but point to microsoft/SignalR-Samples-AI/tree/main/AIStreaming.

Thanks for any clarification.

@wadepickett wadepickett moved this from 🔖 Ready to 🏗 In progress in dotnet/AspNetCore.Docs February 2025 sprint Feb 6, 2025
@dotnetrepoman dotnetrepoman bot added the mapQuest clean move label Feb 6, 2025
@dotnet-policy-service dotnet-policy-service bot removed the mapQuest clean move label Feb 6, 2025
@Rick-Anderson
Copy link
Contributor Author

@wadepickett see move sample request

@wadepickett
Copy link
Contributor

wadepickett commented Feb 7, 2025 •
edited
Loading

@wadepickett see move sample request

Yes. Mike's comment was not clear on this either. Since you were saying the same thing, I thought you might be able to clarify for my specific questions above. I will verify with Mike instead, but if you happen to know that's great.

@wadepickett wadepickett linked a pull request Feb 27, 2025 that will close this issue
Draft
@wadepickett wadepickett added the reQUEST Triggers an issue to be imported into Quest label Mar 1, 2025
@wadepickett wadepickett moved this from 🔖 Ready to 🏗 In progress in dotnet/AspNetCore.Docs March 2025 sprint Mar 5, 2025
@dotnetrepoman dotnetrepoman bot added the mapQuest clean move label Mar 5, 2025
@dotnet-policy-service dotnet-policy-service bot removed the mapQuest clean move label Mar 5, 2025
@wadepickett wadepickett added seQUESTered Identifies that an issue has been imported into Quest. and removed reQUEST Triggers an issue to be imported into Quest labels Mar 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@wadepickett wadepickett

Labels
seQUESTered Identifies that an issue has been imported into Quest. Source - Docs.ms Docs Customer feedback via GitHub Issue
Projects
dotnet/AspNetCore.Docs March 2025 sprint
Status: 🏗 In progress
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

SignalR with Open AI: Finish as Sample Guide dotnet/AspNetCore.Docs
3 participants
@Rick-Anderson @wadepickett @kevinguo-ed