MicrosoftDocs / mixed-reality

Mixed Reality documentation
Creative Commons Attribution 4.0 International
174 stars 184 forks source link

replace wrong procedures + minor typos #689

Closed ErrorRaffyline0 closed 1 year ago

ErrorRaffyline0 commented 1 year ago

Fixes info about tracking issues. Reapplies capitalization on mixed reality.

prmerger-automator[bot] commented 1 year ago

@ErrorRaffyline0 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

ErrorRaffyline0 commented 1 year ago

@vtieto callback to #615

opbld32 commented 1 year ago

Learn Build status updates of commit 8ae6133:

:warning: Validation status: warnings

File Status Preview URL Details
enthusiast-guide/headset-display.md :warning:Warning Details

enthusiast-guide/headset-display.md

For more details, please refer to the build report.

If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Learn Build.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

ErrorRaffyline0 commented 1 year ago

Sorry if the reapplication of capitals on Mixed Reality was the wrong call, but seeing from the rest of the documentation it made more sense to keep it capitalized.

prmerger-automator[bot] commented 1 year ago

@ErrorRaffyline0 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

ErrorRaffyline0 commented 1 year ago

Formatting still needs to be addressed, but I'm not gonna try that as I'd still be here a week after.

opbld31 commented 1 year ago

Learn Build status updates of commit ee60c41:

:warning: Validation status: warnings

File Status Preview URL Details
enthusiast-guide/headset-display.md :warning:Warning Details

enthusiast-guide/headset-display.md

For more details, please refer to the build report.

If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Learn Build.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

ErrorRaffyline0 commented 1 year ago

Sorry if the reapplication of capitals on Mixed Reality was the wrong call, but seeing from the rest of the documentation it made more sense to keep it capitalized.

New commit reverts this because I was stupid for changing this in the first place.

ErrorRaffyline0 commented 1 year ago

@qianw211 plz fix formatting and merge

prmerger-automator[bot] commented 1 year ago

@ErrorRaffyline0 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

opbld33 commented 1 year ago

Learn Build status updates of commit be26fbe:

:white_check_mark: Validation status: passed

File Status Preview URL Details
enthusiast-guide/headset-display.md :white_check_mark:Succeeded

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

ErrorRaffyline0 commented 1 year ago

Hey there @qianw211 thanks for the merge, but there is a part that I added that has poor formatting, I was hoping you could fix that since I'm very bad at markdown formatting...

qianw211 commented 1 year ago

Hi @ErrorRaffyline0, thank you for updating the content. I think the formatting looks fine: https://learn.microsoft.com/en-us/windows/mixed-reality/enthusiast-guide/headset-display#the-images-shown-on-the-headsets-displays-appear-warped-or-incorrectly-scaled

We have limited writing resources, and this page is on our revamping work list. Please look for updates in the near future. Meanwhile, we're trying to minimize work done in this part of the documentation that's not the highest priority for us. Thanks for your understanding.