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

Added guid regeneration for installed package profiles to prevent asset collisions #562

Merged
merged 4 commits into from
May 4, 2020

Conversation

StephenHodgson
Copy link
Contributor

XRTK - Mixed Reality Toolkit Change Request

Overview

To prevent asset collisions and problems when copying prefabs and profiles, I thought it'd be best to regenerate any guid references in the copied assets.

@StephenHodgson StephenHodgson added Enhancement New feature or request In Progress PR currently still being developed labels May 4, 2020
@StephenHodgson StephenHodgson marked this pull request as ready for review May 4, 2020 01:14
@StephenHodgson StephenHodgson added Ready for review PR finished primary development, open for review and removed In Progress PR currently still being developed labels May 4, 2020
@StephenHodgson StephenHodgson mentioned this pull request May 4, 2020
36 tasks
@SimonDarksideJ
Copy link
Contributor

Does this also solve the issue of asset referencing when the profiles are copied?

@StephenHodgson
Copy link
Contributor Author

Yes, that's the whole point

@StephenHodgson StephenHodgson merged commit ea69ed3 into development May 4, 2020
@StephenHodgson StephenHodgson deleted the feature/profile-guid-regeneration branch May 4, 2020 12:38
XRTK-Build-Bot pushed a commit that referenced this pull request Dec 25, 2020
…et collisions (#562)

* Added guid regeneration for installed package profiles to prevent asset collisions

* get full path to destination folder for guid regeneration

* catch null instance type references and log it

Co-authored-by: Dino Fejzagic <dino.f@live.de>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Enhancement New feature or request Ready for review PR finished primary development, open for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants