Update Teams SDK samples in bot-configuration-experience.md#14250
Update Teams SDK samples in bot-configuration-experience.md#14250Pranjal-MSFT wants to merge 8 commits intomainfrom
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as duplicate.
This comment was marked as duplicate.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
Harikrishnan-MSFT
left a comment
There was a problem hiding this comment.
Kindly review the comments on the PR and make the necessary updates.
Acrolinx ScorecardsWe currently enforce a minimum score of 80. Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:
More info about Acrolinx Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally. |
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit 872b3d2: ✅ Validation status: passed
For more details, please refer to the build report. |
| height: 500, | ||
| width: 600, | ||
| title: 'test card', | ||
| # [Python](#tab/Py1) |
There was a problem hiding this comment.
Please update this code snippet
@app.on_config_open
async def handle_config_open(
ctx: ActivityContext[ConfigFetchInvokeActivity]
) -> ConfigInvokeResponse:
card = AdaptiveCard(
body=[
TextBlock(text="Configure your bot", weight="Bolder")
],
actions=[
SubmitAction(title="Submit")
]
)
return ConfigResponse(
config=TaskModuleContinueResponse(
value=CardTaskModuleTaskInfo(
card=card_attachment(AdaptiveCardAttachment(content=card)),
height=500,
width=600,
title="test card",
)
)
)
|
|
||
| ```python | ||
| @app.on_config_submit | ||
| async def handle_config_submit( |
There was a problem hiding this comment.
Update the snippet
@app.on_config_submit
async def handle_config_submit(
ctx: ActivityContext[ConfigSubmitInvokeActivity]
) -> ConfigInvokeResponse:
return ConfigResponse(
config=TaskModuleMessageResponse(
value="You have chosen to finish setting up bot",
)
)
| }); | ||
| ``` | ||
|
|
||
| # [Python](#tab/Py2) |
There was a problem hiding this comment.
Update the code snippets
@app.on_config_open
async def handle_config_open(
ctx: ActivityContext[ConfigFetchInvokeActivity]
) -> ConfigInvokeResponse:
return ConfigResponse(
config=ConfigAuth(
suggested_actions=SuggestedActions(
to=[],
actions=[
CardAction(
type="openUrl",
value="https://example.com/auth",
title="Sign in to this app",
)
],
)
)
)
Acrolinx ScorecardsWe currently enforce a minimum score of 80. Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:
More info about Acrolinx Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally. |
|
Learn Build status updates of commit 96b5ae6: ✅ Validation status: passed
For more details, please refer to the build report. |
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
Acrolinx ScorecardsWe currently enforce a minimum score of 80. Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:
More info about Acrolinx Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally. |
|
Learn Build status updates of commit b1f91c9: ✅ Validation status: passed
For more details, please refer to the build report. |
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
MSFTRickyCastaneda
left a comment
There was a problem hiding this comment.
Implement changes from previous comments
No description provided.