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

Self setup #75

Merged
merged 2 commits into from
Jan 23, 2023
Merged

Self setup #75

merged 2 commits into from
Jan 23, 2023

Conversation

Qubus0
Copy link
Collaborator

@Qubus0 Qubus0 commented Jan 23, 2023

Use this startup flag to enable the self setup --script addons/mod_loader/mod_loader_setup.gd
Changes will apply after one restart. Setup was successful when the window name shows (Modded)
If the ModLoader autoload is already registered in the ProjectSettings, setup will be skipped - remove it when testing the setup.

As discussed before and in this Godot proposal #6137 here, the mod loader will end up as the last autoload when set up this way.
That is completely fine for all mods that do not modify autoloads.
Mods that do modify autoloads will not work properly. To use those mods, you will still have to decompile the game and add it as the first autoload manually.

salvaged from #33
closes #8

@Qubus0 Qubus0 added the enhancement New feature or request label Jan 23, 2023
Copy link
Collaborator

@ithinkandicode ithinkandicode left a comment

Choose a reason for hiding this comment

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

All looks good to me, I think I looked over this before and saw no issues

Fantastic work as always Ste :)

@Qubus0 Qubus0 merged commit 2d503e8 into GodotModding:develop Jan 23, 2023
@Qubus0 Qubus0 deleted the self_setup branch January 23, 2023 22:11
@KANAjetzt KANAjetzt mentioned this pull request Feb 15, 2023
This was referenced Feb 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants