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

Updated Readme for Sample Apps #818

Merged
merged 4 commits into from May 6, 2023
Merged

Conversation

microsoftShannon
Copy link
Contributor

Removed default language from supporting site URLs.
Removed typo misdirecting /apps/ to /app/

@adrianwyatt
Copy link
Contributor

I don't see the change for "Removed typo misdirecting /apps/ to /app/"

samples/apps/copilot-chat-app/README.md Show resolved Hide resolved
@shawncal
Copy link
Member

shawncal commented May 6, 2023

I don't see the change for "Removed typo misdirecting /apps/ to /app/"

I found what @microsoftShannon is referring to... this branch had such a change, but someone else fixed it in main and it disappeared with the merge.

@shawncal shawncal requested a review from adrianwyatt May 6, 2023 03:54
@shawncal shawncal dismissed adrianwyatt’s stale review May 6, 2023 03:54

Looked through comments. Tried links. All good.

@shawncal shawncal merged commit af973b9 into microsoft:main May 6, 2023
10 checks passed
codebrain pushed a commit to searchpioneer/semantic-kernel that referenced this pull request May 16, 2023
Removed default language from supporting site URLs.
dehoward pushed a commit to lemillermicrosoft/semantic-kernel that referenced this pull request Jun 1, 2023
Removed default language from supporting site URLs.
golden-aries pushed a commit to golden-aries/semantic-kernel that referenced this pull request Oct 10, 2023
Removed default language from supporting site URLs.
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.

None yet

3 participants