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

Conversation

@Batta32
Copy link
Contributor

@Batta32 Batta32 commented May 4, 2021

Related to microsoft/botbuilder-java#1165

Proposed Changes

We compared the documentation/code migration/behavior of the 19.custom-dialogs sample between Java and C# and we found disparities and fixes that this PR includes.

  • Include number 16 as shoe size in RootDialog like C#
  • Remove duplicated endDialog execution like C#
  • Update README
  • Add missing comments
  • Clean-up autogenerated files/folder related to Eclipse (e.g. target/.classpath)
  • Add package-info.java file

Testing

Custom Dialogs sample working as expected
image

@Batta32 Batta32 requested a review from a team as a code owner May 4, 2021 19:57
@Batta32 Batta32 requested review from sgellock and tracyboehrer May 4, 2021 19:57
@tracyboehrer tracyboehrer merged commit a24af13 into microsoft:main May 5, 2021
@Batta32 Batta32 deleted the external/fix/southworks/java/samples/custom-dialogs branch May 5, 2021 13:39
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.

3 participants