Skip to content

Conversation

@ImFirstPlace
Copy link
Contributor

@ImFirstPlace ImFirstPlace commented Jan 4, 2026

Changes:

  • Added a readme to the loader.
  • Modularised the settings.
  • Set settings.CreatorPowers default setting to false.
  • Support the old settings module

NOTE: The settings structure has not changed.

Benefits of Modularising the settings:

  • Makes settings a lot easier to configure.
  • Allows for better documentation of each settings category.
  • Allows people to go directly to the setting they want instead of scrolling in a huge file.

PoF: I have tested all cases on my side (see checklist below).

Test checklist:

  • Main module loads with the new settings modularisation.
  • Only modules that have errored get reset back to their default settings and wipes the ranks table.
  • Main module loads with the old settings module.
  • Old settings module error resets settings back to defaults.

Screenshots:

Description Image
Settings image
Modularised module error image
Old settings module error image
No settings folder or module found image

@ImFirstPlace ImFirstPlace mentioned this pull request Jan 4, 2026
@WalkerOfBacon
Copy link
Contributor

Will check this out later

@Dimenpsyonal
Copy link
Member

Guess who gave you merge conflicts

@ImFirstPlace
Copy link
Contributor Author

ImFirstPlace commented Jan 5, 2026

@Dimenpsyonal

Guess who gave you merge conflicts

Aww, but it looks like you forgot to remove lines 748-752 in Admin.luau
image

@Dimenpsyonal
Copy link
Member

Check again you silly minx

@ImFirstPlace
Copy link
Contributor Author

Hmm what am I missing?

@ImFirstPlace ImFirstPlace force-pushed the other/loader-settings-revamp branch from 7a56c10 to ddd9a4a Compare January 5, 2026 11:09
@ImFirstPlace
Copy link
Contributor Author

Removed a duplicate commit

@ImFirstPlace
Copy link
Contributor Author

@Dimenpsyonal I have fixed the merge conflict

@ImFirstPlace ImFirstPlace force-pushed the other/loader-settings-revamp branch 2 times, most recently from ddd9a4a to d5a659f Compare January 5, 2026 11:49
@ImFirstPlace
Copy link
Contributor Author

Continued here: #2050

@ImFirstPlace ImFirstPlace deleted the other/loader-settings-revamp branch January 5, 2026 12:21
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