Skip to content
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

Added Node and C# AdaptiveCardPrompt Samples #1986

Closed
wants to merge 10 commits into from

Conversation

mdrichardson
Copy link
Contributor

@mdrichardson mdrichardson commented Nov 15, 2019

Reference: microsoft/botframework-sdk#5396

Do not review AdaptiveCardPrompt.ts/cs. It will change. Please only review "bot code" for now, if anything.

...I'll have a separate PR for those and merge the finalized code into the sample (or it might be in the SDK by then)

Additional Details

Proposed Changes

Adds C# and Node Samples for using the un-merged AdaptiveCardPrompt

I numbered these Sample "09" because it seemed to make the most sense, but I'm very open to changing that, as well as the name.

These mostly follow the flow of current samples, but I originally created them in May, so I may have missed some design changes.

At some point, I may update this to show all of the functionality, but this is a good start while that gets fleshed out.

Notes

Both of these have class files for AdaptiveCardPrompt that are directly part of the sample (not in the SDK), since this is meant to be merged before we merge in AdaptiveCardPrompt

C# AdaptiveCardPrompt relies on a couple of pieces of the Dialogs package that are internal. It's modified a little in the sample so that the sample can stand on its own.

Node AdaptiveCardPrompt is a compiled version of adaptiveCardPrompt.ts and then edited a little for readability.

Testing

image

@mdrichardson
Copy link
Contributor Author

Closing now that AdaptiveCardPrompt is in the Community Repos

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.

1 participant