-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Documentation improvements: Remove duplicate READMEs, consistent titles #2118
Conversation
docs/data_ingestion.md
Outdated
"search": "*", | ||
"count": true, | ||
"top": 1, | ||
"filter": "sourcefile eq '209884Orig1s000RiskR.pdf'", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you change the title of this PDF please
docs/data_ingestion.md
Outdated
"search": "*", | ||
"count": true, | ||
"top": 1, | ||
"filter": "sourcefile eq '209884Orig1s000RiskR.pdf'", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"filter": "sourcefile eq '209884Orig1s000RiskR.pdf'", | |
"filter": "sourcefile eq 'employee_handbook.pdf'", |
docs/data_ingestion.md
Outdated
"search": "*", | ||
"count": true, | ||
"top": 1, | ||
"filter": "sourcefile eq '209884Orig1s000RiskR.pdf'", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"filter": "sourcefile eq '209884Orig1s000RiskR.pdf'", | |
"filter": "sourcefile eq 'employee_handbook.pdf'", |
Check Broken URLsWe have automatically detected the following broken URLs in your files. Review and fix the paths to resolve this issue. Check the file paths and associated broken URLs inside them. For more details, check our Contributing Guide.
|
Broken links are due to planned techcommunity upgrade, so those are expected. |
Check Broken URLsWe have automatically detected the following broken URLs in your files. Review and fix the paths to resolve this issue. Check the file paths and associated broken URLs inside them. For more details, check our Contributing Guide.
|
Purpose
The Microsoft Learn samples browser now lets us put the YAML inside a comment, so we don't need the duplicate docs anymore. This PR removes them and generally makes the docs more consistent by prefacing each with "RAG chat: ".
This PR also changes the main title to RAG, to match the short title used on other pages.
Does this introduce a breaking change?
When developers merge from main and run the server, azd up, or azd deploy, will this produce an error?
If you're not sure, try it out on an old environment.
Does this require changes to learn.microsoft.com docs?
This repository is referenced by this tutorial
which includes deployment, settings and usage instructions. If text or screenshot need to change in the tutorial,
check the box below and notify the tutorial author. A Microsoft employee can do this for you if you're an external contributor.
Type of change
Code quality checklist
See CONTRIBUTING.md for more details.
N/A