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

[Site] Fix missing expression library dep #5194

Merged
merged 2 commits into from
Dec 17, 2020
Merged

Conversation

golddove
Copy link
Member

@golddove golddove commented Dec 17, 2020

Related Issue

Fixes #5193

Description

Explicitly include adaptive-expression

How Verified

Manually, using test site below.

Microsoft Reviewers: Open in CodeFlow

@ghost
Copy link

ghost commented Dec 17, 2020

Hi @golddove. Thanks for helping make the AdaptiveCards JS renderer + tooling better. As additional verification, once the JS build succeeds, please go to the test site to test out your website/designer changes.

@golddove golddove merged commit c7a27e8 into main Dec 17, 2020
@golddove golddove deleted the golddove/templating-version branch December 17, 2020 06:18
golddove added a commit that referenced this pull request Dec 17, 2020
* expression library rc to 4.11.0

* Add expression library to site footer scripts
golddove added a commit that referenced this pull request Dec 17, 2020
* expression library rc to 4.11.0

* Add expression library to site footer scripts
jwoo-msft pushed a commit that referenced this pull request Jan 20, 2021
* expression library rc to 4.11.0

* Add expression library to site footer scripts
jwoo-msft added a commit that referenced this pull request Jan 23, 2021
* Updated ImageView constraints

* updated intrinsic content size

* reverted build optimazation settings

* [Site] Fix missing expression library dep (#5194)

* expression library rc to 4.11.0

* Add expression library to site footer scripts

* Updating Readme to reflect binary license updates

With our 20.12 release we are fixing a long running bug, where we were shipping our binary packages under the MIT license inadvertently. The MIT license only applies to open source and not shipping binaries which are now governed by our new End User License Agreement for AdaptiveCards.

* Quick verbiage change in the EULA section

* More license terms explanation verbiage

* [.Net HTML]fix IgnoreDefaultStringEnumConverter NullReference issue (#4833)

## Related Issue
Fixes #4831

## Description
Avoid NullReferenceException while the enum is null in JSON string
![Issue](https://user-images.githubusercontent.com/3928402/94417848-2f57f780-01b3-11eb-88d7-38eb24e02457.png)

* columnset alignment to fill, and added spacer for factset

* work in progress

* work in progress

* Mostly work except background images

* removed the wrapperview

* refactoring

* refactoring complete // need unit tests and optimazation

* fixed stretch issues

* Finished fixing small/medium/large images sizes

* Added UnitTests

* cleaned up the code

* complete

* fixed ci failures

* revert project file changes

* chaged made after review comments

Co-authored-by: Risheek Rajolu <risheekrr@gmail.com>
Co-authored-by: Shalini Joshi-MSFT <shalinij@microsoft.com>
Co-authored-by: Tony <lxtx.ljb@163.com>
rankush pushed a commit to rankush/AdaptiveCards that referenced this pull request May 8, 2024
* Updated ImageView constraints

* updated intrinsic content size

* reverted build optimazation settings

* [Site] Fix missing expression library dep (microsoft#5194)

* expression library rc to 4.11.0

* Add expression library to site footer scripts

* Updating Readme to reflect binary license updates

With our 20.12 release we are fixing a long running bug, where we were shipping our binary packages under the MIT license inadvertently. The MIT license only applies to open source and not shipping binaries which are now governed by our new End User License Agreement for AdaptiveCards.

* Quick verbiage change in the EULA section

* More license terms explanation verbiage

* [.Net HTML]fix IgnoreDefaultStringEnumConverter NullReference issue (microsoft#4833)

## Related Issue
Fixes microsoft#4831

## Description
Avoid NullReferenceException while the enum is null in JSON string
![Issue](https://user-images.githubusercontent.com/3928402/94417848-2f57f780-01b3-11eb-88d7-38eb24e02457.png)

* columnset alignment to fill, and added spacer for factset

* work in progress

* work in progress

* Mostly work except background images

* removed the wrapperview

* refactoring

* refactoring complete // need unit tests and optimazation

* fixed stretch issues

* Finished fixing small/medium/large images sizes

* Added UnitTests

* cleaned up the code

* complete

* fixed ci failures

* revert project file changes

* chaged made after review comments

Co-authored-by: Risheek Rajolu <risheekrr@gmail.com>
Co-authored-by: Shalini Joshi-MSFT <shalinij@microsoft.com>
Co-authored-by: Tony <lxtx.ljb@163.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants