Windowstemplatestudio: Should Order.cs from SampleDataService be more clear it is a sample data

Created on 23 Jul 2017  路  2Comments  路  Source: microsoft/WindowsTemplateStudio

Overview: in reference to #870, should Order.cs be actually "SampleOrder.cs" or some way to more clearly outline it is a sample/mock class.

Can Close Out Soon question

Most helpful comment

The linked issue was about some text that was actually wrong. This is more subjective. As there's already a comment in there that says it's meant to be deleted as it only exists for the sample data I'd think that's enough but as you're asking the question it demonstrates that not everyone thinks the same way on this. In that case, yes, change it. Making the change will be quicker than any further discussion and I can't see a downside to doing so.

All 2 comments

The linked issue was about some text that was actually wrong. This is more subjective. As there's already a comment in there that says it's meant to be deleted as it only exists for the sample data I'd think that's enough but as you're asking the question it demonstrates that not everyone thinks the same way on this. In that case, yes, change it. Making the change will be quicker than any further discussion and I can't see a downside to doing so.

verified in dev-nightly
wizardVersion="0.14.17228.65001"
templatesVersion="0.14.17228.1"

Was this page helpful?
0 / 5 - 0 ratings