Skip to content
This repository was archived by the owner on Jul 9, 2025. It is now read-only.

fix: lg file mess up#3141

Merged
cwhitten merged 10 commits intomasterfrom
zhixzhan/lgremove
May 26, 2020
Merged

fix: lg file mess up#3141
cwhitten merged 10 commits intomasterfrom
zhixzhan/lgremove

Conversation

@zhixzhan
Copy link
Contributor

@zhixzhan zhixzhan commented May 20, 2020

Description

Task Item

close #3136
close #3168

Screenshots

Copy link
Contributor

@boydc2014 boydc2014 left a comment

Choose a reason for hiding this comment

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

Please also update the lg package to fix the parsing issue.

@zhixzhan
Copy link
Contributor Author

Please also update the lg package to fix the parsing issue.

Sure, there are some upgrade issue haven't resolve, added WIP tag to hold this PR.

@boydc2014
Copy link
Contributor

Please also update the lg package to fix the parsing issue.

Sure, there are some upgrade issue haven't resolve, added WIP tag to hold this PR.

FYI, the js fix is in master, you should be unblocked now

@zhixzhan zhixzhan requested a review from yeze322 as a code owner May 26, 2020 00:41
@zhixzhan zhixzhan force-pushed the zhixzhan/lgremove branch from 22d6e15 to 35847e5 Compare May 26, 2020 00:51
@zhixzhan
Copy link
Contributor Author

Please also update the lg package to fix the parsing issue.

Sure, there are some upgrade issue haven't resolve, added WIP tag to hold this PR.

FYI, the js fix is in master, you should be unblocked now

updated.

@cwhitten cwhitten merged commit 17f23d2 into master May 26, 2020
@cwhitten cwhitten deleted the zhixzhan/lgremove branch May 26, 2020 23:18
lei9444 pushed a commit to lei9444/BotFramework-Composer-1 that referenced this pull request Jun 15, 2021
* do not remove

* install latest lgparser

* upgrade Expression package

* fix atob in worker

* remove dead code

* update lg package

Co-authored-by: Dong Lei <donglei@microsoft.com>
Co-authored-by: Chris Whitten <christopher.whitten@microsoft.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Approved to merge approved, waiting to be merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Error when trying to use isArray() within an expression LG parse may mess up the content in lg file

3 participants