search

XRTK / com.xrtk.core

The Official Mixed Reality Framework for Unity
https://xrtk.io
MIT License
309 stars 34 forks source link

Backwards compatibility broken for existing cloned profiles #491

Closed SimonDarksideJ closed 4 years ago

SimonDarksideJ commented 4 years ago

XRTK - Mixed Reality Toolkit Bug Report

Describe the bug

Updated Core and SDK for XRTK, this broken existing local Controller Mapping profiles

To Reproduce

  1. Use earlier version of XRTK (e.g. 0.1.29)
  2. Clone profiles
  3. Make changes to configuration
  4. Upgrade to 0.1.32
  5. Shazzam

Scope

Documentation update

Expectation

Considering the scope of profiles, I feel this just a documentation fix/note for an upgrade, that existing profiles need to be re-cloned and configured. Rather than effort to resolve the backwards compatibility

StephenHodgson commented 4 years ago

I think we already knew this with all the changes coming down the line. Even more to come.

StephenHodgson commented 4 years ago

At this point let's just assume that all existing profiles can be broken at any time unless they're from the sdk directly.

StephenHodgson commented 4 years ago

Why mark it as a bug if you feel like it's not worth the effort of supporting backwards compatibility?

StephenHodgson commented 4 years ago

I'm going to close this as adding the Breaking Changes labels to the PRs likely means that profiles need to be regenerated. Let's just assume that we will be doing this until version 1.x