-
Notifications
You must be signed in to change notification settings - Fork 870
Readme for the bot-proactive-messaging-teamsfx sample has been updated. #1726
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
base: main
Are you sure you want to change the base?
Conversation
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.
Pull Request Overview
This PR updates the README for the bot-proactive-messaging-teamsfx
sample by adding metadata, reorganizing usage instructions, and including demo manifest links and images.
- Added front-matter metadata and demo manifest section
- Reorganized "Run the app" and setup instructions with clearer step-by-step guidance
- Included animated GIFs and screenshots for improved clarity
- [NodeJS](https://nodejs.org/en/) | ||
- An M365 account. If you do not have M365 account, apply one from [M365 developer program](https://developer.microsoft.com/en-us/microsoft-365/dev-program) | ||
- [Microsoft 365 Agents Toolkit Visual Studio Code Extension](https://aka.ms/teams-toolkit) version after 1.55 or [Microsoft 365 Agents Toolkit CLI](https://aka.ms/teams-toolkit-cli) | ||
This sample provides following functionality: |
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.
Consider adding the definite article "the": change to "This sample provides the following functionality:".
This sample provides following functionality: | |
This sample provides the following functionality: |
Copilot uses AI. Check for mistakes.
- [Microsoft 365 Agents Toolkit Visual Studio Code Extension](https://aka.ms/teams-toolkit) version after 1.55 or [Microsoft 365 Agents Toolkit CLI](https://aka.ms/teams-toolkit-cli) | ||
This sample provides following functionality: | ||
|
||
- You can send any command to bot to get the same message echoed back. |
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.
Add "the" before "bot": change to "send any command to the bot".
- You can send any command to bot to get the same message echoed back. | |
- You can send any command to the bot to get the same message echoed back. |
Copilot uses AI. Check for mistakes.
1) __*This step is specific to Teams.*__ | ||
- **Edit** the `manifest.json` contained in the `appManifest` folder to replace your Microsoft App Id (that was created when you registered your bot earlier) *everywhere* you see the place holder string `<<YOUR-MICROSOFT-APP-ID>>` (depending on the scenario the Microsoft App Id may occur multiple times in the `manifest.json`) | ||
- **Edit** the `manifest.json` for `validDomains` with base Url domain. E.g. if you are using ngrok it would be `https://1234.ngrok-free.app` then your domain-name will be `1234.ngrok-free.app` and if you are using dev tunnels then your domain will be like: `12345.devtunnels.ms`. | ||
- **Zip** up the contents of the `appManifest` folder to create a `manifest.zip` (Make sure that zip file does not contains any subfolder otherwise you will get error while uploading your .zip package) |
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.
Correct grammar: replace "does not contains" with "does not contain".
- **Zip** up the contents of the `appManifest` folder to create a `manifest.zip` (Make sure that zip file does not contains any subfolder otherwise you will get error while uploading your .zip package) | |
- **Zip** up the contents of the `appManifest` folder to create a `manifest.zip` (Make sure that zip file does not contain any subfolder otherwise you will get error while uploading your .zip package) |
Copilot uses AI. Check for mistakes.
|
||
## Play with the App | ||
Message the bot and it will respond with a proactive hello. Add a message reaction to the bots response, and the bot will reply accordingly. |
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.
Possessive apostrophe missing: change "bots response" to "bot's response".
Message the bot and it will respond with a proactive hello. Add a message reaction to the bots response, and the bot will reply accordingly. | |
Message the bot and it will respond with a proactive hello. Add a message reaction to the bot's response, and the bot will reply accordingly. |
Copilot uses AI. Check for mistakes.
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.
Need to discuss on this.
|
||
## Edit the manifest | ||
## Run the app (Using Microsoft 365 Agents Toolkit for Visual Studio Code) |
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.
We've added this as a new item—can we format it similarly to the README of the other project?
No description provided.