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

Fix compatibility with ONI: Spaced Out version 496912 #8

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ThaRedstoner
Copy link

@ThaRedstoner ThaRedstoner commented Mar 1, 2022

Hi, I've updated your ExtendedBridges mod to support ONI: Spaced Out version 496912.
It seems that this update broke all mods that tried to inject their buildings into the plan screen at specific positions (after specific other buildings, for instance). Including this one.
I'm not really sure if there's still a way to do it like that, but for now on recommendation of Klei, I've made it use ModUtil.AddBuildingToPlanScreen, which should make sure this doesn't break again.

I've included a prebuilt binary for the mod itself, but I didn't update any of the other dlls included in the repository. I would recommend you to remove and/or git ignore them from your public repository as well, as any contributor should simply link those files themselves. (Since the dlls aren't meant to be uploaded, they're still property of their respective owners, like Klei and Unity)

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.

1 participant