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

[Samples]Update test apps samples #8052

Merged
merged 2 commits into from Nov 17, 2022
Merged

Conversation

anna-dingler
Copy link
Contributor

@anna-dingler anna-dingler commented Nov 14, 2022

Description

Update test apps to reference the correct file paths for samples.

Added v1.5 samples to the UWP Visualizer since all of the samples have been moved.

How Verified

Verified manually on the UWP Visualizer.

Outstanding Issue

Issue rendering the table element: #8051

Microsoft Reviewers: Open in CodeFlow

@ghost
Copy link

ghost commented Nov 14, 2022

Hi @anna-dingler. 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.

@beervoley
Copy link
Contributor

@anna-dingler - did you figure out the issue with the tableCell?

@anna-dingler
Copy link
Contributor Author

@anna-dingler - did you figure out the issue with the tableCell?

No, I wasn't able to figure it out. The crash occurs when we cast the tableCell to a Container.

I opened an issue for it #8051

@anna-dingler anna-dingler merged commit 367e91d into anna/addHeadings Nov 17, 2022
@anna-dingler anna-dingler deleted the anna/fixTestApps branch November 17, 2022 19:24
anna-dingler added a commit that referenced this pull request Nov 21, 2022
* Add headings to samples mentioned by accessibility team

* Add headings to v1.5 samples

* Remove duplicates

* Move remaining samples to v1.5

* [Samples]Update test apps samples (#8052)

* WIP: update project files for samples

* Update 1.5 references

* Disable scneario testing for .net

* Fix android visualizer (#8062)

* Resolve PR comments
rankush pushed a commit to rankush/AdaptiveCards that referenced this pull request May 8, 2024
* Add headings to samples mentioned by accessibility team

* Add headings to v1.5 samples

* Remove duplicates

* Move remaining samples to v1.5

* [Samples]Update test apps samples (microsoft#8052)

* WIP: update project files for samples

* Update 1.5 references

* Disable scneario testing for .net

* Fix android visualizer (microsoft#8062)

* Resolve PR comments
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.

None yet

3 participants