-
Notifications
You must be signed in to change notification settings - Fork 705
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
Sample bot-sso-adaptivecard still using botframeworkadapter which is deprecated #1039
Comments
@KennethBWSong - Thank you for reporting this, we will check this and get the sample updated if necessary. Thank you. PR has raised for this: https://github.com/OfficeDev/Microsoft-Teams-Samples/pull/1042/files |
Any updates? |
PR has raised for this: https://github.com/OfficeDev/Microsoft-Teams-Samples/pull/1042/files. Changes will reflect once the PR is merged. |
@Meghana-MSFT Seems the PR is merged, and I got following error when debug with Teams Toolkit: |
@Meghana-MSFT Also, in this sample to get SSO token: |
We will check this and get back to you. |
This is a known issue, bug has been raised for this already - MicrosoftDocs/msteams-docs#8584 |
@Meghana-MSFT Any updates on this? |
Hello @KennethBWSong - Sorry for delay in response. |
Any update on this? The samples are really hard to follow in general, let alone when they contain wrong code. |
Code here:
Microsoft-Teams-Samples/samples/bot-sso-adaptivecard/nodejs/index.js
Line 24 in 3a30497
The text was updated successfully, but these errors were encountered: