Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

panden zuidoost in objectenbibliotheek + bij nieuwbouwpand voor zuidoost oude gebouwen verbergen #305

Open
wants to merge 22 commits into
base: main
Choose a base branch
from

Conversation

bozmir
Copy link
Contributor

@bozmir bozmir commented Oct 28, 2024

bozmir and others added 20 commits October 23, 2024 16:23
…e bagId list.

When this script becomes enabled it will hide all buildings with bagIds and vice versa
Controller hider script having a reference to the data  &
ZuidOoost prefab, when spawning hiding the buildings immediately
…ablibrary because its an hierarchical layer gameobject
…ated when loading hierarchical objects &

made a temporary runtime prefab list to not make the data polluted &
assets can now be loaded from assetbundles for the objectenbibliotheek &
assetbundle objects are now also added to the ui buttons of the objectenbibliotheek
Hierarchicalstaticobjectlayergameobject
Copy link
Contributor

@TomSimons TomSimons left a comment

Choose a reason for hiding this comment

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

Dit moet niet naar main maar naar de ATM branch toch?

Assets/Scripts/Projects/PrefabLibrary.cs Outdated Show resolved Hide resolved
Assets/Scripts/Projects/PrefabLibrary.cs Show resolved Hide resolved
}));
}

private void FixShadersForEditor(GameObject asset)
Copy link
Contributor

Choose a reason for hiding this comment

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

als dit alleen in de editor is, kan je er dan #if Unity_Editor omheen zetten?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

staat er al omheen?

@mvriel
Copy link
Contributor

mvriel commented Oct 29, 2024

Dit moet niet naar main maar naar de ATM branch toch?

Wel naar main, dit is zuidoost / migratie 3d amsterdam

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.

3 participants