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

Move to latest versions of SDK #2378

Merged
merged 9 commits into from
May 12, 2020
Merged

Move to latest versions of SDK #2378

merged 9 commits into from
May 12, 2020

Conversation

chrimc62
Copy link
Contributor

This updates to the latest SDK packages including bug fixes to LG and LU.
For LG, allow # in multi-line.
For LU, set line numbers for entities.

@chrimc62 chrimc62 requested a review from coldplaying42 May 12, 2020 15:34
Add templates to package.
@chrimc62 chrimc62 requested a review from tomlm May 12, 2020 17:05
@@ -5,15 +5,17 @@
"author": "Microsoft",
"bugs": "https://github.com/Microsoft/BotBuilder-Samples/issues",
"dependencies": {
"@microsoft/bf-cli-command": "^4.8.0",
"@microsoft/bf-cli-command": "^4.9.0-RC6",
"@microsoft/bf-lu": "^4.10.0-preview.130225",
Copy link
Contributor

@tomlm tomlm May 12, 2020

Choose a reason for hiding this comment

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

"^4.10.0-preview.130225", [](start = 24, length = 25)

this doesn't seem right. #Resolved

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That is correct. I have to pick up bug fixes.


In reply to: 423897684 [](ancestors = 423897684)

@@ -5,15 +5,17 @@
"author": "Microsoft",
"bugs": "https://github.com/Microsoft/BotBuilder-Samples/issues",
"dependencies": {
"@microsoft/bf-cli-command": "^4.8.0",
"@microsoft/bf-cli-command": "^4.9.0-RC6",
Copy link
Contributor

@tomlm tomlm May 12, 2020

Choose a reason for hiding this comment

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

", [](start = 44, length = 2)

this is no rc42? #Resolved

@@ -5,7 +5,7 @@

# template
- ```
# booleanEntity(value)
${'#'} booleanEntity(value)
Copy link
Contributor

@tomlm tomlm May 12, 2020

Choose a reason for hiding this comment

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

${'#' [](start = 0, length = 5)

what the wha?
#Resolved

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Bug in LG that they are working on fixing and this is the work-around until they do.


In reply to: 423898362 [](ancestors = 423898362)


# ValueTemplate
- ```
# ${property}(values)
${'#'} ${property}(values)
Copy link
Contributor

@tomlm tomlm May 12, 2020

Choose a reason for hiding this comment

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

${'#'} [](start = 0, length = 7)

can you # ? #Resolved

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No. That then passes through as #


In reply to: 423898807 [](ancestors = 423898807)

Copy link
Contributor

@tomlm tomlm left a comment

Choose a reason for hiding this comment

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

🕐

Copy link
Contributor

@tomlm tomlm left a comment

Choose a reason for hiding this comment

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

:shipit:

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.

2 participants