Skip to content

Overhaul game settings with Kenyoni AppSettings integration#697

Merged
Spartan322 merged 1 commit into
masterfrom
overhaul/settings
Jun 26, 2026
Merged

Overhaul game settings with Kenyoni AppSettings integration#697
Spartan322 merged 1 commit into
masterfrom
overhaul/settings

Conversation

@Spartan322

@Spartan322 Spartan322 commented Jun 24, 2026

Copy link
Copy Markdown
Member

Implement startup music disable option
Replace GameSettings with Kenyoni AppSettings plugin
Remove the C++ GameSettings resource and registration
Add Godot-side Settings Autoload Globals:

  • GameSettings
  • ModSettings
  • Vic2Settings

Add mods load list to setting GUI
Add Victoria 2 base defines path to setting GUI
Refactor callers (GameStart, MusicManager, MapView, OptionsMenu, MainMenu) to use the new API
Add SettingsContainer UI to generate options dynamically
Update project.godot to register autoloads and enable the plugin
Update COPYRIGHT to attribute the new plugin files

@Spartan322

Copy link
Copy Markdown
Member Author

Should fix the Failed to generate any valid setting_optionbutton setting options. error by sidestepping the problem.

@Spartan322 Spartan322 force-pushed the overhaul/settings branch 15 times, most recently from 395cfde to 9082554 Compare June 26, 2026 17:20
Implement startup music disable option
Replace GameSettings with Kenyoni AppSettings plugin
Remove the C++ GameSettings resource and registration
Add Godot-side Settings Autoload Globals:
	- GameSettings
	- ModSettings
	- Vic2Settings
Add mods load list to setting GUI
Add Victoria 2 base defines path to setting GUI
Refactor callers (GameStart, MusicManager, MapView, OptionsMenu, MainMenu) to use the new API
Add SettingsContainer UI to generate options dynamically
Update project.godot to register autoloads and enable the plugin
Update COPYRIGHT to attribute the new plugin files

@Catylist0 Catylist0 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Tested - Seems to work 💯

@Spartan322 Spartan322 added this pull request to the merge queue Jun 26, 2026
Merged via the queue into master with commit 89ab619 Jun 26, 2026
11 checks passed
@Spartan322 Spartan322 deleted the overhaul/settings branch June 26, 2026 18:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants