Skip to content

Fixing Oculus Deployment Issue, whitespace error#10584

Merged
RogPodge merged 3 commits into
microsoft:prerelease/2.8.0from
RogPodge:QoLFixes
May 16, 2022
Merged

Fixing Oculus Deployment Issue, whitespace error#10584
RogPodge merged 3 commits into
microsoft:prerelease/2.8.0from
RogPodge:QoLFixes

Conversation

@RogPodge

Copy link
Copy Markdown
Contributor

Overview

When deploying to Oculus the 618 error will still pop up, this adds that code to the csc to fix that issue.

Fixed some whitespace that was introduced accidentally.

@RogPodge

Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 2 pipeline(s).

Comment thread Assets/MRTK/SDK/Features/UX/Scripts/Pointers/BaseControllerPointer.cs Outdated
@@ -231,6 +231,7 @@ static void UpdateCSC()
// Empty as of Oculus Integration 39.0 https://developer.oculus.com/downloads/package/unity-integration/39.0

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this comment still accurate? I also still have an open comment (along with some others) about this on a previous PR: #10576 (comment)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Addressed the comments in the other PR.

@RogPodge

Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 2 pipeline(s).

@RogPodge RogPodge requested a review from keveleigh May 16, 2022 20:43
@RogPodge RogPodge merged commit 7bfc823 into microsoft:prerelease/2.8.0 May 16, 2022
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.

2 participants