Skip to content
This repository was archived by the owner on Jan 5, 2026. It is now read-only.

Conversation

@axelsrz
Copy link
Member

@axelsrz axelsrz commented Jun 14, 2019

This PR contains:

  • Added OAuth and OAuthPrompt functionality with tests
  • Removed duplicated token_api code from connector.
  • Bug fix in botframework-connector that was giving Unauthorized errors
  • TestAdapter and TestFlow now fully on par with C# and JS
  • Removed some unnecessary getters and setters from library classes

fixes #183
partially #186 , #174

@axelsrz axelsrz changed the title Support oauth and gov Support oauth and oauth prompt Jun 25, 2019
@axelsrz axelsrz marked this pull request as ready for review June 25, 2019 18:48
@axelsrz axelsrz requested a review from carlosscastro June 28, 2019 21:28
key.user_id = context.activity.from_property.id

if magic_code:
magic_code_record = list(filter(lambda x: key.equals_key(x.key), self._magic_codes))
Copy link
Member

Choose a reason for hiding this comment

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

consider a list comprehension?

inspector(reply, description)

else:
await asyncio.sleep(0.05)
Copy link
Member

Choose a reason for hiding this comment

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

sketchy?

@axelsrz axelsrz dismissed stevengum’s stale review July 3, 2019 00:50

changes addressed

@axelsrz axelsrz merged commit 6be1edb into master Jul 3, 2019
@axelsrz axelsrz deleted the axsuarez/support-oauth-and-gov branch July 3, 2019 00:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Port of OAuth functionality

5 participants