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

Add a setting to enable mod manager and disable it by default #2969

Closed
wants to merge 3 commits into from

Conversation

red-001
Copy link
Contributor

@red-001 red-001 commented Jul 29, 2015

Currently mod manager seems to be broken.

@est31
Copy link
Contributor

est31 commented Jul 29, 2015

The defaults for settings are managed in defaultsettings.cpp

@@ -603,4 +603,5 @@
# Comma-separated list of trusted mods that are allowed to access insecure
# functions even when mod security is on (via request_insecure_environment()).
#secure.trusted_mods =

# enable mod manager
Copy link
Member

Choose a reason for hiding this comment

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

This is the wrong place, it's a client setting. Put it next to the other main menu settings (such as mainmenu_last_idx or something)

@est31
Copy link
Contributor

est31 commented Aug 24, 2015

Not needed anymore, since d565ce9
I don't think we need a setting for this.

@est31 est31 closed this Aug 24, 2015
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