Skip to content

Fix accessing threadbare_project_settings in non-editor builds#2139

Merged
manuq merged 1 commit intomainfrom
wjt/fix-accessing-threadbare-project-settings-in-non-editor-builds
Apr 21, 2026
Merged

Fix accessing threadbare_project_settings in non-editor builds#2139
manuq merged 1 commit intomainfrom
wjt/fix-accessing-threadbare-project-settings-in-non-editor-builds

Conversation

@wjt
Copy link
Copy Markdown
Member

@wjt wjt commented Apr 21, 2026

Previously ThreadbareProjectSettings extended EditorPlugin, which means
it is not available in release builds. As a result any code which
accessed ThreadbareProjectSettings did not work. This includes the
splash screen, main menu, and (transitively) the story book.

Extend Node instead, and move the actual plugin piece to a separate
script. Also fix an excessive use of the letter 't' in the word
'setttings'.

Resolves #2138

Previously ThreadbareProjectSettings extended EditorPlugin, which means
it is not available in release builds. As a result any code which
accessed ThreadbareProjectSettings did not work. This includes the
splash screen, main menu, and (transitively) the story book.

Extend Node instead, and move the actual plugin piece to a separate
script. Also fix an excessive use of the letter 't' in the word
'setttings'.

Resolves #2138
@wjt wjt requested a review from a team as a code owner April 21, 2026 14:21
@wjt
Copy link
Copy Markdown
Member Author

wjt commented Apr 21, 2026

(Found while testing #2134.)

@github-actions
Copy link
Copy Markdown

Play this branch at https://play.threadbare.game/branches/endlessm/wjt/fix-accessing-threadbare-project-settings-in-non-editor-builds/.

(This launches the game from the start, not directly at the change(s) in this pull request.)

Copy link
Copy Markdown
Collaborator

@manuq manuq left a comment

Choose a reason for hiding this comment

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

Thanks for the fix.

const SKIP_SPLASH = "threadbare/debugging/skip_splash"

static var setttings_configuration = {
static var settings_configuration = {
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Typo fixed even!

@manuq
Copy link
Copy Markdown
Collaborator

manuq commented Apr 21, 2026

"an excessive use of the letter 't' " 💯

@manuq manuq merged commit ec0b9c7 into main Apr 21, 2026
6 checks passed
@manuq manuq deleted the wjt/fix-accessing-threadbare-project-settings-in-non-editor-builds branch April 21, 2026 16:40
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.

ThreadbareProjectSettings parser error in exported game

2 participants