Skip to content

Conversation

@DrymarchonShaun
Copy link
Contributor

No description provided.

@Pyrolyzed
Copy link

Pyrolyzed commented Feb 8, 2025

This doesn't work for me, the issue seems to be you aren't calling write_setting() in Settings::save_settings_from_ui.

Edit: Yes, adding the write_setting function fixes it.

@DrymarchonShaun
Copy link
Contributor Author

Oops, missed that bit when I was copying everything over from my PR for the RF cdlc. Should be fixed.

@Pyrolyzed
Copy link

Awesome! Hopefully @muttleyxd merges the PR then, this is definitely a needed feature.

@muttleyxd muttleyxd merged commit 7d4bcb1 into muttleyxd:master Feb 10, 2025
10 checks passed
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