Skip to content

Conversation

@Zerryth
Copy link
Contributor

@Zerryth Zerryth commented Jun 10, 2019

Fixes #185, #186, #197

Description

  • Adding missing prompts and choices-related classes into Python repo:

    • ActvityPrompt
    • AttachmentPrompt
    • ChoicePrompt (Work in Progress)
    • ChoiceRecognizers (Work in Progress)
    • Find
    • FindChoicesOptions
    • FindValuesOptions
    • FoundChoice
    • ModelResult
    • SortedValue
    • Token
    • Tokenizer
  • TurnContext also no longer will hard-code Activity.input_hint to always be acceptingInput--only when none is set, will it default to acceptingInput

@Zerryth
Copy link
Contributor Author

Zerryth commented Jun 14, 2019

Per chat with @axelsrz, going to abandon work on OAuthPrompt, as he will be working on that himself 👍

Zerryth added 22 commits June 20, 2019 19:34
@Zerryth
Copy link
Contributor Author

Zerryth commented Jul 3, 2019

@axelsrz I would move this out of Draft into actual full-fledged PR, since prompts and choices are finally ready!
...except I think I'll need your help untangling the conflicts that have surfaced after your PR merged earlier this evening

I'll talk to you in-person tomorrow :)

@Zerryth Zerryth marked this pull request as ready for review July 3, 2019 17:30
@Zerryth Zerryth requested a review from axelsrz July 3, 2019 17:30
Copy link
Member

@axelsrz axelsrz left a comment

Choose a reason for hiding this comment

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

Awesome!! just a couple of styling comments, but besides that this looks great.

@Zerryth Zerryth merged commit 969442f into master Jul 3, 2019
@Zerryth Zerryth deleted the Zerryth/DialogPrompts branch July 3, 2019 18:24
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.

Missing classes in Dialogs.Choices

3 participants