-
Notifications
You must be signed in to change notification settings - Fork 540
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 scenario samples to ensure match between templated and non-templated versions #4695
Conversation
…nd non-templated versions
@@ -160,7 +160,7 @@ | |||
"items": [ | |||
{ | |||
"type": "TextBlock", | |||
"text": "November 12, 2017", | |||
"text": "Sat, Mar 4, 2017", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't we have to update this textblocks to use {Date} formatting in the string? Seems like we're missing an opportunity here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@almedina-ms with @RebeccaAnne out can you pick up where she left off here to shepherd this change in? Thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should probably fix the date stuff later rather than hold up the release. I'll file a new issue.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
filed #4709
…nd non-templated versions (#4695) * [Samples] Update scenario samples to ensure match between templated and non-templated versions * Fix error * Fix WeatherLarge.json line endings Co-authored-by: Paul Campbell <paulcam@microsoft.com> Co-authored-by: shalinijoshi19 <shalinij@microsoft.com>
Related Issue
Fixes #4578
Description
Updated scenario samples to match the templated and non-templated versions.
How Verified
Ensured all modified samples rendered as expected.
Microsoft Reviewers: Open in CodeFlow