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

Revert "Update OAuthPrompt.cs" - #1887

Closed
mingweiw wants to merge 1 commit into
masterfrom
revert-1579-patch-1
Closed

Revert "Update OAuthPrompt.cs"#1887
mingweiw wants to merge 1 commit into
masterfrom
revert-1579-patch-1

Conversation

@mingweiw

@mingweiw mingweiw commented May 9, 2019

Copy link
Copy Markdown
Contributor

Reverts #1579

@fuselabs

fuselabs commented May 9, 2019

Copy link
Copy Markdown
Collaborator

✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.dll

✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.AI.Luis.dll

✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.AI.QnA.dll

✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.ApplicationInsights.dll

✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.Azure.dll

✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.Dialogs.dll

✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.TemplateManager.dll

✔️ No Binary Compatibility issues for Microsoft.Bot.Configuration.dll

✔️ No Binary Compatibility issues for Microsoft.Bot.Connector.dll

✔️ No Binary Compatibility issues for Microsoft.Bot.Schema.dll

@cleemullins

Copy link
Copy Markdown
Contributor

@mingweiw Can you add more detail here around "Why revert?" Why do we need to revert? Should we do anything else in place of the code we're reverting?

The original changes were made and verified with @Jeffders, and if they're wrong we should figure out what to do.

@cleemullins cleemullins self-assigned this May 10, 2019
@cleemullins
cleemullins requested a review from Jeffders May 10, 2019 18:59
@cleemullins

Copy link
Copy Markdown
Contributor

Also, if we're reverting this here - what about the JS (any Python) codebase?

@benbrown

Copy link
Copy Markdown
Contributor

We need to create a new issue to track this change in both languages, etc.

@cleemullins

Copy link
Copy Markdown
Contributor

In REviewing all of Jeff's recent email w/ teams, the code that's there appears to be correct. I don't think this PR is needed.

@cleemullins
cleemullins deleted the revert-1579-patch-1 branch September 12, 2019 22: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.

4 participants