Skip to content
This repository was archived by the owner on Aug 11, 2024. It is now read-only.

Implement BoundaryVisibility#822

Merged
StephenHodgson merged 5 commits intodevelopmentfrom
feature/boundary-visibility
Apr 30, 2021
Merged

Implement BoundaryVisibility#822
StephenHodgson merged 5 commits intodevelopmentfrom
feature/boundary-visibility

Conversation

@FejZa
Copy link
Contributor

@FejZa FejZa commented Apr 23, 2021

XRTK - Mixed Reality Toolkit Pull Request

Overview

Added a new enum type BoundaryVisibility. Mostly to be able to specify the BoundaryVisibility.Unknown state, which is useful when we don't know what the current visibility state is.

Breaking Changes

  • New IMixedRealityBoundaryDataProvider.BoundaryVisibility property instead of bool type

Related Submodule Changes

XRTK/com.xrtk.wmr#132

@FejZa FejZa self-assigned this Apr 23, 2021
@FejZa FejZa added the Ready for review PR finished primary development, open for review label Apr 23, 2021
@FejZa
Copy link
Contributor Author

FejZa commented Apr 26, 2021

/azp run

@azure-pipelines
Copy link
Contributor

Azure Pipelines successfully started running 1 pipeline(s).

@StephenHodgson
Copy link
Contributor

/azp run

@azure-pipelines
Copy link
Contributor

Azure Pipelines successfully started running 1 pipeline(s).

@StephenHodgson
Copy link
Contributor

Not sure what's up with this Gradle daemon error. maybe it has something to do with the new VMs?

@FejZa
Copy link
Contributor Author

FejZa commented Apr 27, 2021

/azp run

@azure-pipelines
Copy link
Contributor

Azure Pipelines successfully started running 1 pipeline(s).

@FejZa
Copy link
Contributor Author

FejZa commented Apr 27, 2021

/azp run

@azure-pipelines
Copy link
Contributor

Azure Pipelines successfully started running 1 pipeline(s).

@StephenHodgson
Copy link
Contributor

/azp run

@azure-pipelines
Copy link
Contributor

Azure Pipelines successfully started running 1 pipeline(s).

@StephenHodgson
Copy link
Contributor

/azp run

@azure-pipelines
Copy link
Contributor

Azure Pipelines successfully started running 1 pipeline(s).

@StephenHodgson StephenHodgson merged commit 611e1e4 into development Apr 30, 2021
@StephenHodgson StephenHodgson deleted the feature/boundary-visibility branch April 30, 2021 02:06
XRTK-Build-Bot pushed a commit that referenced this pull request Apr 30, 2021
* Implement BoundaryVisibility

* updated submodules

updated packages

Co-authored-by: Stephen Hodgson <hodgson.designs@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Breaking Change Ready for review PR finished primary development, open for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants