8000 Support SSO with skill dialog and expected replies by tracyboehrer · Pull Request #1038 · microsoft/botbuilder-python · GitHub
[go: up one dir, main page]

Skip to content

Support SSO with skill dialog and expected replies #1038

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

Merged
merged 9 commits into from
May 4, 2020

Conversation

tracyboehrer
Copy link
Member

token=TestAdapter.__EXCEPTION_EXPECTED,
)

self.exchangeable_tokens[key.to_key()] = key
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I noticed the same code in JavaScript; in JS you can use objects as keys Maps, so I might change the JS impl to do that (parity with C#).

I don't recall, but I believe there is something similar in Python...

Note, this isn't actionable for this PR, just potential future code cleanup.

)

# Check response status: true if success, false if failure
return response.status / 100 == 2
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

First time I see this, interesting. Is there no built in or popular function in python for this? i.e. dotnet has HttpStatusCodes and IsSuccessStatusCode() in a package that gets added everywhere anyways. If it requires adding a new package for this check, then it is not worth, but just checking.

@tracyboehrer tracyboehrer merged commit 3b3e7cd into master May 4, 2020
@tracyboehrer tracyboehrer deleted the trboehre/expectReplies branch May 4, 2020 20:14
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.

Add support to SkillDialog so SSO works with expectReply (Python)
5 participants
0