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

[PORT] Extend 'getProperty' builtin function to support accessing top memory. #2448

Closed
tomlm opened this issue Jun 30, 2020 · 0 comments · Fixed by #2437
Closed

[PORT] Extend 'getProperty' builtin function to support accessing top memory. #2448

tomlm opened this issue Jun 30, 2020 · 0 comments · Fixed by #2437

Comments

@tomlm
Copy link
Contributor

tomlm commented Jun 30, 2020

Port this change from botbuilder-dotnet/master branch:
microsoft/botbuilder-dotnet#4176

Fixes #4029
Originally, getProperty accepts two parameters, the first parameter is an instance, and the second parameter is the property name which user wants to access the value from. In this mode, the user can not access the top-level memory from dynamic/special string property.

After this change, getProperty accepts only one parameter, that presents the root property from the memory.

Example:
Scope is :

{
"a:b": "value"
}

getProperty('a:b') would get string "value",

Changed projects

  • AdaptiveExpressions
  • AdaptiveExpressions.Tests
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 a pull request may close this issue.

1 participant